rebase -X: do not clobber strategy
authorMartin von Zweigbergk <martin.von.zweigbergk@gmail.com>
Wed, 10 Nov 2010 07:14:26 +0000 (08:14 +0100)
committerJunio C Hamano <gitster@pobox.com>
Wed, 10 Nov 2010 17:38:36 +0000 (09:38 -0800)
If any strategy options are passed to -X, the strategy will always be
set to 'recursive'. According to the documentation, it should default to
'recursive' if it is not set, but it should be possible to set it to
other values.

This fixes a regression introduced in v1.7.3-rc0~67^2 (2010-07-29).

Signed-off-by: Martin von Zweigbergk <martin.von.zweigbergk@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
git-rebase.sh

index e5df23b..10a238a 100755 (executable)
@@ -311,10 +311,6 @@ do
                esac
                strategy_opts="$strategy_opts $(git rev-parse --sq-quote "--$newopt")"
                do_merge=t
-               if test -n "$strategy"
-               then
-                       strategy=recursive
-               fi
                ;;
        -s=*|--s=*|--st=*|--str=*|--stra=*|--strat=*|--strate=*|\
                --strateg=*|--strategy=*|\