Re: [PATCH v3 1/3] Add valgrind support in test scripts

From: Johannes Schindelin <johannes.schindelin@gmx.de>
Date: Jan 25, 2009, 23:35 UTC
Message-ID: <alpine.DEB.1.00.0901260034520.14855@racer>
In-Reply-To: <20090125232954.GC19099@coredump.intra.peff.net>

Hi,

On Sun, 25 Jan 2009, Jeff King wrote:

Show 7 quoted lines

> On Mon, Jan 26, 2009 at 12:18:50AM +0100, Johannes Schindelin wrote:
> 
> > Note: it is safe to run the valgrind tests in parallel, as the links 
> > in t/valgrind/bin/ are created using proper locking.
> 
> I actually kind of liked the original atomic version over the one with
> locking. But I find this one acceptable.

The locking is only in there because of you...

Show 8 quoted lines

> > Initial patch and all the hard work by Jeff King.
> > 
> > Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
> 
> I don't know that there is much of my work left in here, but feel free 
> to add:
> 
>   Signed-off-by: Jeff King <peff@peff.net>

Will do!

Ciao, Dscho