Hi, On Sun, May 12, 2013 at 02:19:34PM -0700, Junio C Hamano wrote: > Heiko Voigt <hvoigt@xxxxxxxxxx> writes: > > > Since ages we do not care about our program enough. Lets not treat them > > as slaves anymore and say please. > > > > Signed-off-by: Heiko Voigt <hvoigt@xxxxxxxxxx> > > Signed-off-by: Jonathan Nieder <jrnieder@xxxxxxxxx> > > Signed-off-by: Jens Lehmann <Jens.Lehmann@xxxxxx> > > Signed-off-by: Thomas Rast <trast@xxxxxxxxxxx> > > Signed-off-by: Johan Herland <johan@xxxxxxxxxxx> > > So these were the ones present on the dev-day? No this was just a random sample of the ones sitting at the same beer garden table that this feature was implemented on. > I actually would have expected, from the "please" title, the > opposite, us saying "please" to the user, either once in a while in > the advice messages we give to them, or perhaps in the en_POLITE > locale ;-) The en_POLITE is also a nice idea :-) We could split up the work on the translations amongst all people joining a git meetup. > > .gitignore | 1 + > > Makefile | 1 + > > builtin.h | 1 + > > builtin/config.c | 23 ++----------------- > > builtin/please.c | 9 ++++++++ > > cache.h | 1 + > > config.c | 23 +++++++++++++++++++ > > contrib/completion/git-prompt.sh | 5 +++- > > git.c | 49 +++++++++++++++++++++++++++++++++++++++- > > There is no test to protect this feature from future breakages? In fact the whole testsuite should fail after the first 10 tests if you said please right before running it. So since battery life is short and this feature so important we did not bother to fix that. We already had enough please complaints during intermediate commits and entering the signed-off messages. It is really annoying, you should try this patch! ;-) And it is especially unuseful when you are committing using git gui. Cheers Heiko -- 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