summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorLibravatar Eric Wong <normalperson@yhbt.net>2006-06-21 03:04:42 -0700
committerLibravatar Junio C Hamano <junkio@cox.net>2006-06-21 03:56:30 -0700
commit693c15dc282c36042eac8d215beae3067db7565c (patch)
treee61b9d272c16360ebca965a77d93feff4e2bfae7
parentAdd renaming-rebase test. (diff)
downloadtgif-693c15dc282c36042eac8d215beae3067db7565c.tar.xz
rebase: error out for NO_PYTHON if they use recursive merge
recursive merge relies on Python, and we can't perform rename-aware merges without the recursive merge. So bail out before trying it. The test won't work w/o recursive merge, either, so skip that, too. Signed-off-by: Eric Wong <normalperson@yhbt.net> Signed-off-by: Junio C Hamano <junkio@cox.net>
-rwxr-xr-xgit-rebase.sh9
-rwxr-xr-xt/t3402-rebase-merge.sh6
2 files changed, 15 insertions, 0 deletions
diff --git a/git-rebase.sh b/git-rebase.sh
index bce7bf84d6..b9ce1125d8 100755
--- a/git-rebase.sh
+++ b/git-rebase.sh
@@ -300,6 +300,15 @@ then
exit $?
fi
+if test "@@NO_PYTHON@@" && test "$strategy" = "recursive"
+then
+ die 'The recursive merge strategy currently relies on Python,
+which this installation of git was not configured with. Please consider
+a different merge strategy (e.g. octopus, resolve, stupid, ours)
+or install Python and git with Python support.'
+
+fi
+
# start doing a rebase with git-merge
# this is rename-aware if the recursive (default) strategy is used
diff --git a/t/t3402-rebase-merge.sh b/t/t3402-rebase-merge.sh
index 0779aaa9ab..d34c6cf6f3 100755
--- a/t/t3402-rebase-merge.sh
+++ b/t/t3402-rebase-merge.sh
@@ -7,6 +7,12 @@ test_description='git rebase --merge test'
. ./test-lib.sh
+if test "$no_python"; then
+ echo "Skipping: no python => no recursive merge"
+ test_done
+ exit 0
+fi
+
T="A quick brown fox
jumps over the lazy dog."
for i in 1 2 3 4 5 6 7 8 9 10