On Thu, Apr 07, 2016 at 12:56:26PM -0700, Junio C Hamano wrote: > Jeff King <peff@xxxxxxxx> writes: > > > Yes, FWIW, those were the sites and reasons I identified last night. > > Your patch looks like the right thing to me. > > Thanks, let's do this then. I'd already anticipated your sign-off ;-). > > -- >8 -- > From: Jeff King <peff@xxxxxxxx> > Subject: commit: do not ignore an empty message given by -m '' > [...] I don't know if I deserve authorship, but I am happy to take the blame. :) Thank you for picking this up. I'm still traveling through the weekend, so I'll be a bit slow on turnaround in general. > builtin/commit.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) I think this should pass Adam's tests, along with some new ones if we cared to write them, but I'm not sure if it's worth testing such specific items (i.e., it is not likely to break in exactly the same way again). I will leave it to you whether you think it's worth squashing them in. -Peff -- 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