diff options
| author | Jeff King <peff@peff.net> | 2017-05-08 22:59:46 -0400 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2017-05-09 12:09:03 +0900 |
| commit | da5a1f81008f89f4e533006df8fa536e43c9c677 (patch) | |
| tree | ab1138031e1b337ef5e428c1fc8b766d6a700787 | |
| parent | 9df4a6074a98740b70b980d43dca5203db0ff688 (diff) | |
| download | git-jk/disable-pack-reuse-when-broken.tar.gz | |
t5310: fix "; do" stylejk/disable-pack-reuse-when-broken
Our usual shell style is to put the "do" of a loop on its
own line, like:
while $cond
do
something
done
instead of:
while $cond; do
something
done
We have a bit of both in our code base, but the former is
what's in CodingGuidelines (and outnumbers the latter in t/
by about 6:1).
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
| -rwxr-xr-x | t/t5310-pack-bitmaps.sh | 12 |
1 files changed, 8 insertions, 4 deletions
diff --git a/t/t5310-pack-bitmaps.sh b/t/t5310-pack-bitmaps.sh index c3ddfa217f..20e2473a03 100755 --- a/t/t5310-pack-bitmaps.sh +++ b/t/t5310-pack-bitmaps.sh @@ -20,11 +20,13 @@ has_any () { } test_expect_success 'setup repo with moderate-sized history' ' - for i in $(test_seq 1 10); do + for i in $(test_seq 1 10) + do test_commit $i done && git checkout -b other HEAD~5 && - for i in $(test_seq 1 10); do + for i in $(test_seq 1 10) + do test_commit side-$i done && git checkout master && @@ -104,7 +106,8 @@ test_expect_success 'clone from bitmapped repository' ' ' test_expect_success 'setup further non-bitmapped commits' ' - for i in $(test_seq 1 10); do + for i in $(test_seq 1 10) + do test_commit further-$i done ' @@ -300,7 +303,8 @@ test_expect_success 'set up reusable pack' ' test_expect_success 'pack reuse respects --honor-pack-keep' ' test_when_finished "rm -f .git/objects/pack/*.keep" && - for i in .git/objects/pack/*.pack; do + for i in .git/objects/pack/*.pack + do >${i%.pack}.keep done && reusable_pack --honor-pack-keep >empty.pack && |