[PATCH] cherry-pick -x: improve handling of one-liner commit messages

2013-03-29 Thread Miklos Vajna
git cherry-pick -x normally just appends the cherry picked from commit line at the end of the message, which is fine. However, in case the original commit message had only one line, first append a newline, otherwise the second line won't be empty, which is against recommendations. --- sequencer.c

Re: [PATCH] cherry-pick -x: improve handling of one-liner commit messages

2013-03-29 Thread Junio C Hamano
Miklos Vajna vmik...@suse.cz writes: git cherry-pick -x normally just appends the cherry picked from commit line at the end of the message, which is fine. However, in case the original commit message had only one line, first append a newline, otherwise the second line won't be empty, which is

Re: [PATCH] cherry-pick -x: improve handling of one-liner commit messages

2013-03-29 Thread Miklos Vajna
Hi, On Fri, Mar 29, 2013 at 10:41:17AM -0700, Brandon Casey draf...@gmail.com wrote: Sign-off? Indeed, I forgot about it, my bad. I think this is part of the bc/append-signed-off-by topic that is about to graduate to 'master'; more specifically, b971e04f54e7 (sequencer.c: always

Re: [PATCH] cherry-pick -x: improve handling of one-liner commit messages

2013-03-29 Thread Junio C Hamano
Miklos Vajna vmik...@suse.cz writes: On Fri, Mar 29, 2013 at 10:41:17AM -0700, Brandon Casey draf...@gmail.com wrote: I think this is part of the bc/append-signed-off-by topic that is about to graduate to 'master'; more specifically, b971e04f54e7 (sequencer.c: always separate (cherry