Browse Source

git-gui: Warn users before making an octopus merge

A coworker who was new to git-gui recently tried to make an octopus
merge when he did not quite mean to.  Unfortunately in his case the
branches had file level conflicts and failed to merge with the octopus
strategy, and he didn't quite know why this happened.  Since most users
really don't want to perform an octopus merge this additional safety
valve in front of the merge process is a good thing.

Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
maint
Shawn O. Pearce 18 years ago
parent
commit
1afd1ec107
  1. 13
      git-gui.sh

13
git-gui.sh

@ -2886,6 +2886,19 @@ proc start_local_merge_action {w} { @@ -2886,6 +2886,19 @@ proc start_local_merge_action {w} {
set unit branch
} elseif {$revcnt <= 15} {
set unit branches

if {[tk_dialog \
$w.confirm_octopus \
[wm title $w] \
"Use octopus merge strategy?

You are merging $revcnt branches at once. This requires using the octopus merge driver, which may not succeed if there are file-level conflicts.
" \
question \
0 \
{Cancel} \
{Use octopus} \
] != 1} return
} else {
tk_messageBox \
-icon error \

Loading…
Cancel
Save