Bug 239297 - [git-webkit] pr command runs 'git rebase --continue' three times in a row
Summary: [git-webkit] pr command runs 'git rebase --continue' three times in a row
Status: RESOLVED WONTFIX
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:
 
Reported: 2022-04-13 12:29 PDT by Michael Catanzaro
Modified: 2022-05-02 10:43 PDT (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Catanzaro 2022-04-13 12:29:33 PDT
Looking at the output of 'git-webkit pr' that I posted in bug #239296, I notice something interesting:

Running 'git rebase --continue'...
Successfully rebased and updated refs/heads/eng/Misc-compiler-warnings-April-2022-edition.
Running 'git rebase --continue'...
fatal: No rebase in progress?
Running 'git rebase --continue'...
fatal: No rebase in progress?
Successfully rebased and updated .
Rewrite 8734aa0fbbe6b93d301c2867aa7a6c19daf1bef3 (1/1) (0 seconds passed, remaining 0 predicted)    
WARNING: Ref 'refs/remotes/origin/main' is unchanged
Ref 'refs/heads/eng/Misc-compiler-warnings-April-2022-edition' was rewritten

Seems git-webkit attempted to rebase three times. O_O That's probably not right.
Comment 1 Ryan Haddad 2022-04-21 17:11:25 PDT
rdar://92044626
Comment 2 Jonathan Bedard 2022-04-28 14:47:46 PDT
Do you know the kind of change this was run on? That has to do with conflict resolution in changelogs, I wonder if you get a different prompt for each changelog.
Comment 3 Michael Catanzaro 2022-04-28 15:25:47 PDT
Looks like it was https://github.com/WebKit/WebKit/pull/282. And there are indeed three changelogs involved there: JavaScriptCore, WebCore, and WebKit.
Comment 4 Jonathan Bedard 2022-05-02 10:43:19 PDT
(In reply to Michael Catanzaro from comment #3)
> Looks like it was https://github.com/WebKit/WebKit/pull/282. And there are
> indeed three changelogs involved there: JavaScriptCore, WebCore, and WebKit.

Looks like another point for "get rid of ChangeLogs"...