From: Junio C Hamano <gitster@pobox.com>
Date: Sun, 04 May 2008 00:08:23 GMT
Subject: Re: [PATCH] I don't known anyone who understands what it means when they do a merge and see "file.txt: needs update". "file.txt: has changes" is much clearer.
Message-ID: <7v63tvszgo.fsf@gitster.siamese.dyndns.org>
In-Reply-To: <3DE78C03-DA35-4CB5-8D3D-0529A89065EC@gmail.com>

Tim Harper <timcharper@gmail.com> writes:

because it is very hard to follow the flow of thought.
Please do not top post.

> On May 3, 2008, at 10:57 AM, Junio C Hamano wrote:
>
>> Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:
>>
>>> On Sat, 3 May 2008, Tim Harper wrote:
>>>
>>>> -			printf("%s: needs update\n", ce->name);
>>>> +			printf("%s: has changes\n", ce->name);
>>>
>>> How about "local changes"?
>>
>> Aren't there Porcelain and end-user scripts that relies on the
>> output by
>> doing "sed -ne s'/: needs update$//p"?

> I ran all of the tests with the patch apply, and they all pass.  Is
> that enough indication?

Of course not.  Where does end-user scripts come into play when you are
running the testsuite?