Re: Failure to extra stable@vger.kernel.org addresses

From: Junio C Hamano <gitster@pobox.com>
Date: Nov 19, 2012, 23:57 UTC
Message-ID: <7vlidxuowf.fsf@alter.siamese.dyndns.org>
In-Reply-To: <CAMP44s0f0zYa1FVf9RhNuwYJbkQ7zPwgJ6=ty3c5knjo5a2TNw@mail.gmail.com>

Felipe Contreras <felipe.contreras@gmail.com> writes:

Show 15 quoted lines

> On Mon, Nov 19, 2012 at 11:58 PM, Krzysztof Mazur <krzysiek@podlesie.net> wrote:
>
>> --- a/git-send-email.perl
>> +++ b/git-send-email.perl
>> @@ -924,6 +924,10 @@ sub quote_subject {
>>  # use the simplest quoting being able to handle the recipient
>>  sub sanitize_address {
>>         my ($recipient) = @_;
>> +
>> +       # remove garbage after email address
>> +       $recipient =~ s/(.*>).*$/$1/;
>> +
>
> Looks fine, but I would do s/(.*?>)(.*)$/$1/, so that 'test
> <foo@bar.com> <#comment>' gets the second comment removed.

Yeah, but do you need to capture the second group? IOW, like "s/(.*?>).*$/$1/" perhaps?