X-Git-Url: http://www.chiark.greenend.org.uk/ucgi/~ianmdlvl/git?p=dgit.git;a=blobdiff_plain;f=infra%2Fdgit-mirror-rsync;h=2d912c8ad1dd30f335c2f52d8e4f6011a718a22c;hp=2858047b57f85307085c1df13f0448d69e29473e;hb=f1d68d5945334f10b96a94f129556656b1543dd8;hpb=86a8ff1a0f02829b07bf7cd2fdd198ce612b37d4

diff --git a/infra/dgit-mirror-rsync b/infra/dgit-mirror-rsync
index 2858047b..2d912c8a 100755
--- a/infra/dgit-mirror-rsync
+++ b/infra/dgit-mirror-rsync
@@ -26,7 +26,7 @@ set -o pipefail
 shopt -s nullglob
 
 case "$DGIT_DRS_DEBUG" in
-''|0!1)		;;
+''|0|1)		;;
 *)		set -x	;;
 esac
 
@@ -54,6 +54,7 @@ rsync=(rsync -rltH --safe-links --delete)
 hooktimeout=30
 rsynctimeout=900
 rsyncssh='ssh -o batchmode=yes'
+mirror_gc_cmd='git gc --auto'
 
 . $distrodir/mirror-settings
 
@@ -74,6 +75,12 @@ case "$remoterepos" in
 esac
 
 actually () {
+	if [ "x$mirror_gc_cmd" != x ]; then
+		(
+			cd "$repos/$package.git"
+			$mirror_gc_cmd
+		)
+	fi
 	"${rsync[@]}" \
 		--timeout=$rsynctimeout				\
 		-e "$rsyncssh"					\
@@ -91,7 +98,7 @@ reinvoke () {
 
 check-package-mirrorable () {
 	local repo=$repos/$package.git
-	local mode=$(stat -c%a "$repo")
+	local mode; mode=$(stat -c%a "$repo")
 	case $mode in
 	*5)	return	0	;;
 	*0)	return	1	;;
@@ -105,13 +112,17 @@ lock-and-process () {
 }
 
 attempt () {
-	exec >"$queue/$package.err" 2>&1
+	exec 3>&2 >"$queue/$package.err" 2>&1
 	if actually; then
 		rm -f "$queue/$package.a"
+		exec 2>&3 2>&1
 		mv -f "$queue/$package.err" "$queue/$package.log"
+		if ! [ -s "$queue/$package.log" ]; then
+			rm "$queue/$package.log"
+		fi
 		rm "$queue/$package.lock"
 	else
-		cat >&2 "$queue/$package.err"
+		cat >&3 "$queue/$package.err"
 		exit 127
 	fi
 }
@@ -146,13 +157,13 @@ reinvoke-locked)
 
 backlog)
 	for f in $queue/*.[na]; do
-		lock-and-process-baseof-f
+		(lock-and-process-baseof-f ||:)
 	done
 	;;
 
 all)
 	for f in [a-z0-9]*.git; do
-		lock-and-process-baseof-f
+		(lock-and-process-baseof-f)
 	done
 	;;