[webkit-dev] ChangeLog Deprecation in WebKit

Frédéric Wang fwang at igalia.com
Tue May 31 02:40:16 PDT 2022


Hello Jonathan,

Probably you are aware of that but 
https://github.com/WebKit/WebKit#contribute / 
https://webkit.org/contributing-code/ still mentions the ChangeLogs and 
old workflow.

Le 16/05/2022 à 22:46, Jonathan Bedard via webkit-dev a écrit :
> Starting tomorrow, Tuesday May 17th, WebKit will no longer be using 
> ChangeLogs and will instead require commit messages in new 
> commits. These were the norm when we started the project, but now they 
> are unusual, antiquated, and get in the way of common git tooling.
>
> This means that we need a commit message in every patch or PR that's 
> submitted for review. From prior discussion, it is very clear that 
> there are legitimate downsides too, and I want to take this 
> opportunity to suggest recommended paths forward.
>
> You will need to take action if you:
> - Have a pure-Subversion checkout
> - Draft patches without making local commits
> - Draft patches from multiple local commits
> - Have patches with ChangeLogs already in bugzilla
> - Have pull requests with ChangeLogs
>
> https://github.com/WebKit/WebKit/wiki/Migration covers migration to 
> GitHub workflows in general, but Tuesday’s change is quite a bit more 
> narrow in its effects. Below I’ve listed the actions you may need to take
>
> - Have a pure-Subversion checkout
>
> Follow https://github.com/WebKit/WebKit/wiki/Migration#subversion to 
> clone github.com/WebKit/WebKit <http://github.com/WebKit/WebKit> and 
> start developing from a GitHub checkout. The patch workflow works just 
> as well from a GitHub checkout. One significant difference is that 
> it's not possible to commit directly, but this is going away soon as 
> GitHub becomes the source of truth, so it is probably not worth 
> migrating to git-svn for. Also, some people find git-svn handy to 
> convert between revision formats; we have `git webkit find` as a 
> replacement.
>
> - Draft patches without making local commits
>
> Run `git-webkit setup` to configure your local git hooks to provide a 
> WebKit commit-message template. Before running `webkit-patch upload`, 
> run `git commit -a` to create a local commit with all local changes. 
> To amend that commit, run `git commit -a —amend`.
>
> - Draft patches from multiple local commits
>
> EWS will test patches with multiple commits in them, but Commit-Queue 
> will not land those patches. If you have a multi-commit patch (or 
> branch) applied locally, you need to squash them. I’d recommend `git 
> rebase -i <base>`. To do this, run `git rebase -i HEAD~3`, git will 
> then open your editor and offer to `pick` the last 3 commits. Keep 
> `pick` for the first listed commit, but use  `squash` for the other 
> two and close your editor, the last 3 commits will be squashed into a 
> single commit.
>
> - Have patches with ChangeLogs
>
> Apply your patch with `webkit-patch apply-from-bug <bug-number>`. If 
> we’ve already removed the ChangeLog files from the repository, this 
> patch application will fail. Resolve the conflicts, and commit your 
> local changes and draft a commit message.
>
> If we haven’t yet removed the ChangeLog files, your patch should 
> apply. In that case, make sure you’ve run `git-webkit setup` and 
> commit your local changes include the ChangeLog. This will create a 
> commit message derived from your ChangeLogs. Then, remove the 
> ChangeLog edits locally, amend your previous commit with `git commit 
> -a —amend` and re-upload your ChangeLog free patch.
>
> - Have pull requests with ChangeLogs
>
> Check out your pull request, remove any ChangeLog edits locally and 
> run `git-webkit pr --amend`.
>
> Jonathan
> WebKit Continuous Integration
>
>
> _______________________________________________
> webkit-dev mailing list
> webkit-dev at lists.webkit.org
> https://lists.webkit.org/mailman/listinfo/webkit-dev


-- 
Frédéric Wang
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-dev/attachments/20220531/4b72f540/attachment.htm>


More information about the webkit-dev mailing list