Re: t4015 broken on pu

2013-06-03 Thread Antoine Pelisse
On Mon, Jun 3, 2013 at 9:01 PM, Torsten Bögershausen tbo...@web.de wrote:
 The new 4015 does not pass under Mac OS:

 a) The seq command, does not exits (we can use printf)

I was planning to reroll with test_seq instead of seq, as advised by
Johannes Sixt.

 b) One sed expression is not understood, sed exporst needed ;-)

Thanks, I will fix sed also in next reroll !
--
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: t4015 broken on pu

2013-06-03 Thread Lukas Fleischer
On Mon, Jun 03, 2013 at 09:01:45PM +0200, Torsten Bögershausen wrote:
 The new 4015 does not pass under Mac OS:
 
 a) The seq command, does not exits (we can use printf)
 [...]
 diff --git a/t/t4015-diff-whitespace.sh b/t/t4015-diff-whitespace.sh
 index b3c4fcc..f00d20b 100755
 --- a/t/t4015-diff-whitespace.sh
 +++ b/t/t4015-diff-whitespace.sh
 @@ -143,18 +143,18 @@ git diff --ignore-space-at-eol  out
  test_expect_success 'another test, with --ignore-space-at-eol' 'test_cmp 
 expect out'
  
  test_expect_success 'ignore-blank-lines: only new lines' '
 -   seq 5 x 
 +   printf 1\n2\n3\n4\n5\n x 

I'd rather use test_seq here.

 git update-index x 
 -   seq 5 | sed /3/i  x 
 +   printf 1\n2\n3\n4\n5\n | sed /3/i  x 
 git diff --ignore-blank-lines out 
 [...]
--
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html