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.

Tim Harper <timcharper@xxxxxxxxx> · Sat, 3 May 2008 14:10:48 -0600

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

Tim

On May 3, 2008, at 10:57 AM, Junio C Hamano wrote:

Johannes Schindelin <Johannes.Schindelin@xxxxxx> 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"?

--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html