On Tue, Nov 16, 2010 at 04:11:17PM +0100, Michael J Gruber wrote: > Currently, cherry-pick -x sticks the pick note immediately after the > existing commit message. This > > * is bad for commits with 1 line subject (it makes a 2 line subject) > * is different from git-svn, e.g., which leaves an empty line before. > > Make cherry-pick always insert an empty line before the pick note. Hmm. Should this respect pseudo-header blocks at the end? E.g., if I have: message subject Message body. Signed-off-by: Jeff King <peff@xxxxxxxx> shouldn't it result in: message subject Message body. (cherry picked from commit ...) Signed-off-by: Jeff King <peff@xxxxxxxx> ? Even better, I wonder if it should actually be: message subject Message body. Signed-off-by: Jeff King <peff@xxxxxxxx> Cherry-picked-from: ... And then you could actually sign off the cherry-pick separately, too, if you wanted, by adding a line _below_ the cherry-picked-from. I have no idea if people are trying to grep for "cherry picked from commit...", which my proposal would break. Note that none of this is introduced by your patch. The current output for this case is terribly ugly. But I thought I would mention it, as my third version means we _do_ want the current behavior in some cases (i.e., when there is already a pseudo-header block). -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