Re: dt-bindings: treewide: Update @st.com email address to @foss.st.com

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

 



On Wed, 27 Oct 2021 14:56:35 +0100,
Patrice CHOTARD <patrice.chotard@xxxxxxxxxxx> wrote:
> 
> Hi Marc
> 
> +Joe Perches
> 
> On 10/27/21 8:11 AM, Patrice CHOTARD wrote:
> > Hi Marc
> > 
> > On 10/20/21 1:39 PM, Marc Zyngier wrote:
> >> On Wed, 20 Oct 2021 08:45:02 +0100,
> >> Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxxxxx> wrote:
> >>>
> >>> On 20/10/2021 08:50, patrice.chotard@xxxxxxxxxxx wrote:
> >>>> From: Patrice Chotard <patrice.chotard@xxxxxxxxxxx>
> >>>>
> >>>> Not all @st.com email address are concerned, only people who have
> >>>> a specific @foss.st.com email will see their entry updated.
> >>>> For some people, who left the company, remove their email.
> >>>>
> >>>
> >>> Please split simple address change from maintainer updates (removal,
> >>> addition).
> >>>
> >>> Also would be nice to see here explained *why* are you doing this.
> >>
> >> And why this can't be done with a single update to .mailmap, like
> >> anyone else does.
> > 
> > Thanks for the tips, yes, it will be simpler.
> > 
> > Thanks
> > Patrice
> > 
> >>
> >> 	M.
> >>
> 
> I made a try by updating .mailmap with adding a new entry with my @foss.st.com email :
> 
>  Pali Rohár <pali@xxxxxxxxxx> <pali.rohar@xxxxxxxxx>
>  Paolo 'Blaisorblade' Giarrusso <blaisorblade@xxxxxxxx>
> +Patrice Chotard <patrice.chotard@xxxxxxxxxxx> <patrice.chotard@xxxxxx>
>  Patrick Mochel <mochel@xxxxxxxxxxxxxxxxxx>
>  Paul Burton <paulburton@xxxxxxxxxx> <paul.burton@xxxxxxxxxx>
> 
> But when running ./scripts/get_maintainer.pl Documentation/devicetree/bindings/arm/sti.yaml, by old email is still displayed
> 
> Rob Herring <robh+dt@xxxxxxxxxx> (maintainer:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS)
> Patrice Chotard <patrice.chotard@xxxxxx> (in file)
> devicetree@xxxxxxxxxxxxxxx (open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS)
> linux-kernel@xxxxxxxxxxxxxxx (open list)
> 
> By default, the get_maintainer.pl script is using .mailmap file
> ($email_use_mailmap = 1).
> 
> It seems there is an issue with get_maintainer.pl and maintainer
> name/e-mail found in yaml file ?

Try this (warning though: my Perl-foo is non-existent).

	M.

diff --git a/scripts/get_maintainer.pl b/scripts/get_maintainer.pl
index 2075db0c08b8..2a84a3fb0130 100755
--- a/scripts/get_maintainer.pl
+++ b/scripts/get_maintainer.pl
@@ -986,6 +986,7 @@ sub get_maintainers {
 	    my ($name, $address) = parse_email($email);
 
 	    my $tmp_email = format_email($name, $address, $email_usename);
+	    $tmp_email = mailmap_email($tmp_email);
 	    push_email_address($tmp_email, '');
 	    add_role($tmp_email, 'in file');
 	}

-- 
Without deviation from the norm, progress is not possible.




[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Pulse Audio]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux