Browse Source

Detect ls-remote failure properly.

The part that can fail is before the pipe, so we need to propagate the
error properly to the main process.

Signed-off-by: Junio C Hamano <junkio@cox.net>
maint
Junio C Hamano 20 years ago
parent
commit
a5cd85e084
  1. 15
      git-ls-remote.sh

15
git-ls-remote.sh

@ -42,12 +42,16 @@ http://* | https://* )
if [ -n "$GIT_SSL_NO_VERIFY" ]; then if [ -n "$GIT_SSL_NO_VERIFY" ]; then
curl_extra_args="-k" curl_extra_args="-k"
fi fi
curl -nsf $curl_extra_args "$peek_repo/info/refs" || exit 1 curl -nsf $curl_extra_args "$peek_repo/info/refs" ||
echo "failed slurping"
;; ;;


rsync://* ) rsync://* )
mkdir $tmpdir mkdir $tmpdir
rsync -rq "$peek_repo/refs" $tmpdir || exit 1 rsync -rq "$peek_repo/refs" $tmpdir || {
echo "failed slurping"
exit
}
(cd $tmpdir && find refs -type f) | (cd $tmpdir && find refs -type f) |
while read path while read path
do do
@ -58,12 +62,17 @@ rsync://* )
;; ;;


* ) * )
git-peek-remote "$peek_repo" git-peek-remote "$peek_repo" ||
echo "failed slurping"
;; ;;
esac | esac |
sort -t ' ' -k 2 | sort -t ' ' -k 2 |
while read sha1 path while read sha1 path
do do
case "$sha1" in
failed)
die "Failed to find remote refs"
esac
case "$path" in case "$path" in
refs/heads/*) refs/heads/*)
group=heads ;; group=heads ;;

Loading…
Cancel
Save