WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
NEW
270507
Guide outside contributors to use run-minibrowser rather than run-safari
https://bugs.webkit.org/show_bug.cgi?id=270507
Summary
Guide outside contributors to use run-minibrowser rather than run-safari
sideshowbarker
Reported
2024-03-04 22:47:06 PST
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
Comment 1
2024-03-04 23:56:38 PST
Pull request:
https://github.com/WebKit/WebKit/pull/25476
Radar WebKit Bug Importer
Comment 2
2024-03-11 23:48:13 PDT
<
rdar://problem/124433085
>
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