tags 475952 patch thanks Hi,
It appears that the intention of the maintainer was to build on most (all?) architectures with -O0 to avoid problems (as there is a longish trail of icon bugs on various architectures, related to its inability to work properly when compiled with -O2). However, due to a wrong sed expression in configure target the CFLAGS setting to include -O0 never happened. The attached patch fixes the issue, and I confirmed that it builds successfully on sparc with it. Please let me know what you think, if I'll not hear anything by the time I get back from Debconf (early next week), I'll NMU it. Cheers. -- Jurij Smakov [EMAIL PROTECTED] Key: http://www.wooyd.org/pgpkey/ KeyID: C99E03CC
diff -aur icon-9.4.3.old/debian/rules icon-9.4.3/debian/rules --- icon-9.4.3.old/debian/rules 2008-08-15 02:22:14.000000000 +0000 +++ icon-9.4.3/debian/rules 2008-08-15 02:21:39.000000000 +0000 @@ -48,7 +48,7 @@ configure: $(checkdir) $(MAKE) X-Configure name=$(system) - sed "s/CFLAGS = -O2/CFLAGS = $(CFLAGS)/" Makedefs > Makedefs.tmp + sed "s/CFLAGS = .*/CFLAGS = $(CFLAGS)/" Makedefs > Makedefs.tmp mv Makedefs.tmp Makedefs touch configure
