Re: [PATCH] Add tests for diff.color-words configuration option.

From: Boyd Stephen Smith Jr. <bss@iguanasuicide.net>
Date: Jan 20, 2009, 16:34 UTC
Message-ID: <200901201034.22478.bss@iguanasuicide.net>
In-Reply-To: <alpine.DEB.1.00.0901201057080.3586@pacific.mpi-cbg.de>

On Tuesday 2009 January 20 03:58:37 Johannes Schindelin wrote:

Show 5 quoted lines

>On Mon, 19 Jan 2009, Boyd Stephen Smith Jr. wrote:
>> I'm not sure why the diff is crazy long.
>
>Because you changed things that need no changing, such as "cat > expect"
>-> "cat > expect.blabla",

I suppose I could have gotten away with doing this differently, but I did need to save off some of those results to different files because I wanted to resuse the results.

>and because you inserted your test instead of 
>adding it at the end.

I put the tests in that order explicitly to test that .gitattributes overrides the configuration option.

I'm going to be reworking both patches anyway, so I should be able to rearrange things less, in this file.

Thanks for the feedback.

-- 
Boyd Stephen Smith Jr.                     ,= ,-_-. =. 
bss@iguanasuicide.net                     ((_/)o o(\_))
ICQ: 514984 YM/AIM: DaTwinkDaddy           `-'(. .)`-' 
http://iguanasuicide.net/                      \_/