Hi list,

the attached patch fixes a few typos in popplerkit config.sh, and also uses
LIBRARIES_DEPEND_UPON, instead of ADDITIONAL_LDFLAGS, to fix compilation
with --as-needed

This allows correct linking (tested with stable 0.4.0) when using
--as-needed

Gentoo bug is here (with resulting compilation failure log for vindaloo):
http://bugs.gentoo.org/show_bug.cgi?id=250175

Regards,
-- 
Bernard Cafarelli (Voyageur)
Gentoo developer (NX, GNUstep, net-misc, ...)
--- Frameworks/PopplerKit/config.sh.orig        2008-12-08 14:43:36.000000000 
+0100
+++ Frameworks/PopplerKit/config.sh     2008-12-08 14:43:51.000000000 +0100
@@ -22,7 +22,7 @@
     exit 1
 fi
 POPPLER_CFLAGS="${POPPLER_CFLAGS} `${PKG_CONFIG} --cflags fontconfig`"
-POPPLER_LIBS="${POPPLER_LDFLAGS} `${PKG_CONFIG} --libs fontconfig`"
+POPPLER_LIBS="${POPPLER_LIBS} `${PKG_CONFIG} --libs fontconfig`"
 
 # poppler splash device
 ${PKG_CONFIG} --exists poppler-splash
@@ -31,7 +31,7 @@
     exit 1
 fi
 POPPLER_CFLAGS="${POPPLER_CFLAGS} `${PKG_CONFIG} --cflags poppler-splash`"
-POPPLER_LIBS="${POPPLER_LDFLAGS} `${PKG_CONFIG} --libs poppler-splash`"
+POPPLER_LIBS="${POPPLER_LIBS} `${PKG_CONFIG} --libs poppler-splash`"
 
 # poppler cairo device
 ${PKG_CONFIG} --exists poppler-cairo
@@ -42,7 +42,7 @@
 #   Disable Cairo support for now to avoid most of problem
 #   HAVE_CAIRO="YES"
 #   POPPLER_CFLAGS="${POPPLER_CFLAGS} `${PKG_CONFIG} --cflags poppler-cairo`"
-#   POPPLER_LIBS="${POPPLER_LDFLAGS} `${PKG_CONFIG} --libs poppler-cairo`"
+#   POPPLER_LIBS="${POPPLER_LIBS} `${PKG_CONFIG} --libs poppler-cairo`"
    HAVE_CAIRO="NO"
 fi
 
@@ -85,7 +85,7 @@
 echo "FT_CFLAGS=${FT_CFLAGS}" >> config.make
 echo "FT_LIBS=${FT_LIBS}" >> config.make
 echo "ADDITIONAL_CFLAGS+=\$(POPPLER_CFLAGS) \$(FT_CFLAGS)" >> config.make
-echo "ADDITIONAL_LDFLAGS+=\$(POPPLER_LIBS) \$(POPPLER_LIBS)" >> config.make
+echo "LIBRARIES_DEPEND_UPON+=\$(POPPLER_LIBS) \$(FT_LIBS)" >> config.make
 # we add -I/usr/X11R6/include for older FreeBSD version.
 echo "ADDITIONAL_INCLUDE_DIRS += -I/usr/X11R6/include" >> config.make
 echo "HAVE_CAIRO=${HAVE_CAIRO}" >>config.make
_______________________________________________
Etoile-discuss mailing list
[email protected]
https://mail.gna.org/listinfo/etoile-discuss

Répondre à