summaryrefslogtreecommitdiff
path: root/t/perf/p5302-pack-index.sh
diff options
context:
space:
mode:
authorElia Pinto <gitter.spiros@gmail.com>2014-04-23 13:44:06 (GMT)
committerJunio C Hamano <gitster@pobox.com>2014-04-23 22:17:03 (GMT)
commit4717659144984c7fbfd2c01a2c22f342756e2abd (patch)
treecd6be81bfd2da044816a052eedc4ac55888906f2 /t/perf/p5302-pack-index.sh
parentbe194d53c0ce38ab3e040c6be249550fab4e8e1c (diff)
downloadgit-4717659144984c7fbfd2c01a2c22f342756e2abd.zip
git-4717659144984c7fbfd2c01a2c22f342756e2abd.tar.gz
git-4717659144984c7fbfd2c01a2c22f342756e2abd.tar.bz2
p5302-pack-index.sh: use the $( ... ) construct for command substitution
The Git CodingGuidelines prefer the $(...) construct for command substitution instead of using the backquotes `...`. The backquoted form is the traditional method for command substitution, and is supported by POSIX. However, all but the simplest uses become complicated quickly. In particular, embedded command substitutions and/or the use of double quotes require careful escaping with the backslash character. The patch was generated by: for _f in $(find . -name "*.sh") do sed -i 's@`\(.*\)`@$(\1)@g' ${_f} done and then carefully proof-read. Signed-off-by: Elia Pinto <gitter.spiros@gmail.com> Reviewed-by: Matthieu Moy <Matthieu.Moy@imag.fr> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't/perf/p5302-pack-index.sh')
-rwxr-xr-xt/perf/p5302-pack-index.sh2
1 files changed, 1 insertions, 1 deletions
diff --git a/t/perf/p5302-pack-index.sh b/t/perf/p5302-pack-index.sh
index 6cb5b0d..5ee9211 100755
--- a/t/perf/p5302-pack-index.sh
+++ b/t/perf/p5302-pack-index.sh
@@ -8,7 +8,7 @@ test_perf_large_repo
test_expect_success 'repack' '
git repack -ad &&
- PACK=`ls .git/objects/pack/*.pack | head -n1` &&
+ PACK=$(ls .git/objects/pack/*.pack | head -n1) &&
test -f "$PACK" &&
export PACK
'