Bug 244653

Summary: [webkitdirs] Detect when internal builds are using the open-source workspace
Product: WebKit Reporter: Elliott Williams <emw>
Component: Tools / TestsAssignee: Elliott Williams <emw>
Status: REOPENED    
Severity: Normal CC: kkinnunen, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   

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
Radar WebKit Bug Importer
Comment 1 2022-08-31 19:05:48 PDT
Elliott Williams
Comment 2 2022-08-31 19:18:10 PDT
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
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.