Summary: | test-webkitpy fails on Mac without iphoneos SDK | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Daniel Bates <dbates> | ||||
Component: | Tools / Tests | Assignee: | Daniel Bates <dbates> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | ap, commit-queue, ddkilzer, glenn | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | iPhone / iPad | ||||||
OS: | iOS 8.1 | ||||||
Attachments: |
|
Description
Daniel Bates
2015-02-04 12:07:58 PST
Created attachment 246041 [details]
Patch and unit test
As mentioned in the ChangeLog entry for PlatformInfo.xcode_sdk_version(), I took advantage of the behavior that xcrun --show-sdk-version only writes to standard output if the SDK is installed to return the empty string when the SDK is not installed. Let me know if it is preferred to explicitly check the exit status of xcrun and return the empty string instead of making use of this behavior.
Comment on attachment 246041 [details]
Patch and unit test
r=me
Comment on attachment 246041 [details] Patch and unit test View in context: https://bugs.webkit.org/attachment.cgi?id=246041&action=review > Tools/Scripts/webkitpy/port/ios.py:57 > + raise Exception("Ensure that the Xcode command line tools and the iphoneos SDK are installed.") Maybe add a "please" somewhere here? E.g. "Please install Xcode command line tools and the iPhone SDK". Also, why do we need command line tools? I thought that we didn't rely on them. (In reply to comment #4) > Comment on attachment 246041 [details] > Patch and unit test > > View in context: > https://bugs.webkit.org/attachment.cgi?id=246041&action=review > > > Tools/Scripts/webkitpy/port/ios.py:57 > > + raise Exception("Ensure that the Xcode command line tools and the iphoneos SDK are installed.") > > Maybe add a "please" somewhere here? E.g. "Please install Xcode command line > tools and the iPhone SDK". > > Also, why do we need command line tools? I thought that we didn't rely on > them. After talking with Alexey Proskuryakov on IRC today (02/04) we confirmed that xcrun is included with a default OS install. So, I will update the exception message to read: "Please install the iOS SDK." Committed r179622: <http://trac.webkit.org/changeset/179622> Fixed exception message per comment 5 and committed this in <http://trac.webkit.org/changeset/179629>. |