[GTK] The jhbuild environment should set CMAKE_PREFIX_PATH and CMAKE_LIBRARY_PATH
authormrobinson@webkit.org <mrobinson@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 11 Mar 2014 15:09:12 +0000 (15:09 +0000)
committermrobinson@webkit.org <mrobinson@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 11 Mar 2014 15:09:12 +0000 (15:09 +0000)
https://bugs.webkit.org/show_bug.cgi?id=130064

Reviewed by Philippe Normand.

* efl/jhbuildrc: Remove duplicated code.
* gtk/jhbuildrc: Remove duplicated code.
* jhbuild/jhbuildrc_common.py:
(init): Move duplicated code from jhbuildrc's and extend setting CMake environment variables to
GTK+. We always set the CMAKE_LIBRARY_PATH, which should be harmless for EFL 32-bit systems.
and is apparently necessary for GTK+.

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

Tools/ChangeLog
Tools/efl/jhbuildrc
Tools/gtk/jhbuildrc
Tools/jhbuild/jhbuildrc_common.py

index 450433e..9e612ce 100644 (file)
@@ -1,3 +1,17 @@
+2014-03-11  Martin Robinson  <mrobinson@igalia.com>
+
+        [GTK] The jhbuild environment should set CMAKE_PREFIX_PATH and CMAKE_LIBRARY_PATH
+        https://bugs.webkit.org/show_bug.cgi?id=130064
+
+        Reviewed by Philippe Normand.
+
+        * efl/jhbuildrc: Remove duplicated code.
+        * gtk/jhbuildrc: Remove duplicated code.
+        * jhbuild/jhbuildrc_common.py:
+        (init): Move duplicated code from jhbuildrc's and extend setting CMake environment variables to
+        GTK+. We always set the CMAKE_LIBRARY_PATH, which should be harmless for EFL 32-bit systems.
+        and is apparently necessary for GTK+.
+
 2014-03-11  László Langó  <llango.u-szeged@partner.samsung.com>
 
         check-webkit-style failed to complain about missing braces
index 7825951..e4c6213 100644 (file)
@@ -22,33 +22,7 @@ sys.path.insert(0, os.path.join(os.path.dirname(__file__), "../jhbuild") )
 import jhbuildrc_common
 jhbuildrc_common.init(globals(), "efl") 
 
-if 'NUMBER_OF_PROCESSORS' in os.environ:
-    jobs = os.environ['NUMBER_OF_PROCESSORS']
-
-# Use system libraries while building.
-if use_lib64:
-    _libdir = 'lib64'
-else:
-    _libdir = 'lib'
-addpath('PKG_CONFIG_PATH', os.path.join(os.sep, 'usr', _libdir, 'pkgconfig'))
-addpath('PKG_CONFIG_PATH', os.path.join(os.sep, 'usr', 'share', 'pkgconfig'))
-
 addpath('XDG_DATA_DIRS', '/usr/share')
 addpath('XDG_CONFIG_DIRS', '/etc/xdg')
 
-addpath('CMAKE_PREFIX_PATH', prefix)
-
-# jhbuild installs libraries into lib64/ by default when building on
-# 64-bit Linux installations, while CMake does not look for libraries
-# in lib64/ if /etc/debian_version exists. For now we just pass the
-# lib64/ directory to the CMAKE_LIBRARY_PATH environment variable to
-# make sure the libraries installed by jhbuild are found by CMake on
-# Debian-based systems as well.
-if use_lib64:
-    addpath('CMAKE_LIBRARY_PATH', os.path.join(prefix, _libdir))
-
 partial_build = False
-
-# Avoid runtime conflicts with GStreamer system-wide plugins. We want
-# to use only the plugins we build in JHBuild.
-os.environ['GST_PLUGIN_SYSTEM_PATH'] = ''
index 1e0bf4b..c35f450 100644 (file)
@@ -25,16 +25,9 @@ __gtk_tools_directory = os.path.abspath(os.path.dirname(__file__))
 sys.path = [__gtk_tools_directory] + sys.path
 import common
 
-if 'NUMBER_OF_PROCESSORS' in os.environ:
-    jobs = os.environ['NUMBER_OF_PROCESSORS']
-
 # GTK+ 3.0.12 misses the -lm flag when linking the tests.
 module_makeargs['gtk+'] = 'LDFLAGS="-lm" ' + makeargs
 
-# Avoid runtime conflicts with GStreamer system-wide plugins. We want
-# to use only the plugins we build in JHBuild.
-os.environ['GST_PLUGIN_SYSTEM_PATH'] = ''
-
 # We always enable introspection so that we can sniff out problems with our
 # annotations sooner rather than later.
 autogenargs='--enable-introspection'
index 20e99b3..7343ae8 100644 (file)
@@ -62,3 +62,23 @@ def init(jhbuildrc_globals, platform):
 
     jhbuildrc_globals["nonotify"] = True
     jhbuildrc_globals["notrayicon"] = True
+
+    if 'NUMBER_OF_PROCESSORS' in os.environ:
+        jhbuildrc_globals['jobs'] = os.environ['NUMBER_OF_PROCESSORS']
+
+    # Avoid runtime conflicts with GStreamer system-wide plugins. We want
+    # to use only the plugins we build in JHBuild.
+    os.environ['GST_PLUGIN_SYSTEM_PATH'] = ''
+
+    # Use system libraries while building.
+    if jhbuildrc_globals['use_lib64']:
+        _library_dir = 'lib64'
+    else:
+        _library_dir = 'lib'
+    addpath = jhbuildrc_globals['addpath']
+    addpath('PKG_CONFIG_PATH', os.path.join(os.sep, 'usr', _library_dir, 'pkgconfig'))
+    addpath('PKG_CONFIG_PATH', os.path.join(os.sep, 'usr', 'share', 'pkgconfig'))
+
+    prefix = jhbuildrc_globals['prefix']
+    addpath('CMAKE_PREFIX_PATH', prefix)
+    addpath('CMAKE_LIBRARY_PATH', os.path.join(prefix, _library_dir))