Summary: | old-run-webkit-tests: allow specifying a simulator app path | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | David Farler <dfarler> | ||||||
Component: | Tools / Tests | Assignee: | David Farler <dfarler> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | ddkilzer | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | iPhone / iPad | ||||||||
OS: | All | ||||||||
Attachments: |
|
Description
David Farler
2014-05-15 16:43:20 PDT
Created attachment 231544 [details]
Patch
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. 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"; } Created attachment 231574 [details]
Patch
Committed r168969: <http://trac.webkit.org/changeset/168969> |