Re: [PATCH] Added a new placeholder '%cm' for full commit message

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

 



Hi,

On Fri, 21 Sep 2007, Michal Vitecek wrote:

> Johannes Schindelin wrote:
> >On Fri, 21 Sep 2007, Michal Vitecek wrote:
> >
> >> I have added a new placeholder '%cm' for a full commit message.
> >
> >You mean the raw message, including the headers?  Why not use "%r" for 
> >that?
> 
>  No, sorry for the incorrect term. I meant the whole commit message
>  (subject + 2*\n + body).

Ah, makes sense.  In that case, "%M" maybe?

> >> I made it because I want to use my own pretty format which currently 
> >> only allows '%s' for subject and '%b' for body. But '%b' is 
> >> substituted with <undefined> if the body is "missing" which I 
> >> obviously don't like :)
> >
> >Then you should fix %b not to show "<undefined>".
> 
>  I'll do it if it is okay. Shall I do the same for the other
>  placeholders as well?

Yeah.  Don't know why I did it that way.

> >And please adher to the tips in Documentation/SubmittingPatches.
> 
>  Will do, sorry about the attachment - still learning :)

No problem.  Thanks for contributing!

Ciao,
Dscho

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

  Powered by Linux