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

From: Joe Perches <[email-masked]>
Date: Tue, 02 Aug 2016 09:36:21 -0700

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 <[email-masked]> <[email-masked]> <[email-masked]> <[email-masked]> <[email-masked]> <[email-masked]> <[email-masked]>

Is this a valid form?

get_maintainer output for Mauro is:

$ ./scripts/get_maintainer.pl drivers/media/ -f
Mauro Carvalho Chehab <[email-masked]> <[email-masked]> <[email-masked]> <[email-masked]> <[email-masked]> <[email-masked]> (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 <[email-masked]> <[email-masked]>
 Linus Lüssing <[email-masked]> <[email-masked]>
 Mark Brown <[email-masked]>
 Matthieu CASTET <[email-masked]>
-Mauro Carvalho Chehab <[email-masked]> <[email-masked]> <[email-masked]> <[email-masked]> <[email-masked]> <[email-masked]> <[email-masked]>
+Mauro Carvalho Chehab <[email-masked]> <[email-masked]>
+Mauro Carvalho Chehab <[email-masked]> <[email-masked]>
+Mauro Carvalho Chehab <[email-masked]> <[email-masked]>
+Mauro Carvalho Chehab <[email-masked]> <[email-masked]>
+Mauro Carvalho Chehab <[email-masked]> <[email-masked]>
+Mauro Carvalho Chehab <[email-masked]> <[email-masked]>
 Matt Ranostay <[email-masked]> Matthew Ranostay <[email-masked]>
 Matt Ranostay <[email-masked]> <[email-masked]>
 Mayuresh Janorkar <[email-masked]>
@@ -132,7 +137,10 @@ Santosh Shilimkar <[email-masked]>
 Sascha Hauer <[email-masked]>
 S.Çağlar Onur <[email-masked]>
 Shiraz Hashim <[email-masked]> <[email-masked]>
-Shuah Khan <[email-masked]> <[email-masked]> <[email-masked]> <[email-masked]> <[email-masked]>
+Shuah Khan <[email-masked]> <[email-masked]>
+Shuah Khan <[email-masked]> <[email-masked]>
+Shuah Khan <[email-masked]> <[email-masked]>
+Shuah Khan <[email-masked]> <[email-masked]>
 Simon Kelley <[email-masked]>
 Stéphane Witzmann <[email-masked]>
 Stephen Hemminger <[email-masked]>