Re: get_maintainer.pl and .mailmap entries with more than 2 addresses

[Joe Perches]

On Wed, 2016-08-03 at 00:17 +0200, Florian Mickler wrote:
> cc'd mchehab@xxxxxxxxxxxxxxxx  (Mauro, is your kernel.org address up?)
> 
>  Am Tue, 02 Aug 2016 09:36:21 -0700
> schrieb Joe Perches <joe@xxxxxxxxxxx>:
> 
> > 
> > Hello Florian.
> > There is at least an oddity with get_maintainer handling of a
> > .mailmap entry form.
> > 
> > For instance:
> > 
> > Mauro's .mailmap entry is:
> > Mauro Carvalho Chehab <mchehab@xxxxxxxxxx> <maurochehab@xxxxxxxxx>
> > <mchehab@xxxxxxxxxxxxx> <mchehab@xxxxxxxxxx> <m.chehab@xxxxxxxxxxx>
> > <mchehab@xxxxxxxxxxxxxxx> <mchehab@xxxxxxxxxxxxxxxx>
> > 
> > Is this a valid form?
> > 
> > get_maintainer output for Mauro is:
> > 
> > $ ./scripts/get_maintainer.pl drivers/media/ -f
> > Mauro Carvalho Chehab <mchehab@xxxxxxxxxx> <maurochehab@xxxxxxxxx>
> > <mchehab@xxxxxxxxxxxxx> <mchehab@xxxxxxxxxx> <m.chehab@xxxxxxxxxxx>
> > <mchehab@xxxxxxxxxxxxxxx> (maintainer:MEDIA INPUT INFRASTRUCTURE
> > (V4L/DVB))
> > 
> > I believe the Mauro's and Shuah's .mailmap entries are improper and
> > should be changed, but I'm not completely aware of git .mailmap
> > handling and the documentation seems weakly specified.
> > 
> Hmm.. looking at Mauros last .mailmap commit it seems like your patch is
> ok for Mauro. 
> 
> Although <mywing81@xxxxxxxxx> and <mchehab@xxxxxxxxxxxxxx> are probably
> missing? (@Mauro) 
> 
> 
> $ git shortlog | grep "^Mauro C"
> Mauro Carvalho Chehab (4404):
> $ git log | grep "^Author:.*Mauro Carvalho Chehab" | sort | uniq -c
>       2 Author: Mauro Carvalho Chehab <maurochehab@xxxxxxxxx>
>     146 Author: Mauro Carvalho Chehab <mchehab@xxxxxxxxxxxxxx>
>     645 Author: Mauro Carvalho Chehab <mchehab@xxxxxxxxxxxxx>
>     794 Author: Mauro Carvalho Chehab <mchehab@xxxxxxxxxxxxxxx>
>    2015 Author: Mauro Carvalho Chehab <mchehab@xxxxxxxxxx>
>     448 Author: Mauro Carvalho Chehab <m.chehab@xxxxxxxxxxx>
>     353 Author: Mauro Carvalho Chehab <mchehab@xxxxxxxxxxxxxxxx>
>       1 Author: Mauro Carvalho Chehab <mywing81@xxxxxxxxx>
> 
> 
> 
> Anyway, from a technical viewpoint your patches seem to fix
> the .mailmap entry as the author intended. (See Junio's Email for the
> documantation part) 
> But I would wait for the ack from Mauro and Shuah. 

As far as I understand, a single entry with just their
name and preferred email address would work too because
the name parts are all spelled identically.