threads / discuss / 50437

[Fix v1] t5403: correct bash ambiguous redirect error in subtest 8 by quoting $GIT_DIR

Subject: [Fix v1] t5403: correct bash ambiguous redirect error in subtest 8 by quoting $GIT_DIR

## tl;dr

2 messages between Feb 8, 2019 and Feb 8, 2019.

replies: 1people: 2as markdown or json

randall.s.becker@rogers.com· Feb 8, 2019, 11:32 UTC · lore
From: "Randall S. Becker" <rsbecker@nexbridge.com>

The embedded blanks in the full path of the test git repository cased bash to generate an ambugious redirect error.

Signed-off-by: Randall S. Becker <rsbecker@nexbridge.com>
---
 t/t5403-post-checkout-hook.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/t/t5403-post-checkout-hook.sh b/t/t5403-post-checkout-hook.sh
index a539ffc08..a39b3b5c7 100755
--- a/t/t5403-post-checkout-hook.sh
+++ b/t/t5403-post-checkout-hook.sh
@@ -67,7 +67,7 @@ test_expect_success 'post-checkout is triggered on rebase with fast-forward' '
 test_expect_success 'post-checkout hook is triggered by clone' '
 	mkdir -p templates/hooks &&
 	write_script templates/hooks/post-checkout <<-\EOF &&
-	echo "$@" >$GIT_DIR/post-checkout.args
+	echo "$@" >"$GIT_DIR/post-checkout.args"
 	EOF
 	git clone --template=templates . clone3 &&
 	test -f clone3/.git/post-checkout.args
-- 
2.12.3
Junio C Hamano· Feb 8, 2019, 19:55 UTC · re: randall.s.becker@rogers.com · lore

Re: [Fix v1] t5403: correct bash ambiguous redirect error in subtest 8 by quoting $GIT_DIR

randall.s.becker@rogers.com writes:
Show 7 quoted lines
> From: "Randall S. Becker" <rsbecker@nexbridge.com>
>
> The embedded blanks in the full path of the test git repository cased bash
> to generate an ambugious redirect error.
>
> Signed-off-by: Randall S. Becker <rsbecker@nexbridge.com>
> ---
Thanks.
Show 16 quoted lines
>  t/t5403-post-checkout-hook.sh | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/t/t5403-post-checkout-hook.sh b/t/t5403-post-checkout-hook.sh
> index a539ffc08..a39b3b5c7 100755
> --- a/t/t5403-post-checkout-hook.sh
> +++ b/t/t5403-post-checkout-hook.sh
> @@ -67,7 +67,7 @@ test_expect_success 'post-checkout is triggered on rebase with fast-forward' '
>  test_expect_success 'post-checkout hook is triggered by clone' '
>  	mkdir -p templates/hooks &&
>  	write_script templates/hooks/post-checkout <<-\EOF &&
> -	echo "$@" >$GIT_DIR/post-checkout.args
> +	echo "$@" >"$GIT_DIR/post-checkout.args"
>  	EOF
>  	git clone --template=templates . clone3 &&
>  	test -f clone3/.git/post-checkout.args

← back to recent threads