Skip to content

Commit

Permalink
difftool: remove merge options for opendiff, tkdiff, kdiff3 and xxdiff
Browse files Browse the repository at this point in the history
We shouldn't try to merge files when using difftool, so remove
any merge-specific options.

Signed-off-by: David Aguilar <[email protected]>
Signed-off-by: Junio C Hamano <[email protected]>
  • Loading branch information
davvid authored and gitster committed Apr 8, 2009
1 parent b98c212 commit 76ca653
Showing 1 changed file with 3 additions and 7 deletions.
10 changes: 3 additions & 7 deletions contrib/difftool/git-difftool-helper
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,7 @@ launch_merge_tool () {
"$merge_tool_path" --auto \
--L1 "$basename (A)" \
--L2 "$basename (B)" \
-o "$MERGED" "$LOCAL" "$REMOTE" \
"$LOCAL" "$REMOTE" \
> /dev/null 2>&1
;;

Expand All @@ -78,7 +78,7 @@ launch_merge_tool () {
;;

tkdiff)
"$merge_tool_path" -o "$MERGED" "$LOCAL" "$REMOTE"
"$merge_tool_path" "$LOCAL" "$REMOTE"
;;

meld)
Expand All @@ -95,17 +95,13 @@ launch_merge_tool () {

xxdiff)
"$merge_tool_path" \
-X \
-R 'Accel.SaveAsMerged: "Ctrl-S"' \
-R 'Accel.Search: "Ctrl+F"' \
-R 'Accel.SearchForward: "Ctrl-G"' \
--merged-file "$MERGED" \
"$LOCAL" "$REMOTE"
;;

opendiff)
"$merge_tool_path" "$LOCAL" "$REMOTE" \
-merge "$MERGED" | cat
"$merge_tool_path" "$LOCAL" "$REMOTE" | cat
;;

ecmerge)
Expand Down

0 comments on commit 76ca653

Please sign in to comment.