Bug 191275 - webkitpy: Generalize trailing SDK specifier
Summary: webkitpy: Generalize trailing SDK specifier
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Jonathan Bedard
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2018-11-05 12:10 PST by Jonathan Bedard
Modified: 2018-11-08 08:16 PST (History)
6 users (show)

See Also:


Attachments
Patch (3.32 KB, patch)
2018-11-05 12:17 PST, Jonathan Bedard
no flags Details | Formatted Diff | Diff
Patch (3.02 KB, patch)
2018-11-06 08:37 PST, Jonathan Bedard
no flags Details | Formatted Diff | Diff
Patch (4.16 KB, patch)
2018-11-06 12:57 PST, Jonathan Bedard
no flags Details | Formatted Diff | Diff
Patch (2.77 KB, patch)
2018-11-06 14:18 PST, Jonathan Bedard
commit-queue: commit-queue-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jonathan Bedard 2018-11-05 12:10:11 PST
List all installed SDKs with version numbers stripped out.
Comment 1 Jonathan Bedard 2018-11-05 12:10:28 PST
<rdar://problem/45703660>
Comment 2 Jonathan Bedard 2018-11-05 12:17:25 PST
Created attachment 353899 [details]
Patch
Comment 3 WebKit Commit Bot 2018-11-05 16:21:44 PST
Comment on attachment 353899 [details]
Patch

Clearing flags on attachment: 353899

Committed r237836: <https://trac.webkit.org/changeset/237836>
Comment 4 WebKit Commit Bot 2018-11-05 16:21:45 PST
All reviewed patches have been landed.  Closing bug.
Comment 5 Radar WebKit Bug Importer 2018-11-05 16:22:21 PST
<rdar://problem/45825307>
Comment 6 Jonathan Bedard 2018-11-06 08:37:16 PST
Reopening to attach new patch.
Comment 7 Jonathan Bedard 2018-11-06 08:37:17 PST
Created attachment 353970 [details]
Patch
Comment 8 Jonathan Bedard 2018-11-06 12:57:31 PST
Created attachment 353991 [details]
Patch
Comment 9 Aakash Jain 2018-11-06 13:07:49 PST
Comment on attachment 353991 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=353991&action=review

> Tools/Scripts/webkitpy/port/ios_simulator.py:84
> +        return ['--sdk', self.SDK] + (['ARCHS={}'.format(self.architecture())] if self.architecture() else [])

Is the architecture change unrelated? If so, it probably needs to be in a separate patch.
Comment 10 Jonathan Bedard 2018-11-06 14:18:16 PST
Created attachment 354004 [details]
Patch
Comment 11 Jonathan Bedard 2018-11-06 14:19:02 PST
(In reply to Aakash Jain from comment #9)
> Comment on attachment 353991 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=353991&action=review
> 
> > Tools/Scripts/webkitpy/port/ios_simulator.py:84
> > +        return ['--sdk', self.SDK] + (['ARCHS={}'.format(self.architecture())] if self.architecture() else [])
> 
> Is the architecture change unrelated? If so, it probably needs to be in a
> separate patch.

It was some code clean-up, I removed it. (It wasn't actually a functional change)
Comment 12 WebKit Commit Bot 2018-11-06 21:01:38 PST
Comment on attachment 354004 [details]
Patch

Clearing flags on attachment: 354004

Committed r237913: <https://trac.webkit.org/changeset/237913>
Comment 13 WebKit Commit Bot 2018-11-06 21:01:40 PST
All reviewed patches have been landed.  Closing bug.
Comment 14 Jonathan Bedard 2018-11-07 11:25:41 PST
Reverted r237913 for reason:

Breaks running tests on Mac without iOS SDK installed

Committed r237931: <https://trac.webkit.org/changeset/237931>
Comment 15 Jonathan Bedard 2018-11-07 15:50:49 PST
(In reply to Jonathan Bedard from comment #14)
> Reverted r237913 for reason:
> 
> Breaks running tests on Mac without iOS SDK installed
> 
> Committed r237931: <https://trac.webkit.org/changeset/237931>

I'll re-land this tonight, this patch was (and is) fine, there is just a specific local configuration problem that causes this patch to fail, I'll reach out to those who might  be impacted.
Comment 16 WebKit Commit Bot 2018-11-07 22:26:31 PST
Comment on attachment 354004 [details]
Patch

Rejecting attachment 354004 [details] from commit-queue.

Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.webkit.org', '--bot-id=webkit-cq-02', 'validate-changelog', '--check-oops', '--non-interactive', 354004, '--port=mac']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit

ChangeLog entry in Tools/ChangeLog contains OOPS!.

Full output: https://webkit-queues.webkit.org/results/9904967
Comment 17 Jonathan Bedard 2018-11-08 08:16:20 PST
Committed r237989: <https://trac.webkit.org/changeset/237989>