diff options
| author | Elia Pinto <gitter.spiros@gmail.com> | 2014-04-16 10:29:56 -0700 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2014-04-17 11:15:00 -0700 |
| commit | b352891021d74528f668306d0882633cc0673a89 (patch) | |
| tree | 5685a4206c51ce97be4d195fd2b2863115f08d04 | |
| parent | fb6644a32fa06ff2698ecc66f4008d2afdf0b9c7 (diff) | |
| download | git-b352891021d74528f668306d0882633cc0673a89.tar.gz git-b352891021d74528f668306d0882633cc0673a89.tar.xz | |
git-tag.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>
| -rwxr-xr-x | contrib/examples/git-tag.sh | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/contrib/examples/git-tag.sh b/contrib/examples/git-tag.sh index 2c15bc955..1bd8f3c58 100755 --- a/contrib/examples/git-tag.sh +++ b/contrib/examples/git-tag.sh @@ -156,7 +156,7 @@ prev=0000000000000000000000000000000000000000 if git show-ref --verify --quiet -- "refs/tags/$name" then test -n "$force" || die "tag '$name' already exists" - prev=`git rev-parse "refs/tags/$name"` + prev=$(git rev-parse "refs/tags/$name") fi shift git check-ref-format "tags/$name" || |