-z: linker input file unused because linking not done

[Andreas Metzler]

On 2006-11-09 Fabian Fagerholm <fabbe at paniq.net> wrote:
> On Wed, 2006-11-08 at 20:32 +0100, Andreas Metzler wrote:
> > currently the buildlog contains tons of
> [...]
> > I guess moving "-Wl,-z,defs" from CFLAGS to LDFLAGS might solve this.

> Yup, right you are. I changed it in debian/rules.

Hej,
thanks for the quick fix, the change applied seems to be incorrect
though, as it drops any setting of CFLAGS, I think this should be
applied:
---------------
Index: rules
===================================================================
--- rules	(Revision 140)
+++ rules	(Arbeitskopie)
@@ -100,7 +100,9 @@
 	touch configure
 	$(AUTOTOOLS)
 	(cd saslauthd && $(AUTOTOOLS) && cd ..)
-	LDFLAGS="$(LDFLAGS) -Wl,-z,defs" ./configure \
+	LDFLAGS="$(LDFLAGS) -Wl,-z,defs" \
+		CFLAGS="$(CFLAGS)" \
+		./configure \
 	       --host=$(DEB_HOST_GNU_TYPE) \
 	       --build=$(DEB_BUILD_GNU_TYPE) \
 	       --prefix=/usr \
---------------

It does not actually *change* most[1] executed commands in a signigicant
way, gcc is executed with "-Wall -W -Wall -g -O2" instead of the
currently used "-Wall -W -g -O2", but should be fixed anyway.

The fact that -O2 is used by default is a bug on its own,
DEB_BUILD_OPTIONS=noopt does not work. I have no idea how to fix it,
autoconf seems to use it by default if gcc is detected.

cu andreas
[1] except in the saslauthd directory, which does not use -Wall by
upstream default settings.
-- 
The 'Galactic Cleaning' policy undertaken by Emperor Zhark is a personal
vision of the emperor's, and its inclusion in this work does not constitute
tacit approval by the author or the publisher for any such projects,
howsoever undertaken.                                (c) Jasper Ffforde