81d340d40a
If a push fails because the remote-helper died (with fast-export), the user may not see any error message. We do correctly die with a failed exit code, as we notice that the helper has died while reading back the ref status from the helper. However, we don't print any message. This is OK if the helper itself printed a useful error message, but we cannot count on that; let's let the user know that the helper failed. In the long run, it may make more sense to propagate the error back up to push, so that it can present the usual status table and give a nicer message. But this is a much simpler fix that can help immediately. While we're adding tests, let's also confirm that the remote-helper dying is also detected when importing refs. We currently do so robustly when the helper uses the "done" feature (and that is what we test). We cannot do so reliably when the helper does not use the "done" feature, but it is not even worth testing; the right solution is for the helper to start using "done". Suggested-by: Jeff King <peff@peff.net> Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com> Signed-off-by: Jeff King <peff@peff.net> Acked-by: Sverre Rabbelier <srabbelier@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>
110 lines
2.1 KiB
Bash
Executable File
110 lines
2.1 KiB
Bash
Executable File
#!/usr/bin/env bash
|
|
# Copyright (c) 2012 Felipe Contreras
|
|
|
|
alias=$1
|
|
url=$2
|
|
|
|
dir="$GIT_DIR/testgit/$alias"
|
|
prefix="refs/testgit/$alias"
|
|
|
|
default_refspec="refs/heads/*:${prefix}/heads/*"
|
|
|
|
refspec="${GIT_REMOTE_TESTGIT_REFSPEC-$default_refspec}"
|
|
|
|
test -z "$refspec" && prefix="refs"
|
|
|
|
export GIT_DIR="$url/.git"
|
|
|
|
mkdir -p "$dir"
|
|
|
|
if test -z "$GIT_REMOTE_TESTGIT_NO_MARKS"
|
|
then
|
|
gitmarks="$dir/git.marks"
|
|
testgitmarks="$dir/testgit.marks"
|
|
test -e "$gitmarks" || >"$gitmarks"
|
|
test -e "$testgitmarks" || >"$testgitmarks"
|
|
testgitmarks_args=( "--"{import,export}"-marks=$testgitmarks" )
|
|
fi
|
|
|
|
while read line
|
|
do
|
|
case $line in
|
|
capabilities)
|
|
echo 'import'
|
|
echo 'export'
|
|
test -n "$refspec" && echo "refspec $refspec"
|
|
if test -n "$gitmarks"
|
|
then
|
|
echo "*import-marks $gitmarks"
|
|
echo "*export-marks $gitmarks"
|
|
fi
|
|
echo
|
|
;;
|
|
list)
|
|
git for-each-ref --format='? %(refname)' 'refs/heads/'
|
|
head=$(git symbolic-ref HEAD)
|
|
echo "@$head HEAD"
|
|
echo
|
|
;;
|
|
import*)
|
|
# read all import lines
|
|
while true
|
|
do
|
|
ref="${line#* }"
|
|
refs="$refs $ref"
|
|
read line
|
|
test "${line%% *}" != "import" && break
|
|
done
|
|
|
|
if test -n "$gitmarks"
|
|
then
|
|
echo "feature import-marks=$gitmarks"
|
|
echo "feature export-marks=$gitmarks"
|
|
fi
|
|
|
|
if test -n "$GIT_REMOTE_TESTGIT_FAILURE"
|
|
then
|
|
echo "feature done"
|
|
exit 1
|
|
fi
|
|
|
|
echo "feature done"
|
|
git fast-export "${testgitmarks_args[@]}" $refs |
|
|
sed -e "s#refs/heads/#${prefix}/heads/#g"
|
|
echo "done"
|
|
;;
|
|
export)
|
|
if test -n "$GIT_REMOTE_TESTGIT_FAILURE"
|
|
then
|
|
# consume input so fast-export doesn't get SIGPIPE;
|
|
# git would also notice that case, but we want
|
|
# to make sure we are exercising the later
|
|
# error checks
|
|
while read line; do
|
|
test "done" = "$line" && break
|
|
done
|
|
exit 1
|
|
fi
|
|
|
|
before=$(git for-each-ref --format='%(refname) %(objectname)')
|
|
|
|
git fast-import "${testgitmarks_args[@]}" --quiet
|
|
|
|
after=$(git for-each-ref --format='%(refname) %(objectname)')
|
|
|
|
# figure out which refs were updated
|
|
join -e 0 -o '0 1.2 2.2' -a 2 <(echo "$before") <(echo "$after") |
|
|
while read ref a b
|
|
do
|
|
test $a == $b && continue
|
|
echo "ok $ref"
|
|
done
|
|
|
|
echo
|
|
;;
|
|
'')
|
|
exit
|
|
;;
|
|
esac
|
|
done
|