[lttng-dev] [LTTV PATCH] Test for gmodule inclusion in all cases

Alexandre Montplaisir alexandre.montplaisir at gmail.com
Wed Feb 8 18:19:12 EST 2012


On recent glib versions, it seems gmodule needs to be included
separately from glib/gtk, or it leads to compilation errors,
e.g. undefined reference to `g_module_close'

(This patch goes on top of the small branch posted earlier.)

Signed-off-by: Mathieu Denis <mathieu.denis55 at gmail.com>
Signed-off by: Yannick Brosseau <yannick.brosseau at gmail.com>
Signed-off-by: Alexandre Montplaisir <alexandre.montplaisir at gmail.com>
---
 configure.ac |    8 +++-----
 1 files changed, 3 insertions(+), 5 deletions(-)

diff --git a/configure.ac b/configure.ac
index 51aabd4..2cd8ee9 100644
--- a/configure.ac
+++ b/configure.ac
@@ -113,13 +113,11 @@ AM_CONDITIONAL([BUILD_JNI_INTERFACE], [ test "$with_jni_interface" = "yes" ] )
 AS_IF([test "x$with_lttv_gui" = "xyes"],[
 	pkg_modules="gtk+-2.0 >= 2.0.0"
 	PKG_CHECK_MODULES(PACKAGE, [$pkg_modules])
-],[
-	# Else, we still have a dependency on gmodule
-	# We will add flags for gmodule alone
-	pkg_modules="gmodule-2.0 >= 2.0.0"
-	PKG_CHECK_MODULES(GMODULE, [$pkg_modules])
 ])
 
+pkg_modules="gmodule-2.0 >= 2.0.0"
+PKG_CHECK_MODULES(GMODULE, [$pkg_modules])
+
 pkg_modules="gobject-2.0 >= 2.0.0"
 PKG_CHECK_MODULES(GOBJECT, [$pkg_modules])
 
-- 
1.7.9




More information about the lttng-dev mailing list