[Xen-devel] Re: [PATCH 00 of 10] blkfront pvops updates, v2

[Jeremy Fitzhardinge <jeremy@xxxxxxxx>]

On 04/30/2010 03:01 PM, Daniel Stodden wrote:
>  * Found the switch [again] for HG to strip those headers -- cheers.
>   

The ideal for me is if you have a git tree I can pull from, but failing
that, the most useful format for mailed patches is something that "git
am" can accept.  It looks like it will take the attachments OK, but it
cats the mail body onto the attachment, so it ends up with the commit
comment twice.  If you have a non-mangling mailer, it's probably easiest
if you just include the full commit comment+patch in the mail body (it's
easier to comment on the patches in mail that way too).

> Oh beautiful. This apparently strips the first comment line together
> with the patch header. Looks buggy.
>
> F*! 
>
> Sorry. Fix/resend?
>   

Did it lose something other than the subject?  git am picked it up from
the email Subject: line, so it didn't matter it was missing from the
patch comment itself.

    J

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel