From: Michael J Gruber <git@drmicha.warpmail.net>
To: Nicolas Pitre <nico@fluxnic.net>
Cc: Junio C Hamano <gitster@pobox.com>, git@vger.kernel.org
Subject: Re: [PATCH] .mailmap: Entries for Alex Bennée, Deskin Miller, Vitaly "_Vi" Shukela
Date: Sun, 21 Mar 2010 14:36:13 +0100 [thread overview]
Message-ID: <4BA620CD.3060101@drmicha.warpmail.net> (raw)
In-Reply-To: <alpine.LFD.2.00.1003201405100.31128@xanadu.home>
Nicolas Pitre venit, vidit, dixit 20.03.2010 19:10:
> On Sat, 20 Mar 2010, Junio C Hamano wrote:
>
>> Michael J Gruber <git@drmicha.warpmail.net> writes:
>>
>>> With the current .mailmap, git shortlog shows the following for these:
>>
>> Thanks for doing the gruntwork; very much appreciated.
>>
>> I'll backburner this and wait until the people who are described by these
>> entries say they want these entries in the .mailmap file. If they do not
>> like the variant that your patch suggested, and if they care, then we will
>> get updates that should be applied. Otherwise they themselves don't care,
>> so why should we ;-)?
>
> I think at least one person (Michael) cared, so if the people described
> by those entries don't provide feedback then that means they don't care
> and you shouldn't wait after them to apply the patch including those
> entries.
Maybe I should have cc'ed Vitaly, the others are straight typo fixes.
(One may even argue that Vitaly's change is, too.) Smothering on the
backburner for a while, is fine, though ;)
I think for git.git it's simple enough to have "consistent" authorship
information, partly due to the scale, partly due to the fact that by
definition, submitters have a certain affluence with Git...
Also, if used without "-e", shortlog coalesces authors by full name. I
reckon this is the most common way it is used, and it's the mode for
which I provided cleanup.
If you distinguish by e-mail (-e) then there are many obvious multiple
entries, but it's not up to me to decide which is the main e-mail
address. One could take the latest one used, but some people distinguish
between work for hire (maybe for several employers) and private submissions.
Michael
next prev parent reply other threads:[~2010-03-21 13:36 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-03-19 11:02 [PATCH] .mailmap: Entries for Alex Bennée, Deskin Miller, Vitaly "_Vi" Shukela Michael J Gruber
2010-03-20 15:15 ` Junio C Hamano
2010-03-20 18:10 ` Nicolas Pitre
2010-03-21 13:36 ` Michael J Gruber [this message]
2010-03-21 13:43 ` [PATCH] .mailmap: Map the the first submissions of MJG by e-mail Michael J Gruber
2010-03-21 19:34 ` [PATCH] .mailmap: Entries for Alex Bennée, Deskin Miller, Vitaly "_Vi" Shukela Junio C Hamano
2010-03-21 22:49 ` Alex Bennee
2010-03-20 17:47 ` Sverre Rabbelier
2010-03-22 17:15 ` Deskin Miller
2010-03-21 19:51 ` Jon Smirl
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
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4BA620CD.3060101@drmicha.warpmail.net \
--to=git@drmicha.warpmail.net \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.com \
--cc=nico@fluxnic.net \
/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.
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 NNTP newsgroup(s).