[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH] get-maintainer.pl: Dont fall over when L: contains a display name



Hi Lars,

On 15/01/2020 18:11, Lars Kurth wrote:
I should have added more people to this change. The issue without this fix is 
that entries such as

L: xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxxx>

break get_maintainer.pl and thus add_maintainers.pl

Lars

On 10/01/2020, 21:19, "Lars Kurth" <lars.kurth@xxxxxxxxxxxxxx> wrote:

     From: Lars Kurth <lars.kurth@xxxxxxxxxx>
Prior to this change e-mail addresses of the form "display name
     <email>" would result into empty output. Also see
     https://lists.xenproject.org/archives/html/xen-devel/2020-01/msg00753.html
Signed-off-by: Lars Kurth <lars.kurth@xxxxxxxxxx>

Reviewed-by: Julien Grall <julien@xxxxxxx>

Cheers,

     ---
     CC: jgross@xxxxxxxx
     ---
      scripts/get_maintainer.pl | 2 +-
      1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/scripts/get_maintainer.pl b/scripts/get_maintainer.pl
     index 2e661f47d8..48e07370e8 100755
     --- a/scripts/get_maintainer.pl
     +++ b/scripts/get_maintainer.pl
     @@ -1073,7 +1073,7 @@ sub add_categories {
            my $ptype = $1;
            my $pvalue = $2;
            if ($ptype eq "L") {
     -          my $list_address = $pvalue;
     +          my ($list_name, $list_address) = parse_email($pvalue);
                my $list_additional = "";
                my $list_role = get_list_role($i);
--
     2.13.0

--
Julien Grall

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.