WebKit Bugzilla
New
Browse
Search+
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED WONTFIX
239297
[git-webkit] pr command runs 'git rebase --continue' three times in a row
https://bugs.webkit.org/show_bug.cgi?id=239297
Summary
[git-webkit] pr command runs 'git rebase --continue' three times in a row
Michael Catanzaro
Reported
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.
Attachments
Add attachment
proposed patch, testcase, etc.
Ryan Haddad
Comment 1
2022-04-21 17:11:25 PDT
rdar://92044626
Jonathan Bedard
Comment 2
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.
Michael Catanzaro
Comment 3
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.
Jonathan Bedard
Comment 4
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"...
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