Title: [123986] trunk/Tools
Revision
123986
Author
[email protected]
Date
2012-07-29 15:26:55 -0700 (Sun, 29 Jul 2012)

Log Message

[EFL][jhbuild] Make run-with-jhbuild become the jhbuild process itself.
https://bugs.webkit.org/show_bug.cgi?id=92592

Reviewed by Eric Seidel.

Adapt to the changes made to GTK+'s run-with-jhbuild in r123979
and make EFL's run-with-jhbuild call os.execve() instead of
launching a subprocess as well.

This should make it possible to interrupt execution of jhbuild
cleanly.

* efl/run-with-jhbuild:

Modified Paths

Diff

Modified: trunk/Tools/ChangeLog (123985 => 123986)


--- trunk/Tools/ChangeLog	2012-07-29 22:03:35 UTC (rev 123985)
+++ trunk/Tools/ChangeLog	2012-07-29 22:26:55 UTC (rev 123986)
@@ -1,3 +1,19 @@
+2012-07-29  Raphael Kubo da Costa  <[email protected]>
+
+        [EFL][jhbuild] Make run-with-jhbuild become the jhbuild process itself.
+        https://bugs.webkit.org/show_bug.cgi?id=92592
+
+        Reviewed by Eric Seidel.
+
+        Adapt to the changes made to GTK+'s run-with-jhbuild in r123979
+        and make EFL's run-with-jhbuild call os.execve() instead of
+        launching a subprocess as well.
+
+        This should make it possible to interrupt execution of jhbuild
+        cleanly.
+
+        * efl/run-with-jhbuild:
+
 2012-07-28  Ryosuke Niwa  <[email protected]>
 
         run-perf-tests should generate a results page

Modified: trunk/Tools/efl/run-with-jhbuild (123985 => 123986)


--- trunk/Tools/efl/run-with-jhbuild	2012-07-29 22:03:35 UTC (rev 123985)
+++ trunk/Tools/efl/run-with-jhbuild	2012-07-29 22:26:55 UTC (rev 123986)
@@ -17,10 +17,9 @@
 # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301  USA
 
 import common
-import subprocess
+import os
 import sys
 
+
 jhbuild_wrapper = common.top_level_path('Tools', 'jhbuild', 'jhbuild-wrapper')
-process = subprocess.Popen([jhbuild_wrapper, '--efl', 'run'] + sys.argv[1:])
-process.wait()
-sys.exit(process.returncode)
+os.execve(jhbuild_wrapper, [jhbuild_wrapper, '--efl', 'run'] + sys.argv[1:], os.environ)
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to