* [PATCH] t4038: Remove non-portable '-a' option passed to test_cmp
@ 2019-09-20 22:06 CB Bailey
2019-09-23 16:52 ` Elijah Newren
0 siblings, 1 reply; 2+ messages in thread
From: CB Bailey @ 2019-09-20 22:06 UTC (permalink / raw)
To: git; +Cc: Elijah Newren
From: CB Bailey <cbailey32@bloomberg.net>
Signed-off-by: CB Bailey <cbailey32@bloomberg.net>
---
t/t4038-diff-combined.sh | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/t/t4038-diff-combined.sh b/t/t4038-diff-combined.sh
index d4afe12554..b9d876efa2 100755
--- a/t/t4038-diff-combined.sh
+++ b/t/t4038-diff-combined.sh
@@ -509,7 +509,7 @@ test_expect_success FUNNYNAMES '--combined-all-paths and --raw and funny names'
test_expect_success FUNNYNAMES '--combined-all-paths and --raw -and -z and funny names' '
printf "aaf8087c3cbd4db8e185a2d074cf27c53cfb75d7\0::100644 100644 100644 f00c965d8307308469e537302baa73048488f162 088bd5d92c2a8e0203ca8e7e4c2a5c692f6ae3f7 333b9c62519f285e1854830ade0fe1ef1d40ee1b RR\0file\twith\ttabs\0i\tam\ttabbed\0fickle\tnaming\0" >expect &&
git diff-tree -c -M --raw --combined-all-paths -z HEAD >actual &&
- test_cmp -a expect actual
+ test_cmp expect actual
'
test_expect_success FUNNYNAMES '--combined-all-paths and --cc and funny names' '
--
2.23.0.1.g68c7121409
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] t4038: Remove non-portable '-a' option passed to test_cmp
2019-09-20 22:06 [PATCH] t4038: Remove non-portable '-a' option passed to test_cmp CB Bailey
@ 2019-09-23 16:52 ` Elijah Newren
0 siblings, 0 replies; 2+ messages in thread
From: Elijah Newren @ 2019-09-23 16:52 UTC (permalink / raw)
To: CB Bailey; +Cc: Git Mailing List
On Fri, Sep 20, 2019 at 3:07 PM CB Bailey <cb@hashpling.org> wrote:
>
> From: CB Bailey <cbailey32@bloomberg.net>
>
> Signed-off-by: CB Bailey <cbailey32@bloomberg.net>
> ---
> t/t4038-diff-combined.sh | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/t/t4038-diff-combined.sh b/t/t4038-diff-combined.sh
> index d4afe12554..b9d876efa2 100755
> --- a/t/t4038-diff-combined.sh
> +++ b/t/t4038-diff-combined.sh
> @@ -509,7 +509,7 @@ test_expect_success FUNNYNAMES '--combined-all-paths and --raw and funny names'
> test_expect_success FUNNYNAMES '--combined-all-paths and --raw -and -z and funny names' '
> printf "aaf8087c3cbd4db8e185a2d074cf27c53cfb75d7\0::100644 100644 100644 f00c965d8307308469e537302baa73048488f162 088bd5d92c2a8e0203ca8e7e4c2a5c692f6ae3f7 333b9c62519f285e1854830ade0fe1ef1d40ee1b RR\0file\twith\ttabs\0i\tam\ttabbed\0fickle\tnaming\0" >expect &&
> git diff-tree -c -M --raw --combined-all-paths -z HEAD >actual &&
> - test_cmp -a expect actual
> + test_cmp expect actual
> '
This will mean slightly less useful diagnostic output should the test
ever fail on a platform that does support diff -a, but that's a small
price to pay to make sure the test is portable. If anyone does ever
see this test fail, they can go in and inspect further themselves.
Thanks for the fix.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2019-09-23 16:52 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-09-20 22:06 [PATCH] t4038: Remove non-portable '-a' option passed to test_cmp CB Bailey
2019-09-23 16:52 ` Elijah Newren
Code repositories for project(s) associated with this public inbox
https://80x24.org/mirrors/git.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).