[PATCH 4/6] t: local VAR="VAL" (quote command substitution)
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Apr 6, 2024, 00:09 UTC
- Message-ID
- <20240406000902.3082301-5-gitster@pobox.com>
- In-Reply-To
- <20240406000902.3082301-1-gitster@pobox.com>
Future-proof test scripts that do
local VAR=VAL
without quoting VAL (which is OK in POSIX but broken in some shells) that is a $(command substitution).
Signed-off-by: Junio C Hamano <gitster@pobox.com> --- t/t4011-diff-symlink.sh | 4 ++-- t/test-lib-functions.sh | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/t/t4011-diff-symlink.sh b/t/t4011-diff-symlink.sh index d7a5f7ae78..bc8ba88719 100755 --- a/t/t4011-diff-symlink.sh +++ b/t/t4011-diff-symlink.sh @@ -13,13 +13,13 @@ TEST_PASSES_SANITIZE_LEAK=true # Print the short OID of a symlink with the given name. symlink_oid () { - local oid=$(printf "%s" "$1" | git hash-object --stdin) && + local oid="$(printf "%s" "$1" | git hash-object --stdin)" && git rev-parse --short "$oid" } # Print the short OID of the given file. short_oid () { - local oid=$(git hash-object "$1") && + local oid="$(git hash-object "$1")" && git rev-parse --short "$oid" } diff --git a/t/test-lib-functions.sh b/t/test-lib-functions.sh index 3204afbafb..3dc638f7dc 100644 --- a/t/test-lib-functions.sh +++ b/t/test-lib-functions.sh @@ -1764,7 +1764,7 @@ test_subcommand () { shift fi - local expr=$(printf '"%s",' "$@") + local expr="$(printf '"%s",' "$@")" expr="${expr%,}" if test -n "$negate"
-- 2.44.0-501-g19981daefd