run-safari --ios-simulator is no longer working with Xcode 11

[Frédéric Wang (:fredw)]

XCode relase and beta use different paths for MobileSafari.app:

$ find Xcode.app -name MobileSafari.app
Xcode.app/Contents/Developer/Platforms/iPhoneOS.platform/Developer/Library/CoreSimulator/Profiles/Runtimes/iOS.simruntime/Contents/Resources/RuntimeRoot/Applications/MobileSafari.app

$ find Xcode-beta.app -name MobileSafari.app
Xcode-beta.app/Contents/Developer/Platforms/iPhoneOS.platform/Library/Developer/CoreSimulator/Profiles/Runtimes/iOS.simruntime/Contents/Resources/RuntimeRoot/Applications/MobileSafari.app

(note that "Developer" and "Library" have been switched)

A trivial patch to make things work again is:

diff --git a/Tools/Scripts/webkitdirs.pm b/Tools/Scripts/webkitdirs.pm
index 1374b4b3f2..c011655669 100755
--- a/Tools/Scripts/webkitdirs.pm
+++ b/Tools/Scripts/webkitdirs.pm
@@ -2503,7 +2503,7 @@ sub setupIOSWebKitEnvironment($)
 sub iosSimulatorApplicationsPath()
 {
     # FIXME: We should ask simctl for this information, instead of guessing from available runtimes.
-    my $runtimePath = File::Spec->catdir(sdkPlatformDirectory("iphoneos"), "Developer", "Library", "CoreSimulator", "Profiles", "Runtimes");
+    my $runtimePath = File::Spec->catdir(sdkPlatformDirectory("iphoneos"), "Library", "Developer", "CoreSimulator", "Profiles", "Runtimes");
     opendir(RUNTIMES, $runtimePath);
     my @runtimes = grep {/.*\.simruntime/} readdir(RUNTIMES);
     close(RUNTIMES);

but of course that will break backward compatibility...

[Radar WebKit Bug Importer]

<rdar://problem/55738122>

[Frédéric Wang (:fredw)]

This bug now happens on Mojave with the recent XCode 11 release.

[Jonathan Bedard]

We can totally break iOS 12 at this point, so the 'trivial patch' is probably the way to go.

[Frédéric Wang (:fredw)]

Created attachment 379676 [details]
Patch

OK let's do that.

[WebKit Commit Bot]

Comment on attachment 379676 [details]
Patch

Clearing flags on attachment: 379676

Committed r250400: <https://trac.webkit.org/changeset/250400>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.