[GTK] Allow to run any jhbuild command with jhbuild-wrapper script
authorcarlosgc@webkit.org <carlosgc@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 22 Mar 2012 10:50:32 +0000 (10:50 +0000)
committercarlosgc@webkit.org <carlosgc@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 22 Mar 2012 10:50:32 +0000 (10:50 +0000)
https://bugs.webkit.org/show_bug.cgi?id=81888

Reviewed by Philippe Normand.

* Scripts/update-webkit-libs-jhbuild:
(runJhbuild): Helper function to run jhbuild with a given command.
Instead of running jhbuild directly, use the jhbuild-wrapper, so
that we don't need to run it first to check whether jhbuild is
installed or not.
* efl/run-with-jhbuild: Call jhbuild-wrapper with run command.
* gtk/run-with-jhbuild: Ditto.
* jhbuild/jhbuild-wrapper: Allow to execute any jhbuild command,
not only run.

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

Tools/ChangeLog
Tools/Scripts/update-webkit-libs-jhbuild
Tools/efl/run-with-jhbuild
Tools/gtk/run-with-jhbuild
Tools/jhbuild/jhbuild-wrapper

index 52726cc..4d84e46 100644 (file)
@@ -1,3 +1,20 @@
+2012-03-22  Carlos Garcia Campos  <cgarcia@igalia.com>
+
+        [GTK] Allow to run any jhbuild command with jhbuild-wrapper script
+        https://bugs.webkit.org/show_bug.cgi?id=81888
+
+        Reviewed by Philippe Normand.
+
+        * Scripts/update-webkit-libs-jhbuild:
+        (runJhbuild): Helper function to run jhbuild with a given command.
+        Instead of running jhbuild directly, use the jhbuild-wrapper, so
+        that we don't need to run it first to check whether jhbuild is
+        installed or not.
+        * efl/run-with-jhbuild: Call jhbuild-wrapper with run command.
+        * gtk/run-with-jhbuild: Ditto.
+        * jhbuild/jhbuild-wrapper: Allow to execute any jhbuild command,
+        not only run.
+
 2012-03-22  Philippe Normand  <pnormand@igalia.com>
 
         Unreviewed, test-webkitpy fix after r111661.
index 7e2440a..48343fb 100755 (executable)
@@ -41,21 +41,18 @@ if (!$getOptionsResult) {
     }
 }
 
+sub runJhbuild
+{
+    my $command = shift;
+    my @jhbuildArgs = ("./jhbuild-wrapper", "--".$platform, $command);
+    push(@jhbuildArgs, @ARGV[2..-1]);
+    system(@jhbuildArgs) == 0 or die "Running jhbuild-wrapper " . $command . " failed.\n";
+}
+
 delete $ENV{AR_FLAGS} if exists $ENV{AR_FLAGS};
 
 chdir(relativeScriptsDir() . "/../jhbuild") or die $!;
 
-my @ensureJhbuildArgs = ("./jhbuild-wrapper", "--".$platform, "echo", "Ensured jhbuild setup.");
-if (system(@ensureJhbuildArgs) != 0) {
-    die "Failed to ensure jhbuild installed.\n";
-}
-
-chdir(relativeScriptsDir() . "/../".$platform) or die $!;
-
 my %prettyPlatform = ( "efl" => "EFL", "gtk" => "GTK+" );
 print "Updating " . $prettyPlatform{$platform} . " port dependencies using jhbuild...\n";
-my @jhbuildArgs = ("../../WebKitBuild/Dependencies/Root/bin/jhbuild", "--no-interact", "-f", "jhbuildrc");
-push(@jhbuildArgs, @ARGV[2..-1]);
-if (system(@jhbuildArgs) != 0) {
-    die "Running jhbuild failed.\n"
-}
+runJhbuild("build");
index 624191a..8cde7da 100755 (executable)
@@ -21,6 +21,6 @@ import subprocess
 import sys
 
 jhbuild_wrapper = common.top_level_path('Tools', 'jhbuild', 'jhbuild-wrapper')
-process = subprocess.Popen([jhbuild_wrapper, '--efl'] + sys.argv[1:])
+process = subprocess.Popen([jhbuild_wrapper, '--efl', 'run'] + sys.argv[1:])
 process.wait()
 sys.exit(process.returncode)
index fdf6f0f..9ff0034 100755 (executable)
@@ -22,6 +22,6 @@ import subprocess
 import sys
 
 jhbuild_wrapper = common.top_level_path('Tools', 'jhbuild', 'jhbuild-wrapper')
-process = subprocess.Popen([jhbuild_wrapper, '--gtk'] + sys.argv[1:])
+process = subprocess.Popen([jhbuild_wrapper, '--gtk', 'run'] + sys.argv[1:])
 process.wait()
 sys.exit(process.returncode)
index ac4d77c..5b423fe 100755 (executable)
@@ -135,6 +135,6 @@ except ValueError as e:
     sys.exit(e)
 ensure_jhbuild()
 
-process = subprocess.Popen([jhbuild_path, '--no-interact', '-f', top_level_path('Tools', platform, 'jhbuildrc'), 'run'] + sys.argv[2:])
+process = subprocess.Popen([jhbuild_path, '--no-interact', '-f', top_level_path('Tools', platform, 'jhbuildrc')] + sys.argv[2:])
 process.wait()
 sys.exit(process.returncode)