git/list[1] front-page[2] threads[3] people[4] search[5] about
 

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

From
SKShuah Khan <shuah@kernel.org>
Date
Aug 2, 2016, 23:11 UTC
Message-ID
<57A128A6.4040303@kernel.org>
In-Reply-To
<1470176777.3998.203.camel@perches.com>
On 08/02/2016 04:26 PM, Joe Perches wrote:
Show 63 quoted lines
> On Wed, 2016-08-03 at 00:17 +0200, Florian Mickler wrote:
>> cc'd mchehab@s-opensource.com  (Mauro, is your kernel.org address up?)
>>
>>  Am Tue, 02 Aug 2016 09:36:21 -0700
>> schrieb Joe Perches <joe@perches.com>:
>>
>>>
>>> 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@kernel.org> <maurochehab@gmail.com>
>>> <mchehab@infradead.org> <mchehab@redhat.com> <m.chehab@samsung.com>
>>> <mchehab@osg.samsung.com> <mchehab@s-opensource.com>
>>>
>>> Is this a valid form?
>>>
>>> get_maintainer output for Mauro is:
>>>
>>> $ ./scripts/get_maintainer.pl drivers/media/ -f
>>> Mauro Carvalho Chehab <mchehab@kernel.org> <maurochehab@gmail.com>
>>> <mchehab@infradead.org> <mchehab@redhat.com> <m.chehab@samsung.com>
>>> <mchehab@osg.samsung.com> (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@gmail.com> and <mchehab@brturbo.com.br> 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@gmail.com>
>>     146 Author: Mauro Carvalho Chehab <mchehab@brturbo.com.br>
>>     645 Author: Mauro Carvalho Chehab <mchehab@infradead.org>
>>     794 Author: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
>>    2015 Author: Mauro Carvalho Chehab <mchehab@redhat.com>
>>     448 Author: Mauro Carvalho Chehab <m.chehab@samsung.com>
>>     353 Author: Mauro Carvalho Chehab <mchehab@s-opensource.com>
>>       1 Author: Mauro Carvalho Chehab <mywing81@gmail.com>
>>
>>
>>
>> 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.
> 
> 
I am fine with change to my entry. Thanks for fixing it.
Acked-by: Shuah Khan <shuahkh@osg.samsung.com>

thanks, -- Shuah

Previous: Joe PerchesNext: Mauro Carvalho Chehab
Message 4 of 5 in “get_maintainer.pl and .mailmap entries with more than 2 addresses”
  1. Joe PerchesAug 2, 2016
  2. Junio C HamanoAug 2, 2016
  3. Joe PerchesAug 2, 2016
  4. Shuah KhanAug 2, 2016
  5. Mauro Carvalho ChehabAug 3, 2016

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.