Re: [PATCH] Handle "git show" output correctly.

2012-09-12 Thread Peter Jones
On Wed, 2012-09-12 at 17:40 +0200, Matthieu Moy wrote: > > How does this react to multi-line subject, e.g > > This should be the > subject line. > > And this is the body. > > ? > > git format-patch will merge the lines in a single Subject: header, and > your version seems to take only the firs

Re: [PATCH] Handle "git show" output correctly.

2012-09-12 Thread Matthieu Moy
Peter Jones writes: > (this version with fixed tabs) This will end up being the commit message. If you add text here, then the maintainer will have to manually fix it when applying (or reject your patch). Please, be nice with him and put your comments below the --- : > Signed-off-by: Peter Jone

Re: [PATCH] Handle "git show" output correctly.

2012-09-12 Thread Matthieu Moy
> Subject: Re: [PATCH] Handle "git show" output correctly. No final period please. This does not say which part of git is made to handle "git show". What about [PATCH] am: handle "git show" output correctly Peter Jones writes: This lacks a proper commit messa