Teach git-pull to pass -X<option> to git-merge
authorAvery Pennarun <apenwarr@gmail.com>
Thu, 26 Nov 2009 02:23:57 +0000 (21:23 -0500)
committerJunio C Hamano <gitster@pobox.com>
Mon, 18 Jan 2010 06:45:17 +0000 (22:45 -0800)
This needs the usual sq then eval trick to allow IFS characters
in the option.

Signed-off-by: Avery Pennarun <apenwarr@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
git-pull.sh
t/t6037-merge-ours-theirs.sh

index bfeb4a0ff65fabaa49b212411ee2451cbf656f3d..6d961b6995d3760f8d5855a66619739823ee2c37 100755 (executable)
@@ -18,6 +18,7 @@ test -z "$(git ls-files -u)" ||
 
 strategy_args= diffstat= no_commit= squash= no_ff= ff_only=
 log_arg= verbosity=
+merge_args=
 curr_branch=$(git symbolic-ref -q HEAD)
 curr_branch_short=$(echo "$curr_branch" | sed "s|refs/heads/||")
 rebase=$(git config --bool branch.$curr_branch_short.rebase)
@@ -62,6 +63,18 @@ do
                esac
                strategy_args="${strategy_args}-s $strategy "
                ;;
+       -X*)
+               case "$#,$1" in
+               1,-X)
+                       usage ;;
+               *,-X)
+                       xx="-X $2"
+                       shift ;;
+               *,*)
+                       xx="$1" ;;
+               esac
+               merge_args="$merge_args$xx "
+               ;;
        -r|--r|--re|--reb|--reba|--rebas|--rebase)
                rebase=true
                ;;
@@ -216,7 +229,7 @@ fi
 
 merge_name=$(git fmt-merge-msg $log_arg <"$GIT_DIR/FETCH_HEAD") || exit
 test true = "$rebase" &&
-       exec git-rebase $diffstat $strategy_args --onto $merge_head \
+       exec git-rebase $diffstat $strategy_args $merge_args --onto $merge_head \
        ${oldremoteref:-$merge_head}
-exec git-merge $diffstat $no_commit $squash $no_ff $ff_only $log_arg $strategy_args \
+exec git-merge $diffstat $no_commit $squash $no_ff $ff_only $log_arg $strategy_args $merge_args \
        "$merge_name" HEAD $merge_head $verbosity
index 08c9f7989a578d57a3c0f868786fa6b4cc2bf777..8ab3d61f445725ec48b8b4ce28cfbf4a9ce13844 100755 (executable)
@@ -53,4 +53,12 @@ test_expect_success 'recursive favouring ours' '
        ! grep 1 file
 '
 
+test_expect_success 'pull with -X' '
+       git reset --hard master && git pull -s recursive -Xours . side &&
+       git reset --hard master && git pull -s recursive -X ours . side &&
+       git reset --hard master && git pull -s recursive -Xtheirs . side &&
+       git reset --hard master && git pull -s recursive -X theirs . side &&
+       git reset --hard master && ! git pull -s recursive -X bork . side
+'
+
 test_done