WebKit Bugzilla
New
Browse
Search+
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
REOPENED
244653
[webkitdirs] Detect when internal builds are using the open-source workspace
https://bugs.webkit.org/show_bug.cgi?id=244653
Summary
[webkitdirs] Detect when internal builds are using the open-source workspace
Elliott Williams
Reported
2022-08-31 19:05:12 PDT
It's easy for an Apple engineer to accidentally try to build WebKit from the open-source workspace, rather than an internal workspace which contains extra bits needed to compile WebKit successfully on internal SDKs. The failure mode is pretty cryptic: builds usually fail with missing symbols during linking, etc. Let's check for this condition before building, so build-webkit or Make can fail early with a useful message.
Attachments
Add attachment
proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1
2022-08-31 19:05:48 PDT
<
rdar://problem/99415058
>
Elliott Williams
Comment 2
2022-08-31 19:18:10 PDT
Pull request:
https://github.com/WebKit/WebKit/pull/3901
EWS
Comment 3
2022-09-01 12:06:00 PDT
Committed
254059@main
(d398a88b431e): <
https://commits.webkit.org/254059@main
> Reviewed commits have been landed. Closing PR #3901 and removing active labels.
EWS
Comment 4
2022-09-01 13:21:38 PDT
Committed
254064@main
(1c3a52cd871e): <
https://commits.webkit.org/254064@main
> Reviewed commits have been landed. Closing PR #3927 and removing active labels.
Elliott Williams
Comment 5
2022-09-01 16:21:22 PDT
Reverted by
https://github.com/WebKit/WebKit/pull/3938
EWS
Comment 6
2022-09-01 18:12:58 PDT
Committed
254083@main
(3f0eab0693e7): <
https://commits.webkit.org/254083@main
> Reviewed commits have been landed. Closing PR #3938 and removing active labels.
Elliott Williams
Comment 7
2022-09-02 10:37:00 PDT
Reverted because this change was breaking builds when a Makefile sets WORKSPACE_PATH.
Elliott Williams
Comment 8
2022-10-06 13:46:18 PDT
Another thing we could do here, once this is functional, is to have webkitdirs.pm change its default workspace to the internal one when the SDK ends in ".internal".
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