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

Re: [PATCH] .mailmap: Update mailmap

From
Junio C Hamano <gitster@pobox.com>
Date
Sep 11, 2021, 00:32 UTC
Message-ID
<xmqqa6kkosst.fsf@gitster.g>
In-Reply-To
<YTt6RTwJw64tYJRw@coredump.intra.peff.net>
Jeff King <peff@peff.net> writes:
Show 17 quoted lines
> On Fri, Sep 10, 2021 at 03:22:47PM +0000, Gwyneth Morgan wrote:
>
>> The line for Jessica Clarke should probably just be
>> 
>> Jessica Clarke <jrtc27@jrtc27.com>
>> 
>> That works the same and doesn't put a reference to an old name.
>
> Thanks, that's a good suggestion. I kind of wonder if these mass
> mailmap-cleanup patches are a good idea in general. They are making
> assumptions about how people want their names to be represented, and
> whether and how they want any mappings to appear. Maybe that's something
> we should be leaving to people to propose for their own identities.
>
> Of course people who aren't active in the project anymore may not bother
> to do the cleanup, and of course messy data makes me sad. But on the
> whole, I'm not sure it's that big a deal either way.

I am not enthused by the idea of replying to this thread, knowing that many of the CC'ed addresses will bounce X-<, but I agree with you on all three counts. Even for those who are no longer active, it makes sense to unify multiple idents that are spelled differently to help "git shortlog", but which one to unify to is not something we can decide without their input.

Which leads me to suggest something like the attached patch. I wrote "Please notify us" for those who are no longer active and forgot how .mailmap entries are spelled to ask for help correcting.

Of course, the updated instruction does not prevent a motivated volunteer to contact the people _individually_ and then send in a patch with entries that the volunteer secured consent, perhaps in the form of Acked-by ;-)

 .mailmap | 5 +++++
 1 file changed, 5 insertions(+)
diff --git i/.mailmap w/.mailmap
index 9c6a446bdf..20b581c879 100644
--- i/.mailmap
+++ w/.mailmap
@@ -4,6 +4,11 @@
 # and/or not always written the same way, making contributions from the
 # same person appearing not to be so.
 #
+# If you find an incorrect entry that affects yourself, please notify us
+# at <git@vger.kernel.org> and suggest corrections.  Because the way people
+# want their names to be represented varies, please refrain from touching
+# entries for other people unless you positively know that the updated
+# entries are what they want.
 
 <nico@fluxnic.net> <nico@cam.org>
 Alejandro R. Sedeño <asedeno@MIT.EDU> <asedeno@mit.edu>
Previous: Sibi SiddharthanNext: Ævar Arnfjörð Bjarmason
Message 5 of 13 in “.mailmap: Update mailmap”
  1. .mailmap: Update mailmapFangyi Zhou, Sep 10, 2021
  2. Gwyneth MorganSep 10, 2021
  3. Jeff KingSep 10, 2021
  4. Sibi SiddharthanSep 10, 2021
  5. Junio C HamanoSep 11, 2021
  6. Ævar Arnfjörð BjarmasonSep 11, 2021
  7. Jeff KingSep 11, 2021
  8. Jeff KingSep 11, 2021
  9. Oddidies in the .mailmap parser & future syntax extensionsÆvar Arnfjörð Bjarmason, Sep 10, 2021
  10. Junio C HamanoSep 10, 2021
  11. Ævar Arnfjörð BjarmasonSep 10, 2021
  12. Junio C HamanoSep 10, 2021
  13. Ævar Arnfjörð BjarmasonSep 13, 2021

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.