RESOLVED FIXED 259831
[Xcode] Stop requiring public SDK modifications 
https://bugs.webkit.org/show_bug.cgi?id=259831
Summary [Xcode] Stop requiring public SDK modifications
Elliott Williams
Reported 2023-08-04 12:13:05 PDT
For years, we have required that `configure-xcode-for-embedded-development` is run before building WebKit for iOS/watchOS/tvOS with a public SDK. This is annoying for open-source developers, and a problem for CI systems where we have less control over the build environment (i.e. where the Xcode toolchain is provided for us). Let's gradually remove the things we need configure-xcode-for-embedded-development for, to the point where it can be removed entirely.
Attachments
Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2023-08-11 12:14:14 PDT
<rdar://problem/113764577>
Elliott Williams
Comment 2 2023-08-11 17:58:14 PDT
Pull request: https://github.com/WebKit/WebKit/pull/16632
EWS
Comment 3 2023-08-19 14:03:15 PDT
Committed 267073@main (d88653e7f78c): <https://commits.webkit.org/267073@main> Reviewed commits have been landed. Closing PR #16632 and removing active labels.
EWS
Comment 4 2023-08-20 00:18:25 PDT
Committed 267075@main (77443bc703ee): <https://commits.webkit.org/267075@main> Reviewed commits have been landed. Closing PR #16871 and removing active labels.
Elliott Williams
Comment 5 2023-08-23 13:22:24 PDT
Re-opening for pull request https://github.com/WebKit/WebKit/pull/16986
EWS
Comment 6 2023-08-24 11:06:53 PDT
Committed 267235@main (7e52ab84d9d5): <https://commits.webkit.org/267235@main> Reviewed commits have been landed. Closing PR #16986 and removing active labels.
EWS
Comment 7 2023-08-28 16:00:24 PDT
Committed 267370@main (4ee5a0c63c10): <https://commits.webkit.org/267370@main> Reviewed commits have been landed. Closing PR #17148 and removing active labels.
EWS
Comment 8 2023-12-14 12:19:45 PST
Committed 272063@main (0657884b0ee8): <https://commits.webkit.org/272063@main> Reviewed commits have been landed. Closing PR #21810 and removing active labels.
Note You need to log in before you can comment on or make changes to this bug.