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.
- From
Junio C Hamano <gitster@pobox.com>
- Date
- May 4, 2008, 00:08 UTC
- 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.
Show 14 quoted lines
> 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?