A good reason to use make variables instead of autoconf substitutions

directly - fixed problem where the last line of the link command was
'\ @LDAP_LIBS@'.  If @LDAP_LIBS@ is zero then the backslash
incorrectly includes the next line of the Makefile in the current
target.

This should fix a bucketload of build farm failures.
(This used to be commit 895bef1a62f60f4e91dc6b1c244820b024c36a5c)

diff --git a/source3/Makefile.in b/source3/Makefile.in
index 985bdad08f280592d4ca4d3dcd756bad76eee755..7f9c2a8b930b678f594f7406cfb0f56cf4262de9 100644 (file)
--- a/source3/Makefile.in
+++ b/source3/Makefile.in
@@ -976,8 +976,7 @@ nsswitch/libnss_wins.@SHLIBEXT@: $(NSS_OBJ)
 
 bin/winbindd@EXEEXT@: $(WINBINDD_OBJ) @BUILD_POPT@ bin/.dummy
        @echo Linking $@
-       @$(LINK) -o $@ $(WINBINDD_OBJ) $(DYNEXP) $(LIBS) @POPTLIBS@ $(ADSLIBS) \
-               @LDAP_LIBS@
+       @$(LINK) -o $@ $(WINBINDD_OBJ) $(DYNEXP) $(LIBS) @POPTLIBS@ $(ADSLIBS) @LDAP_LIBS@
 
 nsswitch/@WINBIND_NSS@: $(WINBIND_NSS_PICOBJS)
        @echo "Linking $@"