From: René Scharfe Date: Thu, 10 Jan 2013 21:24:52 GMT Subject: Re: [PATCH] t0008: avoid brace expansion Message-ID: <50EF31A4.3000205@lsrfire.ath.cx> In-Reply-To: <7vsj693n6o.fsf@alter.siamese.dyndns.org> Am 10.01.2013 01:18, schrieb Junio C Hamano: > Adam Spiers writes: > >> On Wed, Jan 9, 2013 at 11:49 PM, René Scharfe >> wrote: >>> Brace expansion is not required by POSIX and not supported by dash nor >>> NetBSD's sh. Explicitly list all combinations instead. >> >> Good catch, thanks! > > Yeah; thanks. > > It would also be nice to avoid touch while we are at it, by the way. Good idea! Replacement patch: --- >8 --- Brace expansion is a shell feature that's not required by POSIX and not supported by dash nor NetBSD's sh. Explicitly list all combinations instead. Also avoid calling touch by creating the test files with a redirection instead, as suggested by Junio. Signed-off-by: Rene Scharfe --- t/t0008-ignores.sh | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/t/t0008-ignores.sh b/t/t0008-ignores.sh index 9b0fcd6..d7df719 100755 --- a/t/t0008-ignores.sh +++ b/t/t0008-ignores.sh @@ -129,8 +129,13 @@ test_expect_success 'setup' ' one ignored-* EOF - touch {,a/}{not-ignored,ignored-{and-untracked,but-in-index}} && - git add -f {,a/}ignored-but-in-index + for dir in . a + do + : >$dir/not-ignored && + : >$dir/ignored-and-untracked && + : >$dir/ignored-but-in-index + done && + git add -f ignored-but-in-index a/ignored-but-in-index && cat <<-\EOF >a/.gitignore && two* *three -- 1.8.0