threads / patch / 14216

patchFix t4017-diff-retval for white-space from wc

Subject: [PATCH] Fix t4017-diff-retval for white-space from wc

## tl;dr

2 messages between Jun 30, 2008 and Jun 30, 2008. Diffs are folded; open one to read it.

replies: 1people: 1as markdown or json

Brian Gernhardt· Jun 30, 2008, 01:26 UTC · lore
Signed-off-by: Brian Gernhardt <benji@silverinsanity.com>
---
 t/t4017-diff-retval.sh |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
Show changes to t/t4017-diff-retval.sh +1 −1
diff --git a/t/t4017-diff-retval.sh b/t/t4017-diff-retval.sh
index d748d45..60dd201 100755
--- a/t/t4017-diff-retval.sh
+++ b/t/t4017-diff-retval.sh
@@ -123,7 +123,7 @@ test_expect_success 'check detects leftover conflict markers' '
 		git --no-pager diff --cached --check >test.out
 		test $? = 2
 	) &&
-	test "$(grep "conflict marker" test.out | wc -l)" = 3 &&
+	test 3 = $(grep "conflict marker" test.out | wc -l) &&
 	git reset --hard
 '
 
-- 
1.5.6.105.g6f4b
Brian Gernhardt· Jun 30, 2008, 01:29 UTC · re: Brian Gernhardt · lore

Re: [PATCH] Fix t4017-diff-retval for white-space from wc

Sorry about duplicate. Forgot to remove sendemail.to when testing git- send-email. (Finally tracked down my problems to a bug in Term::ReadLine::Perl when the prompt was longer than the terminal width.)

~~ Brian
On Jun 29, 2008, at 9:26 PM, Brian Gernhardt wrote:
Show 26 quoted lines
> Signed-off-by: Brian Gernhardt <benji@silverinsanity.com>
> ---
> t/t4017-diff-retval.sh |    2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/t/t4017-diff-retval.sh b/t/t4017-diff-retval.sh
> index d748d45..60dd201 100755
> --- a/t/t4017-diff-retval.sh
> +++ b/t/t4017-diff-retval.sh
> @@ -123,7 +123,7 @@ test_expect_success 'check detects leftover  
> conflict markers' '
> 		git --no-pager diff --cached --check >test.out
> 		test $? = 2
> 	) &&
> -	test "$(grep "conflict marker" test.out | wc -l)" = 3 &&
> +	test 3 = $(grep "conflict marker" test.out | wc -l) &&
> 	git reset --hard
> '
>
> -- 
> 1.5.6.105.g6f4b
>
> --
> To unsubscribe from this list: send the line "unsubscribe git" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

← back to recent threads