Skip to contentSkip to conversation

gtk+3: only set GTKIMMODULES_PACKAGES if we have immodules

PATCH New Christopher Larson 1 participant 2 messages Aug 29, 2013

lore
Share & tools
lore
#0 lore raw

PATCH 1/2 gtk+3: only set GTKIMMODULES_PACKAGES if we have immodules

From: Christopher Larson <[email protected]>

---
 meta/recipes-gnome/gtk+/gtk+3.inc | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)
meta/recipes-gnome/gtk+/gtk+3.inc +4 -1
index f4d5254..e769ca5 100644
@@ -94,7 +94,10 @@ python populate_packages_prepend () {
     immodules_root = os.path.join(gtk_libdir, 'immodules')
     printmodules_root = os.path.join(gtk_libdir, 'printbackends');
 
-    d.setVar('GTKIMMODULES_PACKAGES', ' '.join(do_split_packages(d, immodules_root, '^im-(.*)\.so$', 'gtk3-immodule-%s', 'GTK input module for %s')))
+    immodules = do_split_packages(d, immodules_root, '^im-(.*)\.so$', 'gtk3-immodule-%s', 'GTK input module for %s')
+    if immodules:
+        d.setVar("GTKIMMODULES_PACKAGES", " ".join(immodules))
+
     do_split_packages(d, printmodules_root, '^libprintbackend-(.*)\.so$', 'gtk3-printbackend-%s', 'GTK printbackend module for %s')
 
     if (d.getVar('DEBIAN_NAMES', 1)):
Signature · 1 line
1.8.3.4
#1 lore raw

PATCH 2/2 gtk+3: pull forward the gtk+2 hardcoded libtool patch

From: Christopher Larson <[email protected]>

Without this, it tries to run ./libtool, not the sys-prefixed libtool, which fails, resulting in dynamic modules being disabled, which in turn results in compiling the immodules directly into the gtk+3 library.

I tried switching it to using $LIBTOOL rather than hardcoding the path, as LT_INIT sets LIBTOOL, but it didn't work, I didn't have time to dig further, and this gets the job done for now.

---
 .../gtk+/gtk+3/hardcoded_libtool.patch             | 35 ++++++++++++++++++++++
 meta/recipes-gnome/gtk+/gtk+3_3.8.2.bb             |  1 +
 2 files changed, 36 insertions(+)
 create mode 100644 meta/recipes-gnome/gtk+/gtk+3/hardcoded_libtool.patch
meta/recipes-gnome/gtk+/gtk+3/hardcoded_libtool.patch new file +35 -0
new file mode 100644
index 0000000..3b4beeb
@@ -0,0 +1,35 @@
+Upstream-Status: Inappropriate [embedded specific]
+
+Signed-off-by: Marko Lindqvist <[email protected]>
+
+Index: gtk+-3.8.2/configure.ac
+===================================================================
+--- gtk+-3.8.2.orig/configure.ac
++++ gtk+-3.8.2/configure.ac
+@@ -531,7 +531,7 @@ AC_MSG_CHECKING([Whether to write depend
+ case $enable_explicit_deps in
+   auto)
+     export SED
+-    deplibs_check_method=`(./libtool --config; echo 'eval echo \"$deplibs_check_method\"') | sh`
++    deplibs_check_method=`(./$host_alias-libtool --config; echo 'eval echo \"$deplibs_check_method\"') | sh`
+     if test "x$deplibs_check_method" != xpass_all || test "x$enable_static" = xyes ; then
+       enable_explicit_deps=yes
+     else
+@@ -793,7 +793,7 @@ else
+     dnl Now we check to see if our libtool supports shared lib deps
+     dnl (in a rather ugly way even)
+     if $dynworks; then
+-        module_libtool_config="${CONFIG_SHELL-/bin/sh} ./libtool --config"
++        module_libtool_config="${CONFIG_SHELL-/bin/sh} ./$host_alias-libtool --config"
+         module_deplibs_check=`$module_libtool_config | \
+             grep '^[[a-z_]]*check[[a-z_]]*_method=[['\''"]]' | \
+             sed 's/.*[['\''"]]\(.*\)[['\''"]]$/\1/'`
+@@ -1528,7 +1528,7 @@ fi
+ # We are using gmodule-no-export now, but I'm leaving the stripping
+ # code in place for now, since pango and atk still require gmodule.
+ export SED
+-export_dynamic=`(./libtool --config; echo eval echo \\$export_dynamic_flag_spec) | sh`
++export_dynamic=`(./$host_alias-libtool --config; echo eval echo \\$export_dynamic_flag_spec) | sh`
+ if test -n "$export_dynamic"; then
+   GDK_DEP_LIBS=`echo $GDK_DEP_LIBS | sed -e "s/$export_dynamic//"`
+   GTK_DEP_LIBS=`echo $GTK_DEP_LIBS | sed -e "s/$export_dynamic//"`
meta/recipes-gnome/gtk+/gtk+3_3.8.2.bb +1 -0
index d92d8f3..5db3745 100644
@@ -3,6 +3,7 @@ require gtk+3.inc
 MAJ_VER = "${@oe.utils.trim_version("${PV}", 2)}"
 
 SRC_URI = "http://download.gnome.org/sources/gtk+/${MAJ_VER}/gtk+-${PV}.tar.xz \
+           file://hardcoded_libtool.patch \
            file://no-x11-in-wayland.patch \
            file://wayland-attach.patch"
 
Signature · 1 line
1.8.3.4