[iOS] REGRESSION (r168910): webkitdirs.pm: Be more forgiving when parsing --sdk,...
authorddkilzer@apple.com <ddkilzer@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 20 May 2014 00:43:59 +0000 (00:43 +0000)
committerddkilzer@apple.com <ddkilzer@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 20 May 2014 00:43:59 +0000 (00:43 +0000)
<http://webkit.org/b/133089>

Reviewed by Darin Adler.

* Scripts/webkitdirs.pm:
(determineXcodeSDK): If both '--sdk SDKROOT' and '--sim' are
specified on the command line, we would remove the '--sdk'
arguments, but leave '--sim', potentially passing it along to
xcodebuild causing a build failure.  This happens when
'old-run-webkit-tests --release --sim' invokes
build-dumprendertree when DumpRenderTree hasn't been built yet.

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

Tools/ChangeLog
Tools/Scripts/webkitdirs.pm

index 4c3679c..71767c5 100644 (file)
@@ -1,3 +1,18 @@
+2014-05-19  David Kilzer  <ddkilzer@apple.com>
+
+        [iOS] REGRESSION (r168910): webkitdirs.pm: Be more forgiving when parsing --sdk, --device and --sim
+        <http://webkit.org/b/133089>
+
+        Reviewed by Darin Adler.
+
+        * Scripts/webkitdirs.pm:
+        (determineXcodeSDK): If both '--sdk SDKROOT' and '--sim' are
+        specified on the command line, we would remove the '--sdk'
+        arguments, but leave '--sim', potentially passing it along to
+        xcodebuild causing a build failure.  This happens when
+        'old-run-webkit-tests --release --sim' invokes
+        build-dumprendertree when DumpRenderTree hasn't been built yet.
+
 2014-05-19  David Farler  <dfarler@apple.com>
 
         Dashboard: OpenSource revision key changed from WebKitOpenSource to WebKit
index 4b7ab58..2c82099 100755 (executable)
@@ -398,11 +398,13 @@ sub determineXcodeSDK
     my $sdk;
     if (checkForArgumentAndRemoveFromARGVGettingValue("--sdk", \$sdk)) {
         $xcodeSDK = $sdk;
-    } elsif (checkForArgumentAndRemoveFromARGV("--device")) {
-        $xcodeSDK = 'iphoneos.internal';
-    } elsif (checkForArgumentAndRemoveFromARGV("--sim") ||
+    }
+    if (checkForArgumentAndRemoveFromARGV("--device")) {
+        $xcodeSDK ||= 'iphoneos.internal';
+    }
+    if (checkForArgumentAndRemoveFromARGV("--sim") ||
         checkForArgumentAndRemoveFromARGV("--simulator")) {
-        $xcodeSDK = 'iphonesimulator';
+        $xcodeSDK ||= 'iphonesimulator';
     }
 }