WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
NEW
245255
[git-webkit] prepare-commit-msg does not work with --reedit-message and --reuse-message
https://bugs.webkit.org/show_bug.cgi?id=245255
Summary
[git-webkit] prepare-commit-msg does not work with --reedit-message and --reu...
Elliott Williams
Reported
2022-09-15 16:36:17 PDT
These git-commit options allow you to use the commit message from a different commit, and optionally stop for editing. They allow you to "rewrite" a commit by resetting to `main`, staging new changes, then running `git commit --reedit-message "HEAD@{1}"` to make changes to the old commit message. Our prepare-commit-msg hook completely overwrites the commit message staged by these options, making them unusable. It should preserve the original message, and only list modified files in the comment section below the message body.
Attachments
Add attachment
proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1
2022-09-22 16:37:17 PDT
<
rdar://problem/100300158
>
Sam Sneddon [:gsnedders]
Comment 2
2023-05-15 14:14:18 PDT
This looks like it was fixed by
Bug 255121
, except when I just tried to do this I got: gsnedders@gsnedders-marsha webkitpy % git commit -c b318aede1356 Running status to find changed, added, or removed files. Reviewing diff to determine which lines changed. Extracting affected function names from source files. Change author: Sam Sneddon <
gsnedders@apple.com
>. Running status to find changed, added, or removed files. Reviewing diff to determine which lines changed. Extracting affected function names from source files. Change author: Abrar Rahman Protyasha <
a_protyasha@apple.com
>. And it displayed the author and commit message of `main` rather than of `b318aede1356`.
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