repack: use only pack-objects, not rev-list.
authorJunio C Hamano <junkio@cox.net>
Mon, 18 Sep 2006 09:29:01 +0000 (02:29 -0700)
committerJunio C Hamano <junkio@cox.net>
Wed, 20 Sep 2006 18:15:32 +0000 (11:15 -0700)
Signed-off-by: Junio C Hamano <junkio@cox.net>
git-repack.sh

index b525fc5dfd9053a6fb63a4d38517e25e75341199..9ae5092208aab167f9164dd729297d92fa4948c6 100755 (executable)
@@ -32,12 +32,10 @@ trap 'rm -f "$PACKTMP"-*' 0 1 2 3 15
 # There will be more repacking strategies to come...
 case ",$all_into_one," in
 ,,)
-       rev_list='--unpacked'
-       pack_objects='--incremental'
+       args='--unpacked --incremental'
        ;;
 ,t,)
-       rev_list=
-       pack_objects=
+       args=
 
        # Redundancy check in all-into-one case is trivial.
        existing=`test -d "$PACKDIR" && cd "$PACKDIR" && \
@@ -45,11 +43,8 @@ case ",$all_into_one," in
        ;;
 esac
 
-pack_objects="$pack_objects $local $quiet $no_reuse_delta$extra"
-name=$( { git-rev-list --objects --all $rev_list ||
-         echo "git-rev-list died with exit code $?"
-       } |
-       git-pack-objects --non-empty $pack_objects "$PACKTMP") ||
+args="$args $local $quiet $no_reuse_delta$extra"
+name=$(git-pack-objects --non-empty --all $args </dev/null "$PACKTMP") ||
        exit 1
 if [ -z "$name" ]; then
        echo Nothing new to pack.