From: aroben@apple.com <aroben@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date: Thu, 22 Sep 2011 17:47:13 +0000 (+0000)
Subject: Use the "direct object" form of system() to run WebKit.exe
X-Git-Url: http://git.webkit.org/?p=WebKit-https.git;a=commitdiff_plain;h=9d26e0a0a09c5b952f13e97b6902d1875efaa761

Use the "direct object" form of system() to run WebKit.exe

This prevents the shell from parsing (and possibly splitting, if it contains spaces) the
path to WebKit.exe.

Fixes <http://webkit.org/b/68623> run-safari doesn't work if there are spaces in the path to
%WEBKITOUTPUTDIR%

Reviewed by Steve Falkenburg.

* Scripts/webkitdirs.pm:
(runSafari):

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

diff --git a/Tools/ChangeLog b/Tools/ChangeLog
index e2c9c72..7f76c77 100644
--- a/Tools/ChangeLog
+++ b/Tools/ChangeLog
@@ -1,5 +1,20 @@
 2011-09-22  Adam Roben  <aroben@apple.com>
 
+        Use the "direct object" form of system() to run WebKit.exe
+
+        This prevents the shell from parsing (and possibly splitting, if it contains spaces) the
+        path to WebKit.exe.
+
+        Fixes <http://webkit.org/b/68623> run-safari doesn't work if there are spaces in the path to
+        %WEBKITOUTPUTDIR%
+
+        Reviewed by Steve Falkenburg.
+
+        * Scripts/webkitdirs.pm:
+        (runSafari):
+
+2011-09-22  Adam Roben  <aroben@apple.com>
+
         Quote the WEBKITLIBRARIESDIR path before passing it to the shell/cygpath
 
         Fixes <http://webkit.org/b/68621> build-webkit goes totally haywire on Windows if there are
diff --git a/Tools/Scripts/webkitdirs.pm b/Tools/Scripts/webkitdirs.pm
index 410f58b..13133b6 100644
--- a/Tools/Scripts/webkitdirs.pm
+++ b/Tools/Scripts/webkitdirs.pm
@@ -2028,7 +2028,8 @@ sub runSafari
             chomp($safariPath = `cygpath -wa "$safariPath"`);
             $result = system $vcBuildPath, "/debugexe", "\"$safariPath\"", @ARGV;
         } else {
-            $result = system File::Spec->catfile(productDir(), "WebKit.exe"), @ARGV;
+            my $webKitLauncherPath = File::Spec->catfile(productDir(), "WebKit.exe");
+            $result = system { $webKitLauncherPath } $webKitLauncherPath, @ARGV;
         }
         return $result if $result;
     }