Re: [PATCH] cherry-pick -x: add newline before pick note

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Nov 16, 2010 at 2:30 PM, Jeff King <peff@xxxxxxxx> wrote:
> shouldn't it result in:
>
> Âmessage subject
>
> ÂMessage body.
>
> Â(cherry picked from commit ...)
>
> ÂSigned-off-by: Jeff King <peff@xxxxxxxx>

+1.

> Even better, I wonder if it should actually be:
>
> Âmessage subject
>
> ÂMessage body.
>
> ÂSigned-off-by: Jeff King <peff@xxxxxxxx>
> ÂCherry-picked-from: ...

+2.

> 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.

I can fix my regex easily enough, but I'd also be happy to have this
use some other switch than -x.

BTW, I notice that cherry-pick also misbehaves if the original commit
message doesn't end in a newline. I'm not sure whether that's a
cherry-pick bug for not checking that case, or whether it's a commit
bug for not ensuring a newline terminates the commit message.

j.
--
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


[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]