Re: [PATCH] transport, send-pack: append period to up-to-date message

2016-05-26 Thread Yong Bakos
On May 25, 2016, at 5:55 PM, Junio C Hamano wrote: > > Jeff King writes: > >> I think messages to stderr are generally fair game for changing, even in >> plumbing. In many cases they are also translated (and I would argue that >> these messages probably should

Re: [PATCH] transport, send-pack: append period to up-to-date message

2016-05-25 Thread Junio C Hamano
Jeff King writes: > I think messages to stderr are generally fair game for changing, even in > plumbing. In many cases they are also translated (and I would argue that > these messages probably should be translated, too). I think I agree. My first reaction to this thread indeed

Re: [PATCH] transport, send-pack: append period to up-to-date message

2016-05-25 Thread Jeff King
On Tue, May 24, 2016 at 02:21:00PM -0700, Stefan Beller wrote: > On Tue, May 24, 2016 at 1:51 PM, Yong Bakos wrote: > > Appending a period to "Everything up-to-date" makes the output message > > consistent with similar output in builtin/merge.c. > > > > Signed-off-by:

Re: [PATCH] transport, send-pack: append period to up-to-date message

2016-05-24 Thread Stefan Beller
On Tue, May 24, 2016 at 1:51 PM, Yong Bakos wrote: > Appending a period to "Everything up-to-date" makes the output message > consistent with similar output in builtin/merge.c. > > Signed-off-by: Yong Bakos > --- > builtin/send-pack.c | 2 +- >

[PATCH] transport, send-pack: append period to up-to-date message

2016-05-24 Thread Yong Bakos
Appending a period to "Everything up-to-date" makes the output message consistent with similar output in builtin/merge.c. Signed-off-by: Yong Bakos --- builtin/send-pack.c | 2 +- transport.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git