git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [RFC/PATCH] t0081-*.sh: Fix failure of the 'long read' tests

From
Ramsay Jones <ramsay@ramsay1.demon.co.uk>
Date
Apr 30, 2011, 17:12 UTC
Message-ID
<4DBC4310.4000105@ramsay1.demon.co.uk>
In-Reply-To
<20110426193539.GA2616@elie>
Jonathan Nieder wrote:
Show 18 quoted lines
> Ramsay Jones wrote:
> 
>> --- a/t/t0081-line-buffer.sh
>> +++ b/t/t0081-line-buffer.sh
>> @@ -25,8 +25,7 @@ generate_tens_of_lines () {
>>  		do
>>  			echo "$line"
>>  		done &&
>> -		: $((i = $i + 1)) ||
>> -		return
>> +		i=$(($i + 1))
>>  	done
> 
> This test is a mess.  Could you try the patch from the tip of
> 
> 	git://repo.or.cz/git/jrn.git svn-fe
> 
> which just gets rid of it instead?
Ah, so I didn't imagine the discussion to remove this test!

Yes, this works great for me. So, I will keep skipping the test until this patch makes it into git.git.

Thanks!

ATB, Ramsay Jones

Previous: Jonathan NiederNext: Jeff King
Message 3 of 7 in “t0081-*.sh: Fix failure of the 'long read' tests”
  1. t0081-*.sh: Fix failure of the 'long read' testsRamsay Jones, Apr 26, 2011
  2. Jonathan NiederApr 26, 2011
  3. Ramsay JonesApr 30, 2011
  4. Jeff KingApr 26, 2011
  5. Ramsay JonesApr 30, 2011
  6. [PULL svn-fe/maint] t0081-*.sh: Fix failure of the 'long read' testsJonathan Nieder, May 26, 2011
  7. Junio C HamanoMay 26, 2011

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.