Title: [130781] trunk/Tools
Revision
130781
Author
ale...@webkit.org
Date
2012-10-09 11:37:29 -0700 (Tue, 09 Oct 2012)

Log Message

[GTK] Shadow builds are not working anymore.
https://bugs.webkit.org/show_bug.cgi?id=98785

Reviewed by Martin Robinson.

When setting WEBKITOUTPUTDIR the build was failing with a python
error : the os.execve expect three arguments. Also the condition
to check whether we build the GTK port with jhbuild or not was buggy
as it was always assuming an in source build. We can use now jhbuildutils
which has a convenient function to locate the directory where
the dependencies built with jhbuild are and take into account WEBKITOUTPUTDIR.
If the Dependencies path does not exist then the build was not done using
jhbuild so we can fallback to a regular build against system libraries.

* gtk/run-with-jhbuild:

Modified Paths

Diff

Modified: trunk/Tools/ChangeLog (130780 => 130781)


--- trunk/Tools/ChangeLog	2012-10-09 18:36:17 UTC (rev 130780)
+++ trunk/Tools/ChangeLog	2012-10-09 18:37:29 UTC (rev 130781)
@@ -1,3 +1,21 @@
+2012-10-09  Alexis Menard  <ale...@webkit.org>
+
+        [GTK] Shadow builds are not working anymore.
+        https://bugs.webkit.org/show_bug.cgi?id=98785
+
+        Reviewed by Martin Robinson.
+
+        When setting WEBKITOUTPUTDIR the build was failing with a python
+        error : the os.execve expect three arguments. Also the condition
+        to check whether we build the GTK port with jhbuild or not was buggy
+        as it was always assuming an in source build. We can use now jhbuildutils
+        which has a convenient function to locate the directory where 
+        the dependencies built with jhbuild are and take into account WEBKITOUTPUTDIR.
+        If the Dependencies path does not exist then the build was not done using
+        jhbuild so we can fallback to a regular build against system libraries.
+
+        * gtk/run-with-jhbuild:
+
 2012-10-08  Simon Fraser  <simon.fra...@apple.com>
 
         Remove DRT/WTR implementations of layerTreeAsText

Modified: trunk/Tools/gtk/run-with-jhbuild (130780 => 130781)


--- trunk/Tools/gtk/run-with-jhbuild	2012-10-09 18:36:17 UTC (rev 130780)
+++ trunk/Tools/gtk/run-with-jhbuild	2012-10-09 18:37:29 UTC (rev 130781)
@@ -22,10 +22,11 @@
 import subprocess
 import sys
 
-base_build_dir = os.getenv('WEBKITOUTPUTDIR', common.top_level_path('WebKitBuild'))
+sys.path.append(common.top_level_path('Tools', 'jhbuild'))
+import jhbuildutils
 
-if not os.path.exists(os.path.join(base_build_dir, 'Dependencies')):
-    os.execve(sys.argv[1], sys.argv[1])
+if (not os.path.exists(jhbuildutils.get_dependencies_path())):
+    os.execve(sys.argv[1], sys.argv[1:], os.environ)
 else:
     wrapper = common.top_level_path('Tools', 'jhbuild', 'jhbuild-wrapper')
     os.execve(wrapper, [wrapper, '--gtk', 'run'] + sys.argv[1:], os.environ)
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to