2008-09-23 Richard Hult <richard@imendio.com>
authorjmalonzo@webkit.org <jmalonzo@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 24 Sep 2008 07:42:51 +0000 (07:42 +0000)
committerjmalonzo@webkit.org <jmalonzo@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 24 Sep 2008 07:42:51 +0000 (07:42 +0000)
        Reviewed by Alp Toker. Landed by Jan Alonzo.

        http://bugs.webkit.org/show_bug.cgi?id=20582
        [Gtk] Link error on Mac OS X

        * GNUmakefile.am: Only use the -version-script flag for the gnu
        linker.
        * configure.ac: Check for GNU OSes.

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

ChangeLog
GNUmakefile.am
configure.ac

index 7bc9dd4..73d60df 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,14 @@
+2008-09-23  Richard Hult  <richard@imendio.com>
+
+        Reviewed by Alp Toker. Landed by Jan Alonzo.
+
+        http://bugs.webkit.org/show_bug.cgi?id=20582
+        [Gtk] Link error on Mac OS X
+
+        * GNUmakefile.am: Only use the -version-script flag for the gnu
+        linker.
+        * configure.ac: Check for GNU OSes.
+
 2008-09-19  Alp Toker  <alp@nuanti.com>
 
         Build fix for the 'gold' linker and recent binutils. New behaviour
 2008-09-19  Alp Toker  <alp@nuanti.com>
 
         Build fix for the 'gold' linker and recent binutils. New behaviour
index 458b854..6029357 100644 (file)
@@ -78,6 +78,10 @@ if OS_WIN32
 no_undefined = -no-undefined
 endif
 
 no_undefined = -no-undefined
 endif
 
+if OS_GNU
+version_script = Wl,--version-script,$(srcdir)/symbols.filter
+endif
+
 # Shared libraries
 lib_LTLIBRARIES = \
        libwebkit-1.0.la
 # Shared libraries
 lib_LTLIBRARIES = \
        libwebkit-1.0.la
@@ -238,7 +242,7 @@ libwebkit_1_0_la_CPPFLAGS = \
 libwebkit_1_0_la_LDFLAGS = \
        $(COVERAGE_LDFLAGS) \
        -version-info @LIBWEBKITGTK_VERSION@ \
 libwebkit_1_0_la_LDFLAGS = \
        $(COVERAGE_LDFLAGS) \
        -version-info @LIBWEBKITGTK_VERSION@ \
-       -Wl,--version-script,$(srcdir)/symbols.filter \
+       $(version_script) \
        $(no_undefined)
 
 libwebkit_1_0_la_LIBADD = \
        $(no_undefined)
 
 libwebkit_1_0_la_LIBADD = \
index 55dddbd..8962814 100644 (file)
@@ -51,6 +51,15 @@ case "$host" in
        ;;
 esac
 
        ;;
 esac
 
+case "$host_os" in
+     gnu* | linux* | k*bsd*-gnu)
+       os_gnu=yes
+       ;;
+     *)
+       os_gnu=no
+       ;;
+esac
+
 # If CXXFLAGS and CFLAGS are unset, default to empty.
 # This is to tell automake not to include '-g' if CXXFLAGS is not set
 # For more info - http://www.gnu.org/software/automake/manual/autoconf.html#C_002b_002b-Compiler
 # If CXXFLAGS and CFLAGS are unset, default to empty.
 # This is to tell automake not to include '-g' if CXXFLAGS is not set
 # For more info - http://www.gnu.org/software/automake/manual/autoconf.html#C_002b_002b-Compiler
@@ -576,6 +585,7 @@ fi
 AM_CONDITIONAL([OS_WIN32],[test "$os_win32" = "yes"])
 AM_CONDITIONAL([OS_UNIX],[test "$os_win32" = "no"])
 AM_CONDITIONAL([OS_LINUX],[test "$os_linux" = "yes"])
 AM_CONDITIONAL([OS_WIN32],[test "$os_win32" = "yes"])
 AM_CONDITIONAL([OS_UNIX],[test "$os_win32" = "no"])
 AM_CONDITIONAL([OS_LINUX],[test "$os_linux" = "yes"])
+AM_CONDITIONAL([OS_GNU],[test "$os_gnu" = "yes"])
 AM_CONDITIONAL([OS_FREEBSD],[test "$os_freebsd" = "yes"])
 
 # target conditionals
 AM_CONDITIONAL([OS_FREEBSD],[test "$os_freebsd" = "yes"])
 
 # target conditionals