WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
132975
old-run-webkit-tests: allow specifying a simulator app path
https://bugs.webkit.org/show_bug.cgi?id=132975
Summary
old-run-webkit-tests: allow specifying a simulator app path
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
Details
Formatted Diff
Diff
Patch
(2.15 KB, patch)
2014-05-16 08:47 PDT
,
David Farler
andersca
: review+
Details
Formatted Diff
Diff
Show Obsolete
(1)
View All
Add attachment
proposed patch, testcase, etc.
David Farler
Comment 1
2014-05-15 17:22:28 PDT
Created
attachment 231544
[details]
Patch
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
Created
attachment 231574
[details]
Patch
David Farler
Comment 5
2014-05-16 09:35:08 PDT
Committed
r168969
: <
http://trac.webkit.org/changeset/168969
>
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug