build-webkit prints an error on iOS simulator bots
authorap@apple.com <ap@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 6 Mar 2016 08:33:29 +0000 (08:33 +0000)
committerap@apple.com <ap@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sun, 6 Mar 2016 08:33:29 +0000 (08:33 +0000)
https://bugs.webkit.org/show_bug.cgi?id=155078

Reviewed by Dan Bernstein.

This happens when build-layouttestrelay sets $xcodeSDK to undefined, to use host SDK.

* Scripts/webkitdirs.pm: (buildXCodeProject): There is no need to pass -sdk, because
XcodeOptions() already passes SDKROOT as appropriate.

* Scripts/build-layouttestrelay: Added a FIXME. Resetting $xcodeSDK doesn't reset
any other configuration variables that were derived from its initial value, like $portName.

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

Tools/ChangeLog
Tools/Scripts/build-layouttestrelay
Tools/Scripts/webkitdirs.pm

index 62de80f..3d7e4c8 100644 (file)
@@ -1,3 +1,18 @@
+2016-03-06  Alexey Proskuryakov  <ap@apple.com>
+
+        build-webkit prints an error on iOS simulator bots
+        https://bugs.webkit.org/show_bug.cgi?id=155078
+
+        Reviewed by Dan Bernstein.
+
+        This happens when build-layouttestrelay sets $xcodeSDK to undefined, to use host SDK.
+
+        * Scripts/webkitdirs.pm: (buildXCodeProject): There is no need to pass -sdk, because
+        XcodeOptions() already passes SDKROOT as appropriate.
+
+        * Scripts/build-layouttestrelay: Added a FIXME. Resetting $xcodeSDK doesn't reset
+        any other configuration variables that were derived from its initial value, like $portName.
+
 2016-03-05  Tim Horton  <timothy_horton@apple.com>
 
         Create a DOMHTMLVideoElement when wrapping <video> elements
index ceee808..ee40a97 100755 (executable)
@@ -60,6 +60,8 @@ chdir "Tools/LayoutTestRelay" or die;
 #        to avoid breaking some Apple Internal tools we support calling this script
 #        without --ios-simulator so long as we are building on a Mac.
 if (isAppleMacWebKit() && !willUseIOSDeviceSDK()) {
+    # FIXME: Changing SDK at this point results in an inconsistent webkitdirs state.
+    # We should make it so that webkitdirs functions never see the wrong SDK.
     setXcodeSDK(undef); # Force use of Mac SDK if --ios-simulator was passed
     exit exitStatus(buildXCodeProject("LayoutTestRelay", $clean, XcodeOptions(), @ARGV));
 }
index 1797644..94cb8ec 100755 (executable)
@@ -1748,8 +1748,6 @@ sub buildXCodeProject($$@)
         push(@extraOptions, "clean");
     }
 
-    push(@extraOptions, ("-sdk", xcodeSDK())) if isIOSWebKit();
-
     chomp($ENV{DSYMUTIL_NUM_THREADS} = `sysctl -n hw.activecpu`);
     return system "xcodebuild", "-project", "$project.xcodeproj", @extraOptions;
 }