git-gui: Correct unmerged file detection at commit time.
Its impossible to commit an index which has unmerged stages. Unfortunately a bug in git-gui allowed the user to try to do exactly that, as we broke out of our file scanning loop as soon as we found a valid AMD index state. That's wrong, as the files are coming back from our array in pseudo-random order; an unmerged file may get returned only after all merged files. I also noticed the grammer around here in our dialog boxes still used the term 'include', so this has been updated to reflect current usage. Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
This commit is contained in:
parent
68c30b4af1
commit
14efcc7485
@ -896,12 +896,12 @@ The rescan will be automatically started now.
|
||||
_? {continue}
|
||||
A? -
|
||||
D? -
|
||||
M? {set files_ready 1; break}
|
||||
M? {set files_ready 1}
|
||||
U? {
|
||||
error_popup "Unmerged files cannot be committed.
|
||||
|
||||
File [short_path $path] has merge conflicts.
|
||||
You must resolve them and include the file before committing.
|
||||
You must resolve them and add the file before committing.
|
||||
"
|
||||
unlock_index
|
||||
return
|
||||
@ -915,9 +915,9 @@ File [short_path $path] cannot be committed by this program.
|
||||
}
|
||||
}
|
||||
if {!$files_ready} {
|
||||
error_popup {No included files to commit.
|
||||
error_popup {No changes to commit.
|
||||
|
||||
You must include at least 1 file before you can commit.
|
||||
You must add at least 1 file before you can commit.
|
||||
}
|
||||
unlock_index
|
||||
return
|
||||
|
Loading…
Reference in New Issue
Block a user