On 26 June 2007 at 21:45, Riku Voipio wrote: | Package: quantlib | Version: 0.8.1-1 | Severity: wishlist | Tags: patch | User: [EMAIL PROTECTED] | Usertags: eabi | | Hi, | | While compiling quantlib on armel[1], I noticed it was triggering testsuite | and benchmark runs, while arm and m68k ports did not seem to. After a | quick glance on the debian/rules, I think it's better to use | DEB_HOST_GNU_CPU to identify what systems are slow - after all someone | my start using knetbsd-arm some day. armel port also returns arm from | DEB_HOST_GNU_CPU. This patch also adds armeb to slow cpu list.
Yup, much appreciated -- I think I have the same issue on other debian/rules where rather expensive tests are supposed to be suppressed, but aren't b/c I didn't update arch / cpu variable(s). So thanks for the hint. Will go into QL 0.9.0 whenever it comes, or an earlier bugfix if needed. Now, while I have your undivided attention: shall we block some arches from building QL? I see it using vast compiler resources during builds (all those C++ templates) and it is just not meant for 'small' systems (for various definitions of small). Any thoughts? Thanks, Dirk | | Thanks, | Riku | | ps. | | Definetly no need to hurry for applying this, just whenever you are going | to upload anyway ;) | | [1] http://experimental.debian.net/build.php?arch=armel&pkg=quantlib | | -- | "rm -rf" only sounds scary if you don't have backups | --- quantlib-0.8.1.old/debian/rules 2007-06-26 21:29:31.000000000 +0300 | +++ quantlib-0.8.1/debian/rules 2007-06-26 21:29:15.000000000 +0300 | @@ -13,6 +13,7 @@ | debtmp := $(CURDIR)/debian/$(libpkg) | libdoc := $(CURDIR)/debian/$(libpkg)/usr/share/doc/$(libpkg) | arch := $(shell dpkg-architecture -qDEB_HOST_GNU_TYPE) | +cpu := $(shell dpkg-architecture -qDEB_HOST_GNU_CPU) | | #export DH_VERBOSE=1 | | @@ -37,7 +38,7 @@ | | ## edd 18 May 2002 no optimisation or debugging on baby systems | ## edd 14 May 2005 don't do it on mipsel or mips either | -ifneq "$(findstring $(arch), m68k-linux-gnu arm-linux-gnu mipsel-linux-gnu mips-linux-gnu)" "" | +ifneq "$(findstring $(cpu), m68k arm armeb mipsel mips)" "" | compilerflags = -O0 -g0 -D_REENTRANT -fpermissive | endif | | @@ -103,7 +104,7 @@ | | test: test-stamp | test-stamp: build-stamp | -ifeq "$(findstring $(arch), m68k-linux-gnu arm-linux-gnu)" "" | +ifeq "$(findstring $(cpu), m68k arm armeb)" "" | -LD_LIBRARY_PATH=$(CURDIR)/ql/.libs $(MAKE) check | -LD_LIBRARY_PATH=$(CURDIR)/ql/.libs $(MAKE) benchmark | endif -- Hell, there are no rules here - we're trying to accomplish something. -- Thomas A. Edison -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]
