Bug 132975

Summary: old-run-webkit-tests: allow specifying a simulator app path
Product: WebKit Reporter: David Farler <dfarler>
Component: Tools / TestsAssignee: David Farler <dfarler>
Status: RESOLVED FIXED    
Severity: Normal CC: ddkilzer
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: iPhone / iPad   
OS: All   
Attachments:
Description Flags
Patch
none
Patch andersca: review+

David Farler
Reported 2014-05-15 16:43:20 PDT
Since the iOS simulator app bundle may not already be registered with launch services (multiple relocatable Xcode.app bundles may exist on the machine), provide a command line parameter to specify exactly where the simulator app is for the first launch.
Attachments
Patch (1.94 KB, patch)
2014-05-15 17:22 PDT, David Farler
no flags
Patch (2.15 KB, patch)
2014-05-16 08:47 PDT, David Farler
andersca: review+
David Farler
Comment 1 2014-05-15 17:22:28 PDT
David Kilzer (:ddkilzer)
Comment 2 2014-05-15 18:36:58 PDT
Comment on attachment 231544 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=231544&action=review r=me, but consider providing a more specific default as well based on $SDKROOT. > Tools/Scripts/old-run-webkit-tests:191 > +my $simulatorApp = 'iOS Simulator'; Instead of specifying a command-line argument, why not just construct the path based on the $SDKROOT path? It's always installed at a well-known location, so we should be able to construct the path using the $SDKROOT. I'm surprised there isn't a way to find it using 'xcrun -sdk iphonesimulator', but maybe that's only for command-line executables.
David Farler
Comment 3 2014-05-16 08:46:09 PDT
Roger. I guess it would be better to do something like the following, which should provide an accurate default. I guess most people aren't switching between Xcode.app bundles on an hourly basis :). chomp(my $developerDir = $ENV{DEVELOPER_DIR} || `xcode-select --print-path`); my $simulatorApp = "$developerDir/Applications/iOS Simulator.app"; unless (-d $simulatorApp) { $simulatorApp = "$developerDir/Applications/iPhone Simulator.app"; }
David Farler
Comment 4 2014-05-16 08:47:34 PDT
David Farler
Comment 5 2014-05-16 09:35:08 PDT
Note You need to log in before you can comment on or make changes to this bug.