{"thread":{"id":"42986","subject":"get_maintainer.pl and .mailmap entries with more than 2 addresses","startedAt":"2016-08-02T16:47:02Z","lastAt":"2016-08-03T00:50:59Z","messageCount":5,"participants":["Joe Perches","Junio C Hamano","Shuah Khan","Mauro Carvalho Chehab"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"292843","messageId":"1470155781.3998.181.camel@perches.com","threadId":"42986","inReplyTo":null,"subject":"get_maintainer.pl and .mailmap entries with more than 2 addresses","fromName":"Joe Perches","fromEmail":"joe@perches.com","sentAt":"2016-08-02T16:36:21Z","receivedAt":"2016-08-02T16:47:02Z","isPatch":false,"sender":{"key":"joe@perches.com","avatar":"https://avatars.githubusercontent.com/u/13122723?v=4"},"body":"Hello Florian.\n\nThere is at least an oddity with get_maintainer handling of a\n.mailmap entry form.\n\nFor instance:\n\nMauro's .mailmap entry is:\nMauro Carvalho Chehab \n\nIs this a valid form?\n\nget_maintainer output for Mauro is:\n\n$ ./scripts/get_maintainer.pl drivers/media/ -f\nMauro Carvalho Chehab (maintainer:MEDIA INPUT INFRASTRUCTURE (V4L/DVB))\n\nI believe the Mauro's and Shuah's .mailmap entries are improper and\nshould be changed, but I'm not completely aware of git .mailmap\nhandling and the documentation seems weakly specified.\n\nhttps://git-scm.com/docs/git-check-mailmap\n\nMaybe get_maintainer.pl needs a change or perhaps this patch\nis appropriate.\n---\n .mailmap | 12 ++++++++++--\n 1 file changed, 10 insertions(+), 2 deletions(-)\n\ndiff --git a/.mailmap b/.mailmap\nindex c0d5704..c7f92ca 100644\n--- a/.mailmap\n+++ b/.mailmap\n@@ -96,7 +96,12 @@ Linus Lüssing \n Linus Lüssing \n Mark Brown \n Matthieu CASTET \n-Mauro Carvalho Chehab \n+Mauro Carvalho Chehab \n+Mauro Carvalho Chehab \n+Mauro Carvalho Chehab \n+Mauro Carvalho Chehab \n+Mauro Carvalho Chehab \n+Mauro Carvalho Chehab \n Matt Ranostay Matthew Ranostay \n Matt Ranostay \n Mayuresh Janorkar \n@@ -132,7 +137,10 @@ Santosh Shilimkar \n Sascha Hauer \n S.Çağlar Onur \n Shiraz Hashim \n-Shuah Khan \n+Shuah Khan \n+Shuah Khan \n+Shuah Khan  \n+Shuah Khan  \n Simon Kelley \n Stéphane Witzmann \n Stephen Hemminger \n\n"},{"id":"292853","messageId":"xmqqpoprdojp.fsf@gitster.mtv.corp.google.com","threadId":"42986","inReplyTo":"1470155781.3998.181.camel@perches.com","subject":"Re: get_maintainer.pl and .mailmap entries with more than 2 addresses","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2016-08-02T18:16:26Z","receivedAt":"2016-08-02T18:20:39Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Joe Perches writes:\n\n> Hello Florian.\n>\n> There is at least an oddity with get_maintainer handling of a\n> .mailmap entry form.\n>\n> For instance:\n>\n> Mauro's .mailmap entry is:\n> Mauro Carvalho Chehab \n>\n> Is this a valid form?\n\nI do not think so, according to \"git shortlog --help\" (the canonical\nsource of the document is Documentation/mailmap.txt, but shortlog\ndoc includes it). Here is the relevant bits.\n\n In the simple form, each line in the file consists of the canonical\n real name of an author, whitespace, and an email address used in the\n commit (enclosed by '<' and '>') to map to the name. For example:\n --\n Proper Name \n --\n\n The more complex forms are:\n --\n \n --\n which allows mailmap to replace only the email part of a commit, and:\n --\n Proper Name \n --\n which allows mailmap to replace both the name and the email of a\n commit matching the specified commit email address, and:\n --\n Proper Name Commit Name \n --\n which allows mailmap to replace both the name and the email of a\n commit matching both the specified commit name and email address.\n\n"},{"id":"292874","messageId":"1470176777.3998.203.camel@perches.com","threadId":"42986","inReplyTo":"6A56763C-D795-4943-AFE9-8A53ECD61AF6@mickler.org","subject":"Re: get_maintainer.pl and .mailmap entries with more than 2 addresses","fromName":"Joe Perches","fromEmail":"joe@perches.com","sentAt":"2016-08-02T22:26:17Z","receivedAt":"2016-08-02T22:26:49Z","isPatch":false,"sender":{"key":"joe@perches.com","avatar":"https://avatars.githubusercontent.com/u/13122723?v=4"},"body":"On Wed, 2016-08-03 at 00:17 +0200, Florian Mickler wrote:\n> cc'd mchehab@s-opensource.com  (Mauro, is your kernel.org address up?)\n> \n>  Am Tue, 02 Aug 2016 09:36:21 -0700\n> schrieb Joe Perches :\n> \n> > \n> > Hello Florian.\n> > There is at least an oddity with get_maintainer handling of a\n> > .mailmap entry form.\n> > \n> > For instance:\n> > \n> > Mauro's .mailmap entry is:\n> > Mauro Carvalho Chehab \n> > \n> > \n> > \n> > Is this a valid form?\n> > \n> > get_maintainer output for Mauro is:\n> > \n> > $ ./scripts/get_maintainer.pl drivers/media/ -f\n> > Mauro Carvalho Chehab \n> > \n> > (maintainer:MEDIA INPUT INFRASTRUCTURE\n> > (V4L/DVB))\n> > \n> > I believe the Mauro's and Shuah's .mailmap entries are improper and\n> > should be changed, but I'm not completely aware of git .mailmap\n> > handling and the documentation seems weakly specified.\n> > \n> Hmm.. looking at Mauros last .mailmap commit it seems like your patch is\n> ok for Mauro. \n> \n> Although and are probably\n> missing? (@Mauro) \n> \n> \n> $ git shortlog | grep \"^Mauro C\"\n> Mauro Carvalho Chehab (4404):\n> $ git log | grep \"^Author:.*Mauro Carvalho Chehab\" | sort | uniq -c\n>       2 Author: Mauro Carvalho Chehab \n>     146 Author: Mauro Carvalho Chehab \n>     645 Author: Mauro Carvalho Chehab \n>     794 Author: Mauro Carvalho Chehab \n>    2015 Author: Mauro Carvalho Chehab \n>     448 Author: Mauro Carvalho Chehab \n>     353 Author: Mauro Carvalho Chehab \n>       1 Author: Mauro Carvalho Chehab \n> \n> \n> \n> Anyway, from a technical viewpoint your patches seem to fix\n> the .mailmap entry as the author intended. (See Junio's Email for the\n> documantation part) \n> But I would wait for the ack from Mauro and Shuah. \n\nAs far as I understand, a single entry with just their\nname and preferred email address would work too because\nthe name parts are all spelled identically.\n\n\n"},{"id":"292880","messageId":"57A128A6.4040303@kernel.org","threadId":"42986","inReplyTo":"1470176777.3998.203.camel@perches.com","subject":"Re: get_maintainer.pl and .mailmap entries with more than 2 addresses","fromName":"Shuah Khan","fromEmail":"shuah@kernel.org","sentAt":"2016-08-02T23:11:34Z","receivedAt":"2016-08-02T23:20:35Z","isPatch":false,"sender":{"key":"shuah@kernel.org","avatar":null},"body":"On 08/02/2016 04:26 PM, Joe Perches wrote:\n> On Wed, 2016-08-03 at 00:17 +0200, Florian Mickler wrote:\n>> cc'd mchehab@s-opensource.com (Mauro, is your kernel.org address up?)\n>>\n>> Am Tue, 02 Aug 2016 09:36:21 -0700\n>> schrieb Joe Perches :\n>>\n>>>\n>>> Hello Florian.\n>>> There is at least an oddity with get_maintainer handling of a\n>>> .mailmap entry form.\n>>>\n>>> For instance:\n>>>\n>>> Mauro's .mailmap entry is:\n>>> Mauro Carvalho Chehab \n>>> \n>>> \n>>>\n>>> Is this a valid form?\n>>>\n>>> get_maintainer output for Mauro is:\n>>>\n>>> $ ./scripts/get_maintainer.pl drivers/media/ -f\n>>> Mauro Carvalho Chehab \n>>> \n>>> (maintainer:MEDIA INPUT INFRASTRUCTURE\n>>> (V4L/DVB))\n>>>\n>>> I believe the Mauro's and Shuah's .mailmap entries are improper and\n>>> should be changed, but I'm not completely aware of git .mailmap\n>>> handling and the documentation seems weakly specified.\n>>>\n>> Hmm.. looking at Mauros last .mailmap commit it seems like your patch is\n>> ok for Mauro. \n>>\n>> Although and are probably\n>> missing? (@Mauro) \n>>\n>>\n>> $ git shortlog | grep \"^Mauro C\"\n>> Mauro Carvalho Chehab (4404):\n>> $ git log | grep \"^Author:.*Mauro Carvalho Chehab\" | sort | uniq -c\n>> 2 Author: Mauro Carvalho Chehab \n>> 146 Author: Mauro Carvalho Chehab \n>> 645 Author: Mauro Carvalho Chehab \n>> 794 Author: Mauro Carvalho Chehab \n>> 2015 Author: Mauro Carvalho Chehab \n>> 448 Author: Mauro Carvalho Chehab \n>> 353 Author: Mauro Carvalho Chehab \n>> 1 Author: Mauro Carvalho Chehab \n>>\n>>\n>>\n>> Anyway, from a technical viewpoint your patches seem to fix\n>> the .mailmap entry as the author intended. (See Junio's Email for the\n>> documantation part) \n>> But I would wait for the ack from Mauro and Shuah. \n> \n> As far as I understand, a single entry with just their\n> name and preferred email address would work too because\n> the name parts are all spelled identically.\n> \n> \n\nI am fine with change to my entry. Thanks for fixing it.\n\nAcked-by: Shuah Khan \n\nthanks,\n-- Shuah\n\n"},{"id":"292882","messageId":"20160802215006.39462d7b@recife.lan","threadId":"42986","inReplyTo":"6A56763C-D795-4943-AFE9-8A53ECD61AF6@mickler.org","subject":"Re: get_maintainer.pl and .mailmap entries with more than 2 addresses","fromName":"Mauro Carvalho Chehab","fromEmail":"mchehab@s-opensource.com","sentAt":"2016-08-03T00:50:06Z","receivedAt":"2016-08-03T00:50:59Z","isPatch":false,"sender":{"key":"mchehab@s-opensource.com","avatar":null},"body":"Em Wed, 03 Aug 2016 00:17:10 +0200\nFlorian Mickler escreveu:\n\n> cc'd mchehab@s-opensource.com (Mauro, is your kernel.org address up?)\n\nYes, my kernel.org address is up. Better to keep it as the canonical address,\nas this is unlikely to change as it is not associated to an e-mail provider.\n\n> \n> Am Tue, 02 Aug 2016 09:36:21 -0700\n> schrieb Joe Perches :\n> \n> > Hello Florian.\n> > \n> > There is at least an oddity with get_maintainer handling of a\n> > .mailmap entry form.\n> > \n> > For instance:\n> > \n> > Mauro's .mailmap entry is:\n> > Mauro Carvalho Chehab \n> > \n> > \n> > \n> > Is this a valid form?\n> > \n> > get_maintainer output for Mauro is:\n> > \n> > $ ./scripts/get_maintainer.pl drivers/media/ -f\n> > Mauro Carvalho Chehab \n> > \n> > (maintainer:MEDIA INPUT INFRASTRUCTURE\n> > (V4L/DVB))\n> > \n> > I believe the Mauro's and Shuah's .mailmap entries are improper and\n> > should be changed, but I'm not completely aware of git .mailmap\n> > handling and the documentation seems weakly specified.\n> > \n> \n> Hmm.. looking at Mauros last .mailmap commit it seems like your patch is\n> ok for Mauro. \n> \n> Although and are probably\n> missing? (@Mauro) \n\nThe mchehab@brturbo.com.br is indeed an old email that I used on my\nfirst submissions.\n\nI don't know a mywing81@gmail.com address... that looks really weird\non my eyes...\n\nHmm...\n\n$ git log --author \"mywing81@gmail.com\"\ncommit fe8b671306c78a963934cb5d6e354178390b8c87\nAuthor: Mauro Carvalho Chehab \nDate: Thu Sep 30 14:46:47 2010 -0300\n\n [media] saa7134: port Asus P7131 Hybrid to use the new rc-core\n \n The rc map table were corrected thanks to Giorgio input and tests.\n \n Reported-by: Giorgio Vazzana \n Tested-by: Giorgio Vazzana \n Signed-off-by: Mauro Carvalho Chehab \n\nI don't remember the dirty details about such patch anymore... too\nmany years ago. From the custody chain, I _suspect_ that Giorgio\nproposed a patch to be applied against the saa7134 driver, but the \nremote controller map table was moved to a separate file about 7\nmonths before such patch by those changesets:\n\n6686fa6917ca V4L/DVB: Break Remote Controller keymaps into modules\n77b7422d48cd V4L/DVB: ir-common: move IR tables from ir-keymaps.c to a separate file\n\nI likely rewrote his patch against the new driver, but somehow\nI mangled with the author name/email. In any case, mywing81@gmail.com\nshould not be associated with me.\n \n> \n> $ git shortlog | grep \"^Mauro C\"\n> Mauro Carvalho Chehab (4404):\n> $ git log | grep \"^Author:.*Mauro Carvalho Chehab\" | sort | uniq -c\n> 2 Author: Mauro Carvalho Chehab \n> 146 Author: Mauro Carvalho Chehab \n> 645 Author: Mauro Carvalho Chehab \n> 794 Author: Mauro Carvalho Chehab \n> 2015 Author: Mauro Carvalho Chehab \n> 448 Author: Mauro Carvalho Chehab \n> 353 Author: Mauro Carvalho Chehab \n\nAll above are my e-mails. Let's point them all to mchehab@kernel.org.\n\n> 1 Author: Mauro Carvalho Chehab \n\nThis one is mangled and doesn't belong to me. So, it shouldn't be\nat the .mailmap file.\n\n> \n> \n> \n> Anyway, from a technical viewpoint your patches seem to fix\n> the .mailmap entry as the author intended. (See Junio's Email for the\n> documantation part) \n> But I would wait for the ack from Mauro and Shuah. \n\nWith the above changes,\n\nAcked-by: Mauro Carvalho Chehab \n\n\nThanks,\nMauro\n"}]}