Re: [PATCH 8/9] Use git diff instead of diff in t7201

From: Daniel Barkalow <barkalow@iabervon.org>
Date: Feb 5, 2008, 20:39 UTC
Message-ID: <alpine.LNX.1.00.0802042152560.13593@iabervon.org>
In-Reply-To: <alpine.LSU.1.00.0802050139470.8543@racer.site>

On Tue, 5 Feb 2008, Johannes Schindelin wrote:

Show 14 quoted lines

> Hi,
> 
> On Mon, 4 Feb 2008, Daniel Barkalow wrote:
> 
> > If the test failed, it was giving really unclear ed script
> > output. Instead, give a diff that sort of suggests the problem.
> > 
> > Signed-off-by: Daniel Barkalow <barkalow@iabervon.org>
> > ---
> >  t/t7201-co.sh |    4 ++--
> 
> This is not really a part of the series leading to builtin checkout.  But 
> as a standalone patch, I like it (obviously... I tried to push for more 
> --no-index uses in the test suite).

Well, it's something I tripped over when builtin-checkout wasn't passing the tests and I couldn't figure out what it was doing wrong from the output. Like [3/9], it's relevant to evaluating whether the series works, even if it's not important for whether it actually does work.

	-Daniel
*This .sig left intentionally blank*