tag 501114 patch pending thanks Martin Zobel-Helas <zo...@ftbfs.de> (19/01/2009): > I just tried to build it with CFLAGS=-g -O0 and the test-suite didn't > fail any more.
I've also tested it with CFLAGS="-O1 -g" (that's when DEB_BUILD_OPTIONS contains noopt), and the test suite breaks as well. So please find attached a patch that: - make CFLAGS="-O0" the default for sparc. - make CFLAGS="-O0 -g" the noopt for sparc. - continue using no CFLAGS as the default for other archs. - continue using CFLAGS="-O1 -g" as the noopt for other archs. I've tested both builds on sparc and it sounds like the patch does its job. There might be probably more elegant ways to write it but at least the comments in debian/rules should be quite clear. I intend to NMU with the attached patch RSN, until objections arise. Possibly later today to DELAYED/1. I know #497740 remains for hppa, but that's another story, and as suggested by a release manager, 1 FTBFS is better than 2. ;) Mraw, KiBi.
diff -u guile-1.6-1.6.8/debian/rules guile-1.6-1.6.8/debian/rules --- guile-1.6-1.6.8/debian/rules +++ guile-1.6-1.6.8/debian/rules @@ -55,7 +55,22 @@ # http://lists.gnu.org/archive/html/guile-devel/2006-10/msg00007.html # ifneq (,$(findstring noopt,$(DEB_BUILD_OPTIONS))) +# However, using -O1 makes the test suite fail (#501114) on sparc. +# Use -O0 there. +ifneq ($(DEB_HOST_ARCH),sparc) configure_args += CFLAGS="-O1 -g" +else + configure_args += CFLAGS="-O0 -g" +endif +endif + +# Use -O0 on sparc to work around a failure in the test suite with the +# default -O2 (see #501114). +ifeq ($(DEB_HOST_ARCH), sparc) +# Don't set CFLAGS if it was already set because of noopt. +ifeq (,$(findstring CFLAGS, $(configure_args))) + configure_args += CFLAGS="-O0" +endif endif # It looks like Debian's packaging tools can't handle an empty file, diff -u guile-1.6-1.6.8/debian/changelog guile-1.6-1.6.8/debian/changelog --- guile-1.6-1.6.8/debian/changelog +++ guile-1.6-1.6.8/debian/changelog @@ -1,3 +1,13 @@ +guile-1.6 (1.6.8-6.2) unstable; urgency=low + + * Non-maintainer upload. + * Work around a possible compiler bug which leads to FTBFS on sparc, by + setting CFLAGS=-"O0" for that architecture. Also switch the previous + CFLAGS="-01 -g" to CFLAGS="-O0 -g" for the noopt build on sparc, for + the same reason (Closes: #501114). + + -- Cyril Brulebois <k...@debian.org> Mon, 19 Jan 2009 16:48:23 +0100 + guile-1.6 (1.6.8-6.1) unstable; urgency=high * Non-maintainer upload.
signature.asc
Description: Digital signature
