2011-03-31 Xan Lopez <xlopez@igalia.com>
authorxan@webkit.org <xan@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 31 Mar 2011 17:37:59 +0000 (17:37 +0000)
committerxan@webkit.org <xan@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 31 Mar 2011 17:37:59 +0000 (17:37 +0000)
        Reviewed by Martin Robinson.

        [GTK] Link explicitly with XRender on Linux/Unix
        https://bugs.webkit.org/show_bug.cgi?id=57558

        Some linkers, like GNU Gold, require explicit linkage of all the
        libraries used instead of relying on our dependencies to bring
        them in. In this case, the build fails because we use XRender in
        the plugin code but don't link directly to it.

        * configure.ac: check for XRender on UNIX.
2011-03-31  Xan Lopez  <xlopez@igalia.com>

        Reviewed by Martin Robinson.

        [GTK] Link explicitly with XRender on Linux/Unix
        https://bugs.webkit.org/show_bug.cgi?id=57558

        * GNUmakefile.am: add XRender CFLAGS.
2011-03-31  Xan Lopez  <xlopez@igalia.com>

        Reviewed by Martin Robinson.

        [GTK] Link explicitly with XRender on Linux/Unix
        https://bugs.webkit.org/show_bug.cgi?id=57558

        * GNUmakefile.am: add XRender LIBS.

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

ChangeLog
Source/WebCore/ChangeLog
Source/WebCore/GNUmakefile.am
Source/WebKit/gtk/ChangeLog
Source/WebKit/gtk/GNUmakefile.am
configure.ac

index f824e2b..32145ab 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,17 @@
+2011-03-31  Xan Lopez  <xlopez@igalia.com>
+
+        Reviewed by Martin Robinson.
+
+        [GTK] Link explicitly with XRender on Linux/Unix
+        https://bugs.webkit.org/show_bug.cgi?id=57558
+
+        Some linkers, like GNU Gold, require explicit linkage of all the
+        libraries used instead of relying on our dependencies to bring
+        them in. In this case, the build fails because we use XRender in
+        the plugin code but don't link directly to it.
+
+        * configure.ac: check for XRender on UNIX.
+
 2011-03-31  Vamshikrishna.Yellenki  <vamshi@motorola.com> and Alejandro G. Castro  <alex@igalia.com>
 
         Reviewed by Martin Robinson.
index 068eaa7..6266a63 100644 (file)
@@ -1,3 +1,12 @@
+2011-03-31  Xan Lopez  <xlopez@igalia.com>
+
+        Reviewed by Martin Robinson.
+
+        [GTK] Link explicitly with XRender on Linux/Unix
+        https://bugs.webkit.org/show_bug.cgi?id=57558
+
+        * GNUmakefile.am: add XRender CFLAGS.
+
 2011-03-31  Eric Seidel  <eric@webkit.org>
 
         Reviewed by Ryosuke Niwa.
index 88a1f07..99d6841 100644 (file)
@@ -5110,6 +5110,7 @@ libWebCore_la_CPPFLAGS = \
        $(LIBXSLT_CFLAGS) \
        $(SQLITE3_CFLAGS) \
        $(UNICODE_CFLAGS) \
+       $(XRENDER_CFLAGS) \
        $(XT_CFLAGS)
 
 EXTRA_DIST += \
index f6a1e34..f1d79e4 100644 (file)
@@ -1,3 +1,12 @@
+2011-03-31  Xan Lopez  <xlopez@igalia.com>
+
+        Reviewed by Martin Robinson.
+
+        [GTK] Link explicitly with XRender on Linux/Unix
+        https://bugs.webkit.org/show_bug.cgi?id=57558
+
+        * GNUmakefile.am: add XRender LIBS.
+
 2011-03-31  Evan Martin  <evan@chromium.org>
 
         Build fix from previous change.
index 7030ea8..5ace519 100644 (file)
@@ -102,6 +102,7 @@ libwebkitgtk_@WEBKITGTK_API_MAJOR_VERSION@_@WEBKITGTK_API_MINOR_VERSION@_la_LIBA
        $(PNG_LIBS) \
        $(SQLITE3_LIBS) \
        $(UNICODE_LIBS) \
+       $(XRENDER_LIBS) \
        $(XT_LIBS) \
        $(WINMM_LIBS) \
        $(SHLWAPI_LIBS) \
index a0345d1..ddddf0b 100644 (file)
@@ -938,6 +938,14 @@ if test "$enable_geolocation" = "yes"; then
     AC_SUBST([GEOCLUE_LIBS])
 fi
 
+# check for XRender under Linux/Unix. Some linkers require explicit
+# linkage (like GNU Gold), so we cannot rely on GTK+ pulling XRender
+if test "$os_win32" = "no"; then
+   PKG_CHECK_MODULES([XRENDER], [xrender])
+   AC_SUBST([XRENDER_CFLAGS])
+   AC_SUBST([XRENDER_LIBS])
+fi
+
 # check if gstreamer is available
 if test "$enable_video" = "yes"; then
    PKG_CHECK_MODULES([GSTREAMER],