All of lore.kernel.org
 help / color / mirror / Atom feed
* commit following rebase has unexpected commit message
@ 2021-08-04 14:19 Victor Gambier
  2021-08-05 10:02 ` Phillip Wood
  0 siblings, 1 reply; 2+ messages in thread
From: Victor Gambier @ 2021-08-04 14:19 UTC (permalink / raw)
  To: git

[-- Attachment #1: Type: text/plain, Size: 1 bytes --]



[-- Attachment #2: git-bugreport-2021-08-04-1558.txt --]
[-- Type: text/plain, Size: 1751 bytes --]

Thank you for filling out a Git bug report!
Please answer the following questions to help us understand your issue.

What did you do before the bug happened? (Steps to reproduce your issue)

git switch -c first
git switch -c second
[modify a file]
git commit -am "foo"
git switch first
[modify the same file in a conflicting way]
git commit -am "bar"
git rebase second
git restore --staged .
git restore .
git rebase --continue
[modify any file - can be the same file again]
git commit -a

What did you expect to happen? (Expected behavior)

The text editor should pop up and the first line should be blank (no default commit message).

What happened instead? (Actual behavior)

The text editor pops up, and the first four lines of the default commit message seem to pertain to the rebase:

	bar
	
	# Conflicts:
	#	frontend/src/styling/rounds.ts
	
Even though the contents of the commit correctly pertain to the very last changes and are unrelated to the rebase.

What's different between what you expected and what actually happened?

The default commit message.

Anything else you want to add:

A second anomaly is that if you replace the last command with git commit -a --no-edit, the two lines starting with '#' are NOT ignored and will be present in the final commit message.

Please review the rest of the bug report below.
You can delete any lines you don't wish to share.


[System Info]
git version:
git version 2.30.0.windows.2
cpu: x86_64
built from commit: f8cbc844b81bf6b9e72178bbe891a86c8bf5e9e7
sizeof-long: 4
sizeof-size_t: 8
shell-path: /bin/sh
uname: Windows 10.0 18363 
compiler info: gnuc: 10.2
libc info: no libc information available
$SHELL (typically, interactive shell): C:\Program Files\Git\usr\bin\bash.exe


[Enabled Hooks]

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: commit following rebase has unexpected commit message
  2021-08-04 14:19 commit following rebase has unexpected commit message Victor Gambier
@ 2021-08-05 10:02 ` Phillip Wood
  0 siblings, 0 replies; 2+ messages in thread
From: Phillip Wood @ 2021-08-05 10:02 UTC (permalink / raw)
  To: Victor Gambier, git

Hi Victor

On 04/08/2021 15:19, Victor Gambier wrote:
 > Thank you for filling out a Git bug report!
 > Please answer the following questions to help us understand your issue.
 >
 > What did you do before the bug happened? (Steps to reproduce your issue)
 >
 > git switch -c first
 > git switch -c second
 > [modify a file]
 > git commit -am "foo"
 > git switch first
 > [modify the same file in a conflicting way]
 > git commit -am "bar"
 > git rebase second
 > git restore --staged .
 > git restore .
 > git rebase --continue
 > [modify any file - can be the same file again]
 > git commit -a
 >
 > What did you expect to happen? (Expected behavior)
 >
 > The text editor should pop up and the first line should be blank (no 
default commit message).
 >
 > What happened instead? (Actual behavior)
 >
 > The text editor pops up, and the first four lines of the default 
commit message seem to pertain to the rebase:
 >
 > 	bar
 >
 > 	# Conflicts:
 > 	#	frontend/src/styling/rounds.ts
 >
 > Even though the contents of the commit correctly pertain to the very 
last changes and are unrelated to the rebase.


Thanks for the detailed report, I think the problem is that when rebase 
stops for the user to resolve conflicts it writes .git/MERGE_MSG which 
is removed when the conflict resolution is committed or by 'git reset' 
if the user skips the commit. In this case the commit is effectively 
skipped but the file is not removed and so seeds the editor when you 
next make a commit. I'll try and put a patch together next week.

Best Wishes

Phillip

 > What's different between what you expected and what actually happened?
 >
 > The default commit message.
 >
 > Anything else you want to add:
 >
 > A second anomaly is that if you replace the last command with git 
commit -a --no-edit, the two lines starting with '#' are NOT ignored and 
will be present in the final commit message.
 >
 > Please review the rest of the bug report below.
 > You can delete any lines you don't wish to share.
 >
 >
 > [System Info]
 > git version:
 > git version 2.30.0.windows.2
 > cpu: x86_64
 > built from commit: f8cbc844b81bf6b9e72178bbe891a86c8bf5e9e7
 > sizeof-long: 4
 > sizeof-size_t: 8
 > shell-path: /bin/sh
 > uname: Windows 10.0 18363
 > compiler info: gnuc: 10.2
 > libc info: no libc information available
 > $SHELL (typically, interactive shell): C:\Program 
Files\Git\usr\bin\bash.exe
 >
 >
 > [Enabled Hooks]

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2021-08-05 10:02 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-08-04 14:19 commit following rebase has unexpected commit message Victor Gambier
2021-08-05 10:02 ` Phillip Wood

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.