Using "From" to start commit message

Jeremy Kerr jk at ozlabs.org
Wed Mar 19 10:33:00 EST 2014


Hi Thierry,

> I've noticed something strange happening when a commit message starts
> with the word "From". When downloading the commit as mbox, I see that
> the line is "escaped" using a '>' character, which causes git am to
> ignore the line.
> 
> This can be seen in the following patch for example:
> 
> 	http://patchwork.ozlabs.org/patch/330330/
> 
> I'm not familiar with the internals of patchwork, but I would expect
> this to be perfectly legal in a commit message. I could imagine that
> to be special treatment for From lines in mailboxes, but that should
> not apply to the text in commit messages.

Hm, interesting. This escaping is generally required for the file to be
a valid mbox; the 'From<space>' sequence defines the beginning of a new
message (hence the escaping with '>'), but it looks like this isn't what
git am expects.

We could generate the message without the escaping (and drop the initial
From_ line too), but that may break other uses of the mailbox files. I'm
not sure how many other consumers there are though...

Carl - I see you've done some work on git am's parser for From_ syntax;
do you know what's happening here?

Regards,


Jeremy


More information about the Patchwork mailing list