Bug 270507
| Summary: | Guide outside contributors to use run-minibrowser rather than run-safari | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | sideshowbarker <mike> |
| Component: | Tools / Tests | Assignee: | sideshowbarker <mike> |
| Status: | NEW | ||
| Severity: | Normal | CC: | webkit-bug-importer |
| Priority: | P2 | Keywords: | InRadar |
| Version: | WebKit Nightly Build | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| See Also: | https://bugs.webkit.org/show_bug.cgi?id=236968 | ||
sideshowbarker
Outside (non-Apple) contributors who aren’t building with the Apple Internal SDK — even those building on macOS — should be guided to use the run-minibrowser script rather than the run-safari script.
But unless the contributor is building the GTK or WPE port, the current build-webkit script is hardcoded to emit a “congrats” message saying this:
> To run Safari with this newly-built code, use the "Tools/Scripts/run-safari" script.
And the https://github.com/webkit/WebKit/?tab=readme-ov-file#running-webkit Readme.md file says:
> Run the following command to launch Safari with your local build of WebKit:
> Tools/Scripts/run-safari --debug
So the build-webkit script and the Readme.md file should be updated to instead guide all outside/non-Apple contributors — even those building on macOS — to use the run-minibrowser script.
Otherwise, it seems to be a common problem that outside contributors try the run-safari script, find that it doesn’t work expected, and then become confused and unsure how to proceed.
In the best case, their next step is that they show up in the Slack #help room asking what they’ve done wrong — in which case, someone else in the room quickly points out to them that they need to use the run-minibrowser script instead.
But the less-fortunate contributors can end up wasting time their time trying on their own to figure out what they’ve done wrong, and failing.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
sideshowbarker
Pull request: https://github.com/WebKit/WebKit/pull/25476
Radar WebKit Bug Importer
<rdar://problem/124433085>