Changeset 223234 in webkit

Timestamp:

Oct 12, 2017 1:37:53 AM (7 years ago)

Author:

fred.wang@free.fr

Message:

run-safari --ios-simulator is no longer working with Xcode 9
​https://bugs.webkit.org/show_bug.cgi?id=177595

Patch by Frederic Wang <​fwang@igalia.com> on 2017-10-11
Reviewed by Daniel Bates.

In Xcode 9, the path of the simulator application is now in a new CoreSimulator subdirectory
of the iphoneOS SDK platform. This patch updates webkitdirs.pm to use the new path in
Xcode 9 or higher so that it is possible to run Safari on the iOS simulator again. It also
does some minor code refactoring to improve code reuse.

• Scripts/configure-xcode-for-ios-development: Move sdkDirectory() and sdkPlatformDirectory()

into webkitdirs.
(sdkDirectory): Deleted.
(sdkPlatformDirectory): Deleted.

• Scripts/webkitdirs.pm: Expose new sdkDirectory() and sdkPlatformDirectory().

(sdkDirectory): Moved from configure-xcode-for-ios-development.
(sdkPlatformDirectory): Ditto.
(XcodeSDKPath): Rely on sdkDirectory() to implement this function.
(iosSimulatorApplicationsPath): In Xcode 9 or higher use the new path. It is calculated
by relying on sdkPlatformDirectory().

Location:

trunk/Tools

Files:

• ChangeLog (modified) (1 diff)
• Scripts/configure-xcode-for-ios-development (modified) (2 diffs)
• Scripts/webkitdirs.pm (modified) (5 diffs)

trunk/Tools/ChangeLog

@@ +1 @@
+2017-10-11  Frederic Wang  <fwang@igalia.com>
+
+        run-safari --ios-simulator is no longer working with Xcode 9
+        https://bugs.webkit.org/show_bug.cgi?id=177595
+
+        Reviewed by Daniel Bates.
+
+        In Xcode 9, the path of the simulator application is now in a new CoreSimulator subdirectory
+        of the iphoneOS SDK platform. This patch updates webkitdirs.pm to use the new path in
+        Xcode 9 or higher so that it is possible to run Safari on the iOS simulator again. It also
+        does some minor code refactoring to improve code reuse.
+
+        * Scripts/configure-xcode-for-ios-development: Move sdkDirectory() and sdkPlatformDirectory()
+        into webkitdirs.
+        (sdkDirectory): Deleted.
+        (sdkPlatformDirectory): Deleted.
+        * Scripts/webkitdirs.pm: Expose new sdkDirectory() and sdkPlatformDirectory().
+        (sdkDirectory): Moved from configure-xcode-for-ios-development.
+        (sdkPlatformDirectory): Ditto.
+        (XcodeSDKPath): Rely on sdkDirectory() to implement this function.
+        (iosSimulatorApplicationsPath): In Xcode 9 or higher use the new path. It is calculated
+        by relying on sdkPlatformDirectory().
+
 2017-10-11  Dewei Zhu  <dewei_zhu@apple.com>
 

trunk/Tools/Scripts/configure-xcode-for-ios-development

@@ -46 +46 @@
 sub mergeXcodeSpecificationWithSpecificationAndId($$$);
 sub readXcodeSpecificationById($$);
-sub sdkDirectory($);
-sub sdkPlatformDirectory($);
 sub updateXcode7SpecificationFile($);
 sub updateXcodeSpecificationFilesForSDKIfNeeded($);
@@ -213 +211 @@
 }
 
-sub sdkDirectory($)
-{
-    my ($sdkName) = @_;
-    chomp(my $sdkDirectory = `xcrun --sdk '$sdkName' --show-sdk-path`);
-    die "Failed to get SDK path from xcrun: $!" if exitStatus($?);
-    return $sdkDirectory;
-}
-
-sub sdkPlatformDirectory($)
-{
-    my ($sdkName) = @_;
-    chomp(my $sdkPlatformDirectory = `xcrun --sdk '$sdkName' --show-sdk-platform-path`);
-    die "Failed to get SDK platform path from xcrun: $!" if exitStatus($?);
-    return $sdkPlatformDirectory;
-}
-
 sub writeXcodeSpecification($$)
 {

trunk/Tools/Scripts/webkitdirs.pm

@@ -72 +72 @@
        &findOrCreateSimulatorForIOSDevice
        &iosSimulatorDeviceByName
+       &iosVersion
        &nmPath
        &passedConfiguration
@@ -83 +84 @@
        &runMacWebKitApp
        &safariPath
-       &iosVersion
+       &sdkDirectory
+       &sdkPlatformDirectory
        &setConfiguration
        &setupMacWebKitEnvironment
@@ -166 +168 @@
 sub hasArgument($$);
 
+sub sdkDirectory($)
+{
+    my ($sdkName) = @_;
+    chomp(my $sdkDirectory = `xcrun --sdk '$sdkName' --show-sdk-path`);
+    die "Failed to get SDK path from xcrun: $!" if exitStatus($?);
+    return $sdkDirectory;
+}
+
+sub sdkPlatformDirectory($)
+{
+    my ($sdkName) = @_;
+    chomp(my $sdkPlatformDirectory = `xcrun --sdk '$sdkName' --show-sdk-platform-path`);
+    die "Failed to get SDK platform path from xcrun: $!" if exitStatus($?);
+    return $sdkPlatformDirectory;
+}
+
 sub determineSourceDir
 {
@@ -541 +559 @@
 
     die "Can't find the SDK path because no Xcode SDK was specified" if not $xcodeSDK;
-
-    my $sdkPath = `xcrun --sdk $xcodeSDK --show-sdk-path` if $xcodeSDK;
-    die 'Failed to get SDK path from xcrun' if $?;
-    chomp $sdkPath;
-
-    return $sdkPath;
+    return sdkDirectory($xcodeSDK);
 }
 
@@ -2311 +2324 @@
 sub iosSimulatorApplicationsPath()
 {
-    return File::Spec->catdir(XcodeSDKPath(), "Applications");
+    determineXcodeVersion();
+    if (eval "v$xcodeVersion" lt v9.0) {
+        return File::Spec->catdir(XcodeSDKPath(), "Applications");
+    }
+
+    my $iphoneOSPlatformPath = sdkPlatformDirectory("iphoneos");
+    return File::Spec->catdir($iphoneOSPlatformPath, "Developer", "Library", "CoreSimulator", "Profiles", "Runtimes", "iOS.simruntime", "Contents", "Resources", "RuntimeRoot", "Applications");
 }