git@vger.kernel.org mailing list mirror (one of many)
 help / color / mirror / code / Atom feed
From: Junio C Hamano <gitster@pobox.com>
To: Jeff King <peff@peff.net>
Cc: "Gwyneth Morgan" <gwymor@tilde.club>,
	"Fangyi Zhou" <me@fangyi.io>,
	git@vger.kernel.org,
	"Birger Skogeng Pedersen" <birger.sp@gmail.com>,
	"Birger Skogeng Pedersen" <birgersp@gmail.com>,
	"Brandon Williams" <bwilliamseng@gmail.com>,
	"CB Bailey" <cbailey32@bloomberg.net>,
	"Christopher Díaz Riveros" <christopher.diaz.riv@gmail.com>,
	"Christopher Díaz Riveros" <chrisadr@gentoo.org>,
	"Ed Maste" <emaste@freebsd.org>,
	"Jean-Noël Avila" <jn.avila@free.fr>,
	"Jean-Noel Avila" <jean-noel.avila@scantech.fr>,
	"Jessica Clarke" <jrtc27@jrtc27.com>,
	"Jiang Xin" <worldhello.net@gmail.com>,
	"Jiang Xin" <zhiyou.jx@alibaba-inc.com>,
	"Kazuhiro Kato" <kato-k@ksysllc.co.jp>,
	"Kazuhiro Kato" <kazuhiro.kato@hotmail.co.jp>,
	"Kevin Willford" <Kevin.Willford@microsoft.com>,
	"Kevin Willford" <kewillf@microsoft.com>,
	"Peter Kaestle" <peter@piie.net>,
	"Peter Kaestle" <peter.kaestle@nokia.com>,
	"Sibi Siddharthan" <sibisiddharthan.github@gmail.com>,
	"Sibi Siddharthan" <sibisiv.siddharthan@gmail.com>,
	"Slavica Đukić" <slawica92@hotmail.com>,
	"Slavica Djukic" <slavicadj.ip2018@gmail.com>
Subject: Re: [PATCH] .mailmap: Update mailmap
Date: Fri, 10 Sep 2021 17:32:50 -0700	[thread overview]
Message-ID: <xmqqa6kkosst.fsf@gitster.g> (raw)
In-Reply-To: YTt6RTwJw64tYJRw@coredump.intra.peff.net

Jeff King <peff@peff.net> writes:

> 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>

  parent reply	other threads:[~2021-09-11  0:32 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-09-10 13:02 [PATCH] .mailmap: Update mailmap Fangyi Zhou
2021-09-10 15:22 ` Gwyneth Morgan
2021-09-10 15:31   ` Jeff King
2021-09-10 15:35     ` Sibi Siddharthan
2021-09-11  0:32     ` Junio C Hamano [this message]
2021-09-11  1:31       ` Ævar Arnfjörð Bjarmason
2021-09-11 14:52         ` Jeff King
2021-09-11 14:47       ` Jeff King
2021-09-10 16:48   ` Oddidies in the .mailmap parser & future syntax extensions Ævar Arnfjörð Bjarmason
2021-09-10 18:11     ` Junio C Hamano
2021-09-10 19:50       ` Ævar Arnfjörð Bjarmason
2021-09-10 20:20         ` Junio C Hamano
2021-09-13  4:02           ` Ævar Arnfjörð Bjarmason

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: http://vger.kernel.org/majordomo-info.html

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=xmqqa6kkosst.fsf@gitster.g \
    --to=gitster@pobox.com \
    --cc=Kevin.Willford@microsoft.com \
    --cc=birger.sp@gmail.com \
    --cc=birgersp@gmail.com \
    --cc=bwilliamseng@gmail.com \
    --cc=cbailey32@bloomberg.net \
    --cc=chrisadr@gentoo.org \
    --cc=christopher.diaz.riv@gmail.com \
    --cc=emaste@freebsd.org \
    --cc=git@vger.kernel.org \
    --cc=gwymor@tilde.club \
    --cc=jean-noel.avila@scantech.fr \
    --cc=jn.avila@free.fr \
    --cc=jrtc27@jrtc27.com \
    --cc=kato-k@ksysllc.co.jp \
    --cc=kazuhiro.kato@hotmail.co.jp \
    --cc=kewillf@microsoft.com \
    --cc=me@fangyi.io \
    --cc=peff@peff.net \
    --cc=peter.kaestle@nokia.com \
    --cc=peter@piie.net \
    --cc=sibisiddharthan.github@gmail.com \
    --cc=sibisiv.siddharthan@gmail.com \
    --cc=slavicadj.ip2018@gmail.com \
    --cc=slawica92@hotmail.com \
    --cc=worldhello.net@gmail.com \
    --cc=zhiyou.jx@alibaba-inc.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://80x24.org/mirrors/git.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).