Armin Kunaschik <[email protected]> writes:
> From: Armin Kunaschik <[email protected]>
>
> \" in the test t0008 is not treated the same way in bash and in ksh.
Could you refrain from singling out "bash"? We don't write for
"bash" specifically (and the test I ran are with "dash" before I
push things out).
Ideally, if you can try ksh93 and if you find out that ksh93 works,
then the above can be made in line with your "Subject" to mark ksh88
as broken (as opposed to other POSIX shells)? That would help us by
reminding that running test fine with ksh93 is not a sufficient
check to make sure we didn't break ksh88 users.
> In ksh the \ disappears and generates false expect data to
> compare with.
> Using \\" works portable, the same way in bash and in ksh and
> is less ambigous.
All of the above would need s/ksh/&88/g; I'd think. I just tried
make SHELL_PATH=/bin/ksh93
cd t && /bin/ksh93 t0008-*.sh
and this patch is not necessary for ksh93.
> Acked-by: Jeff King <[email protected]>
I didn't see him acking this exact version, so if you didn't include
this line here, I would have missed it. Thanks.
> Signed-off-by: Armin Kunaschik <[email protected]>
> ---
> diff --git a/t/t0008-ignores.sh b/t/t0008-ignores.sh
> index 89544dd..b425f3a 100755
> --- a/t/t0008-ignores.sh
> +++ b/t/t0008-ignores.sh
> @@ -605,7 +605,7 @@ cat <<-EOF >expected-verbose
> a/b/.gitignore:8:!on* a/b/one
> a/b/.gitignore:8:!on* a/b/one one
> a/b/.gitignore:8:!on* a/b/one two
> - a/b/.gitignore:8:!on* "a/b/one\"three"
> + a/b/.gitignore:8:!on* "a/b/one\\"three"
> a/b/.gitignore:9:!two a/b/two
> a/.gitignore:1:two* a/b/twooo
> $global_excludes:2:!globaltwo globaltwo
> @@ -686,7 +686,7 @@ cat <<-EOF >expected-all
> a/b/.gitignore:8:!on* b/one
> a/b/.gitignore:8:!on* b/one one
> a/b/.gitignore:8:!on* b/one two
> - a/b/.gitignore:8:!on* "b/one\"three"
> + a/b/.gitignore:8:!on* "b/one\\"three"
> a/b/.gitignore:9:!two b/two
> :: b/not-ignored
> a/.gitignore:1:two* b/twooo
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
