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

From: Joe Perches
Date: Tue Aug 02 2016 - 12:38:40 EST


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.

https://git-scm.com/docs/git-check-mailmap

Maybe get_maintainer.pl needs a change or perhaps this patch
is appropriate.
---
Â.mailmap | 12 ++++++++++--
Â1 file changed, 10 insertions(+), 2 deletions(-)

diff --git a/.mailmap b/.mailmap
index c0d5704..c7f92ca 100644
--- a/.mailmap
+++ b/.mailmap
@@ -96,7 +96,12 @@ Linus LÃssing <linus.luessing@xxxxxxxxx> <linus.luessing@xxxxxx>
ÂLinus LÃssing <linus.luessing@xxxxxxxxx> <linus.luessing@xxxxxxxx>
ÂMark Brown <broonie@xxxxxxxxxxxxx>
ÂMatthieu CASTET <castet.matthieu@xxxxxxx>
-Mauro Carvalho Chehab <mchehab@xxxxxxxxxx> <maurochehab@xxxxxxxxx> <mchehab@xxxxxxxxxxxxx> <mchehab@xxxxxxxxxx> <m.chehab@xxxxxxxxxxx> <mchehab@xxxxxxxxxxxxxxx> <mchehab@xxxxxxxxxxxxxxxx>
+Mauro Carvalho Chehab <mchehab@xxxxxxxxxx> <maurochehab@xxxxxxxxx>
+Mauro Carvalho Chehab <mchehab@xxxxxxxxxx> <mchehab@xxxxxxxxxxxxx>
+Mauro Carvalho Chehab <mchehab@xxxxxxxxxx> <mchehab@xxxxxxxxxx>
+Mauro Carvalho Chehab <mchehab@xxxxxxxxxx> <m.chehab@xxxxxxxxxxx>
+Mauro Carvalho Chehab <mchehab@xxxxxxxxxx> <mchehab@xxxxxxxxxxxxxxx>
+Mauro Carvalho Chehab <mchehab@xxxxxxxxxx> <mchehab@xxxxxxxxxxxxxxxx>
ÂMatt Ranostay <mranostay@xxxxxxxxx> Matthew Ranostay <mranostay@xxxxxxxxxxxxxxxxx>
ÂMatt Ranostay <mranostay@xxxxxxxxx> <matt.ranostay@xxxxxxxxx>
ÂMayuresh Janorkar <mayur@xxxxxx>
@@ -132,7 +137,10 @@ Santosh Shilimkar <santosh.shilimkar@xxxxxxxxxx>
ÂSascha Hauer <s.hauer@xxxxxxxxxxxxxx>
ÂS.ÃaÄlar Onur <caglar@xxxxxxxxxxxxx>
ÂShiraz Hashim <shiraz.linux.kernel@xxxxxxxxx> <shiraz.hashim@xxxxxx>
-Shuah Khan <shuah@xxxxxxxxxx> <shuahkhan@xxxxxxxxx> <shuah.khan@xxxxxx> <shuahkh@xxxxxxxxxxxxxxx> <shuah.kh@xxxxxxxxxxx>
+Shuah Khan <shuah@xxxxxxxxxx> <shuahkhan@xxxxxxxxx>
+Shuah Khan <shuah@xxxxxxxxxx> <shuah.khan@xxxxxx>
+Shuah KhanÂ<shuah@xxxxxxxxxx> <shuahkh@xxxxxxxxxxxxxxx>
+Shuah KhanÂ<shuah@xxxxxxxxxx> <shuah.kh@xxxxxxxxxxx>
ÂSimon Kelley <simon@xxxxxxxxxxxxxxxxx>
ÂStÃphane Witzmann <stephane.witzmann@xxxxxxxxxxxxxxxxxxxxxxxxx>
ÂStephen Hemminger <shemminger@xxxxxxxx>