Re: [PATCH] t/known-leaky: add list of known-leaky test scripts

2018-02-26 Thread Martin Ågren
On 25 February 2018 at 04:48, Kaartic Sivaraam wrote: > On Wednesday 21 February 2018 02:14 AM, Martin Ågren wrote: >> To sum up: I probably won't be looking into Travis-ing such a blacklist >> in the near future. >> > > Just thinking out loud, how about having a

Re: [PATCH] t/known-leaky: add list of known-leaky test scripts

2018-02-24 Thread Kaartic Sivaraam
On Wednesday 21 February 2018 02:14 AM, Martin Ågren wrote: > To sum up: I probably won't be looking into Travis-ing such a blacklist > in the near future. > Just thinking out loud, how about having a white-list instead of a black-list and using it to run only those tests in the white list.

Re: [PATCH] t/known-leaky: add list of known-leaky test scripts

2018-02-21 Thread Jeff King
On Wed, Feb 21, 2018 at 08:53:16AM -0800, Junio C Hamano wrote: > Even though keeping track of list of known-leaky tests may not be so > interesting, we can still salvage useful pieces from the discussion > and make them available to developers, e.g. something like > > prove --dry

Re: [PATCH] t/known-leaky: add list of known-leaky test scripts

2018-02-21 Thread Junio C Hamano
Martin Ågren writes: > On 19 February 2018 at 22:29, Jeff King wrote: > ... >> Or alternatively, we could just not bother with checking this into the >> repository, and it becomes a local thing for people interested in >> leak-testing. What's the value in

Re: [PATCH] t/known-leaky: add list of known-leaky test scripts

2018-02-20 Thread Jeff King
On Tue, Feb 20, 2018 at 09:44:51PM +0100, Martin Ågren wrote: > For finding this sort of low-hanging fruit (windfall?) any local > scripting and greping will do. I sort of hesitate wasting community CPU > cycles on something which might be appreciated, in theory, but that > doesn't have enough

Re: [PATCH] t/known-leaky: add list of known-leaky test scripts

2018-02-20 Thread Martin Ågren
On 19 February 2018 at 22:29, Jeff King wrote: > On Wed, Feb 14, 2018 at 10:56:37PM +0100, Martin Ågren wrote: > >> Here's what a list of known leaks might look like. It feels a bit >> awkward to post a known-incomplete list (I don't run all tests). Duy >> offered to pick up the

Re: [PATCH] t/known-leaky: add list of known-leaky test scripts

2018-02-19 Thread Jeff King
On Wed, Feb 14, 2018 at 10:56:37PM +0100, Martin Ågren wrote: > Here's what a list of known leaks might look like. It feels a bit > awkward to post a known-incomplete list (I don't run all tests). Duy > offered to pick up the ball if I gave up, maybe you could complete and > post this as your

[PATCH] t/known-leaky: add list of known-leaky test scripts

2018-02-14 Thread Martin Ågren
Here's what a list of known leaks might look like. It feels a bit awkward to post a known-incomplete list (I don't run all tests). Duy offered to pick up the ball if I gave up, maybe you could complete and post this as your own? :-? Even if I (or others) can't reproduce the complete list locally,