lkml.org 
[lkml]   [2011]   [Oct]   [5]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [Quilt-dev] [RFC][PATCH v2][QUILT] Add gpg signing to quilt mail
From
Date
On Wed, 2011-10-05 at 10:23 +0200, Andreas Gruenbacher wrote:
> On Tue, 2011-10-04 at 23:53 -0700, Greg KH wrote:
> > Yes, quoting is the correct thing to do, but that does not usually
> > happen in "Signed-off-by:" lines in kernel patches.
> >
> > Actually, I think this is even a problem when the name is quoted
> > properly, I've had to fix that up in patches by hand.
>
> Yes, proper quoting currently isn't enough.
>
> > And yes, this is a problem, I run into at least 1-2 patches per stable
> > kernel release with this issue. I looked into fixing it and somehow the
> > email parsing logic was wierd and I couldn't figure out how to resolve
> > it.
>
> Okay, I'll have another look ...

I've tried to address this as follows now:

commit f3e7822028616014f933b99d05f42fc028f8d12c
Author: Andreas Gruenbacher <agruen@linbit.com>
Date: Wed Oct 5 12:57:48 2011 +0200

quilt mail: Allow commas in Signed-off-by and Acked-by lines

The standard quilt_mail_patch_filter() in quilt mail tries to extract email
addresses from Signed-off-by and Acked-by lines. It adds Cc headers with those
addresses.

Signed-off-by and Acked-by lines sometimes contain commas as part of the
address. When put in recipient headers (To, Cc, Bcc), the commas are
interpreted as separators, which breaks things.

Try to recognize this case and add the appropriate quotes. For example, the
email address in ``Signed-off-by: Gruenbacher, Andreas <agruen@linbit.com>'' is
now extracted as ``"Gruenbacher, Andreas" <agruen@linbit.com>''.

Note that quilt_mail_patch_filter() can be overridden in .quiltrc; you may need
to update your .quiltrc.

commit c6750845136ff21eb13bc99ec7a6e1689100a13a
Author: Andreas Gruenbacher <agruen@linbit.com>
Date: Wed Oct 5 12:08:05 2011 +0200

quilt mail: Support double quoted special characters in email adresses

Addresses like "Gruenbacher, Andreas" <agruen@linbit.com> will work now.


Please give it a shot.

Thanks,
Andreas



\
 
 \ /
  Last update: 2011-10-05 13:25    [W:0.088 / U:0.060 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site