Vicent Marti <tan...@gmail.com> writes:

> @@ -156,6 +156,11 @@ do
>       fullbases="$fullbases pack-$name"
>       chmod a-w "$PACKTMP-$name.pack"
>       chmod a-w "$PACKTMP-$name.idx"
> +
> +     test -f "$PACKTMP-$name.bitmap" &&
> +     chmod a-w "$PACKTMP-$name.bitmap" &&
> +     mv -f "$PACKTMP-$name.bitmap" "$PACKDIR/pack-$name.bitmap"

If we see a temporary bitmap but somehow failed to move it to the
final name, should we _ignore_ that error, or should we die, like
the next two lines do?

>       mv -f "$PACKTMP-$name.pack" "$PACKDIR/pack-$name.pack" &&
>       mv -f "$PACKTMP-$name.idx"  "$PACKDIR/pack-$name.idx" ||
>       exit
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to