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. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html