Re: [GIT PULL] new mailmap file for the linux kernel git repository

From: Joe Perches
Date: Thu May 31 2012 - 15:25:39 EST


On Thu, 2012-05-31 at 18:08 +0200, Philipp Riemer wrote:
> updated mailmap file for LK repository
>
> The current version of the .mailmap file does only capture a fraction of errors
> and ambiguity when looking at developers emails and names. Hence, this revised
> version tries to address many more names and, as a result, to clarify maintainer
> information. Issues addressed:
> - Typos in names
> - Typos in mails
> - Mapping of ambiguous mail adresses (e.g. {aa@xxxxx, aa@home.(none),
> aa@xxxxxxxxxxxx,...} becomes {aa@xxxxx}) but keeping LKML specific ones
> - Mapping of names to entries with only email information
> - Correct usage of special character (e.g. German umlauts)
> - Harmonization of position and style of last name (no more all-caps names
> like "First LAST" or those of "Last, First")
> - No initials in names if not necessary

Removing initials is probably not a good idea.

> - Conversion of email addresses to lower case
>
> Signed-off-by: Philipp Riemer <ruderphilipp@xxxxxxxxx>
> ---
> Finally I am back from my trip through Germany and after attending
> LinuxTag in Berlin last week, reading Documentation/SubmittingPatches,
> and checking other patch submissions, I give this topic finally a new
> try... Hopefully my submission confirms to the LK standards this
> time. I can certify that this contribution was conscientiously created
> in whole by me and therefore added the sign-off message.
>
> Please pull from
>
> git://github.com/ruderphilipp/linux.git mailmap-changes
>
> to get the changes in the attached patch file (since even the
> web-interface of GMail wraps lines) -- or simply inspect/apply that
> file directly.
>
> $ git diffstat upstream/master..origin/mailmap-changes .mailmap
> .mailmap | 2482 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++-
> 1 file changed, 2444 insertions(+), 38 deletions(-)
>

I'm still not sure this is really necessary.

It's pretty big and would need continual ongoing
maintenance for what seems to me relatively little
value.

Maybe you can diff the git shortlog before and after
this patch and determine if this is really useful.

Also, some of these entries look odd at first glance:

+Adam Brooks <adam.j.brooks@xxxxxxxxx> <From: jiang.adam@xxxxxxxxx>

Is that the same person?

-Al Viro <viro@xxxxxxxxxxxxxxxx>
-Al Viro <viro@xxxxxxxxxxxxxxxxxx>
[]
+Alexander Viro <viro@xxxxxxxxxxxxxxxx>

I think I've never seen that name before.

I stopped looking at the patch after that.

I ran a git shortlog -n -s before and after
your patch and I'm a bit confused by the
result.

Some people seem to be missing altogether.
(Avi Kivity, etc...)
Some people have _fewer_ commits.
(Andrew Morton, etc...)

Any idea why?

Here's just the start, it's a long diff

--- shortlog.ns.old 2012-05-31 12:08:12.776091892 -0700
+++ shortlog.ns.new 2012-05-31 12:07:55.596091644 -0700
@@ -1,262 +1,260 @@
12313 Linus Torvalds
- 4232 David S. Miller
+ 4383 David Miller
3627 Takashi Iwai
- 3566 Ingo Molnar
- 3208 Al Viro
+ 3568 Ingo Molnar
+ 3211 Alexander Viro
+ 2874 Russell King
2860 Mark Brown
- 2770 Russell King
2491 Tejun Heo
2461 Paul Mundt
- 2376 Thomas Gleixner
- 2363 Greg Kroah-Hartman
+ 2378 Thomas Gleixner
+ 2368 Greg Kroah-Hartman
2282 Mauro Carvalho Chehab
2245 Johannes Berg
2063 Bartlomiej Zolnierkiewicz
+ 1947 Stephen Hemminger
1924 Adrian Bunk
1817 Alan Cox
1813 Christoph Hellwig
1727 Ralf Baechle
- 1672 Stephen Hemminger
- 1619 Joe Perches
- 1558 Andrew Morton
- 1545 Randy Dunlap
+ 1625 Joe Perches
+ 1558 Randy Dunlap
+ 1557 Andrew Morton
1473 Eric Dumazet
1466 Jean Delvare
1463 Hans Verkuil
1434 Trond Myklebust
- 1364 Ben Dooks
+ 1368 Ben Dooks
1337 Jeff Garzik
1322 Andi Kleen
1309 Peter Zijlstra
- 1270 Patrick McHardy
+ 1271 Patrick McHardy
1237 Steven Rostedt
1222 Arnaldo Carvalho de Melo
1209 Axel Lin
- 1168 Mike Frysinger
+ 1168 Michael Frysinger
1155 Benjamin Herrenschmidt
- 1147 NeilBrown
+ 1100 Neil Brown
1093 David Woodhouse
1057 Dan Carpenter
1048 Herbert Xu
1028 Magnus Damm
1019 Jiri Slaby
- 1009 Avi Kivity
1006 Yinghai Lu
991 Alexey Dobriyan
987 Arnd Bergmann
979 Chris Mason
979 Dmitry Torokhov
- 972 Len Brown
+ 974 Len Brown
962 Michael Krufky
953 Julia Lawall
948 David Howells
- 945 Rafael J. Wysocki
+ 948 Rafael Wysocki
928 Jens Axboe
918 Alan Stern
914 Dave Airlie
908 Jeremy Fitzhardinge
904 David Brownell
- 898 Heiko Carstens
+ 899 Heiko Carstens
885 Chris Wilson
876 Ben Skeggs
- 857 Alex Deucher
+ 857 Alexander Deucher
856 Ben Hutchings
- 847 Stephen Rothwell
+ 848 Stephen Rothwell
+ 838 Eric Biederman
837 Sam Ravnborg
- 830 Eric W. Biederman
+ 813 Artem Bityutskiy
811 Dan Williams
807 Rusty Russell
- 787 Joerg Roedel
+ 789 JÃrg RÃdel
777 Paul Mackerras
- 774 Stefan Richter
- 767 FUJITA Tomonori
- 765 Uwe Kleine-KÃnig
- 760 John W. Linville
- 759 Roland Dreier
- 753 Artem Bityutskiy
+ 777 Stefan Richter
+ 771 Uwe Kleine-KÃnig
+ 768 Tomonori Fujita
+ 761 John Linville
+ 760 Roland Dreier
750 Kumar Gala
- 739 Steve French
+ 738 Steve French
734 Lennert Buytenhek
- 731 Luis R. Rodriguez
727 Clemens Ladisch
- 725 Christoph Lameter
+ 726 Christoph Lameter
+ 722 J. Bruce Fields
721 Oleg Nesterov
- 719 J. Bruce Fields
717 Chuck Lever
+ 716 Luis Rodriguez
711 Guennadi Liakhovetski
+ 711 Roel Kluin
705 Greg Ungerer
700 Steven Whitehouse
- 695 Bjorn Helgaas
+ 698 Bjorn Helgaas
686 Wey-Yi Guy
685 Pavel Emelyanov
680 Jesper Juhl
678 Harvey Harrison
- 673 Larry Finger
+ 676 Larry Finger
+ 667 Jean-FranÃois Moine
...

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/