Ok for me, thanks for helping out.


> Am 10.05.2017 um 17:52 schrieb Johannes Schindelin 
> <johannes.schinde...@gmx.de>:
> 
> Hi,
> 
>> On Wed, 10 May 2017, tbo...@web.de wrote:
>> 
>> From: Torsten Bögershausen <tbo...@web.de>
>> 
>> The purpose of t0027 is to test all CRLF related conversions at "git 
>> checkout"
>> and "git add".
>> 
>> Running t0027 under Git for Windows takes 3-4 minutes, so the whole script 
>> had
>> been marked as "EXPENSIVE".
>> 
>> The source code for "Git for Windows" overrides this since 2014:
>> "t0027 is marked expensive, but really, for MinGW we want to run these
>> tests always."
>> 
>> Recent "stress" tests show that t0025 if flaky, reported by Lars Schneider,
>> larsxschnei...@gmail.com
>> 
>> All tests in t0025 are covered by t0027 already, so that t0025 can be 
>> retired.
>> t0027 takes less than 14 seconds under Linux, and 63 seconds under Mac Os X,
>> and this is more or less the same with a SSD or a spinning disk.
>> 
>> Acked-by: Johannes Schindelin <johannes.schinde...@gmx.de>
>> Signed-off-by: Torsten Bögershausen <tbo...@web.de>
> 
> This is still formatted very awkwardly. How about this instead (I fixed
> the formatting, reworded a little here and there, and fixed the order of
> the footers)?
> 
> -- snipsnap --
> From: Torsten Bögershausen <tbo...@web.de>
> 
> The purpose of t0027 is to test all CRLF related conversions at "git
> checkout" and "git add".  Running t0027 under Git for Windows takes 3-4
> minutes, so the whole script had been marked as "EXPENSIVE".
> 
> However, the "Git for Windows" fork overrides this since 2014: "t0027
> is marked expensive, but really, for MinGW we want to run these tests
> always."
> 
> The test seems not to be expensive on other platforms at all: it takes
> less than 14 seconds under Linux, and 63 seconds under Mac Os X, and
> this is more or less the same with a SSD or a spinning disk.
> 
> So let's drop the "EXPENSIVE" prereq.
> 
> While at it, retire t0025: Recent "stress" tests show that t0025 if
> flaky, reported by Lars Schneider <larsxschnei...@gmail.com>, but all
> tests in t0025 are covered by t0027 already.
> 
> Signed-off-by: Torsten Bögershausen <tbo...@web.de>
> Acked-by: Johannes Schindelin <johannes.schinde...@gmx.de>

Reply via email to