Bug 239796 - [git-webkit] Support reverting multiple commits at once
Summary: [git-webkit] Support reverting multiple commits at once
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks: 239082
  Show dependency tree / graph
 
Reported: 2022-04-26 17:16 PDT by Elliott Williams
Modified: 2023-12-05 18:46 PST (History)
3 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Elliott Williams 2022-04-26 17:16:18 PDT 
Maybe I'm missing something, but git-webkit revert doesn't appear to accept multiple arguments:

    > git webkit revert -b 239681 2e690e19e3caf94f5f0fac2f15303212c006318b 433f73bc052d7c978c619e28b2574df8faf6cb72 
    usage: git-webkit [-h] [--verbose] [--quiet] [--path REPOSITORY]
                      {blame,branch,canonicalize,checkout,clean,find,info,land,log,pull,up,update,pull-request,pr,pfr,upload,revert,setup,install-git-lfs,credentials,setup-git-svn} ...
    git-webkit: error: unrecognized arguments: 433f73bc052d7c978c619e28b2574df8faf6cb72


I tried comma-separated commits (like webkit-patch) and `..` commit ranges, too.

Plain `git revert` would be great for preparing a revert branch, except for changelogs, which it tries to undo changes to.
Comment 1 Alexey Proskuryakov 2022-04-28 15:43:22 PDT 
This is a very common use case.
Comment 2 Radar WebKit Bug Importer 2022-05-03 17:17:19 PDT 
<rdar://problem/92702616>
Comment 3 Brianna Fan 2023-11-28 17:33:12 PST 
Pull request: https://github.com/WebKit/WebKit/pull/21033
Comment 4 EWS 2023-12-05 18:46:11 PST 
Committed 271587@main (627b31c8383b): <https://commits.webkit.org/271587@main>

Reviewed commits have been landed. Closing PR #21033 and removing active labels.