2008-02-21 Mike Auty <mike.auty@gmail.com>
authoralp@webkit.org <alp@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 21 Feb 2008 19:43:44 +0000 (19:43 +0000)
committeralp@webkit.org <alp@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 21 Feb 2008 19:43:44 +0000 (19:43 +0000)
        Reviewed by Alp Toker.

        http://bugs.webkit.org/show_bug.cgi?id=17445
        [GTK] WebKit doesn't compile with LDFLAGS="-Wl,--as-needed"

        The GNUmakefile.am files make use of the LDFLAGS variable to include library
        additions such as -ljpeg etc.  Unfortunately, if these inclusions aren't made
        in LIBADD/LDADD variables, then they are mis-ordered during the linking.

        The as-needed flag discards libraries whose functions have not been needed by
        earlier libraries, which therefore makes the ordering important.

        This moves all -l library inclusion statements from LDFLAGS variables to
        LIBADD/LDADD variables.

        * GNUmakefile.am:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@30459 268f45cc-cd09-0410-ab3c-d52691b4dbfc

ChangeLog
GNUmakefile.am
WebKitTools/ChangeLog
WebKitTools/GNUmakefile.am

index 21b0b0f..e84dacc 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,22 @@
+2008-02-21  Mike Auty  <mike.auty@gmail.com>
+
+        Reviewed by Alp Toker.
+
+        http://bugs.webkit.org/show_bug.cgi?id=17445
+        [GTK] WebKit doesn't compile with LDFLAGS="-Wl,--as-needed"
+
+        The GNUmakefile.am files make use of the LDFLAGS variable to include library
+        additions such as -ljpeg etc.  Unfortunately, if these inclusions aren't made
+        in LIBADD/LDADD variables, then they are mis-ordered during the linking.
+
+        The as-needed flag discards libraries whose functions have not been needed by
+        earlier libraries, which therefore makes the ordering important.
+
+        This moves all -l library inclusion statements from LDFLAGS variables to
+        LIBADD/LDADD variables.
+
+        * GNUmakefile.am:
+
 2008-02-15  Alp Toker  <alp@atoker.com>
 
         Reviewed by Holger.
index ad5e1f8..8e16585 100644 (file)
@@ -102,7 +102,7 @@ libJavaScriptCore_la_SOURCES = \
        $(javascriptcore_built_sources) \
        $(javascriptcore_sources)
 
-libJavaScriptCore_la_LDFLAGS = $(ICU_LIBS) -lpthread
+libJavaScriptCore_la_LIBADD = $(ICU_LIBS) -lpthread
 
 libJavaScriptCore_la_CXXFLAGS = $(global_cxxflags) $(global_cflags) -fstrict-aliasing
 
@@ -126,7 +126,7 @@ webkitgtk_h_api :=
 webkitgtk_headers :=
 webkitgtk_sources :=
 webkitgtk_cppflags :=
-webkitgtk_ldflags :=
+webkitgtk_libadd :=
 webkitgtk_built_sources :=
 webkitgtk_built_nosources :=
 webkitgtk_cleanfiles :=
@@ -176,20 +176,20 @@ libWebKitGtk_la_CPPFLAGS = \
        $(webkitgtk_cppflags) \
        $(ICU_CPPFLAGS)
 
-libWebKitGtk_la_LIBADD = libJavaScriptCore.la
-
-libWebKitGtk_la_LDFLAGS = \
-       $(webkitgtk_ldflags) \
+libWebKitGtk_la_LIBADD = libJavaScriptCore.la \
+       $(webkitgtk_libadd) \
        $(DEPENDENCIES_LIBS) \
        $(LIBCURL_LIBS) \
        $(ICU_LIBS) \
        $(SQLITE3_LIBS) \
        $(GSTREAMER_LIBS) \
        $(LIBXSLT_LIBS) \
-       $(COVERAGE_LDFLAGS) \
        $(HILDON_LIBS) \
        -lpthread \
-       -ljpeg \
+       -ljpeg
+
+libWebKitGtk_la_LDFLAGS = \
+       $(COVERAGE_LDFLAGS) \
        -version-info @LIBWEBKITGTK_VERSION@
 
 #
@@ -237,7 +237,7 @@ global_cppflags += \
 endif
 
 if ENABLE_VIDEO
-webkitgtk_ldflags += -lgstinterfaces-0.10 -lgstvideo-0.10
+webkitgtk_libadd += -lgstinterfaces-0.10 -lgstvideo-0.10
 endif
 
 #
index 2d53da4..2593f35 100644 (file)
@@ -1,3 +1,24 @@
+2008-02-21  Mike Auty  <mike.auty@gmail.com>
+
+        Reviewed by Alp Toker.
+
+        http://bugs.webkit.org/show_bug.cgi?id=17445
+        [GTK] WebKit doesn't compile with LDFLAGS="-Wl,--as-needed"
+
+        GTK+/autotools build system improvements
+
+        The GNUmakefile.am files make use of the LDFLAGS variable to include library
+        additions such as -ljpeg etc.  Unfortunately, if these inclusions aren't made
+        in LIBADD/LDADD variables, then they are mis-ordered during the linking.
+
+        The as-needed flag discards libraries whose functions have not been needed by
+        earlier libraries, which therefore makes the ordering important.
+
+        This moves all -l library inclusion statements from LDFLAGS variables to
+        LIBADD/LDADD variables.
+
+        * GNUmakefile.am:
+
 2008-02-20  Brent Fulgham  <bfulgham@gmail.com>
 
         Reviewed by Alp.
index e42a064..cc4c440 100644 (file)
@@ -19,7 +19,7 @@ Programs_GtkLauncher_LDADD = \
        libJavaScriptCore.la \
        libWebKitGtk.la
 
-Programs_GtkLauncher_LDFLAGS = -ljpeg -rpath $(CURDIR)/.libs
+Programs_GtkLauncher_LDFLAGS = -rpath $(CURDIR)/.libs
 
 # DumpRenderTree
 Programs_DumpRenderTree_CPPFLAGS = \