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

From: Johannes Schindelin <johannes.schindelin@gmx.de>
Date: Sep 21, 2007, 11:08 UTC
Message-ID: <Pine.LNX.4.64.0709211207070.28395@racer.site>
In-Reply-To: <20070921110646.GA9072@mageo.cz>

Hi,

On Fri, 21 Sep 2007, Michal Vitecek wrote:

Show 10 quoted lines

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

Show 9 quoted lines

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