All of lore.kernel.org
 help / color / mirror / Atom feed
From: Stefan Beller <stefanbeller@googlemail.com>
To: Junio C Hamano <gitster@pobox.com>
Cc: git@vger.kernel.org
Subject: Re: [PATCH 0/6] Corrections to the mailmap file
Date: Thu, 11 Jul 2013 19:52:49 +0200	[thread overview]
Message-ID: <51DEF0F1.5010204@googlemail.com> (raw)
In-Reply-To: <7vzjttq9wk.fsf@alter.siamese.dyndns.org>

On 07/11/2013 07:33 PM, Junio C Hamano wrote:
> Stefan Beller <stefanbeller@googlemail.com> writes:
>>
>> I noticed many duplicates in email addresses but having the same name by
>> running:
>>
>>     # Finding out duplicates by comparing names:
>>     git shortlog -sne |awk '{ NF--; $1=""; print }' |sort |uniq -d
>>
>> Most of these entries are most probably the same person, but we cannot be 
>> sure, as there might be different persons having the same name, then they
>> are only distinguished by the mail address.
>>
>> However I suspect most of these to be the same person, having changed 
>> mail addresses.
>>
>> Here comes an initial batch of corrections to the mailmap file, which
>> maps people with email addresses of different capitalization onto
>> the same entity.
>> (Example Name@MIT.EDU is the same as Name@mit.edu)
>>
>> I intend to contact each of the persons individually and ask whether 
>> just their mail address changed, or if they are indeed different persons.
> 
> Has anything happen to this topic sice you posted?
> 
> I think:
> 
>  - .(none) is obviously bad, and we can fix without waiting for
>    responses as long as we know the replacement address is the
>    address from the list we usually see on the list (3).
> 
>  - Domain part is defined to be case insensitive (e.g. @mit.edu vs
>    @MIT.EDU), so both forms are equally valid.  The owner of the
>    address may have preferences (1 and 4), though.
> 
>    For this, we could just declare we downcase the domain part.
> 
>  - The local-part, even though RFC 2821 says "local-part of a
>    mailbox MUST BE treated as case sensitive", is often case
>    insensitive, and User.Name@domain and user.name@domain often name
>    the same mailbox.  The owner of the address may have preferences
>    (5 and 6), though.
> 
>    For this, we could just declare we Camel.Case the local part,
>    after making sure Camel.Case@domain has been used by the owner of
>    the address on this list.
> 
> So where does that leave us?
> 
> We can apply without waiting:
> 
>   1: downcase domain
>   3: .(none) and we know paulus@samba.org
>   4: downcase domain
>   5: We recently saw Dshco calls himself <Johannes.Schindelin@domain>.
>   6: The latest one from Nov 2009 uses <Toby.Allsopp@domain>.
> 
> The only possibly iffy one is Alex Riesen, but <raa.lkml@domain>
> seems to be the one he uses here, so I think 2. is also fine.
> 

To be honest, I did not continue to work on this. I was 
waiting for the first few patches to be reviewed, because 
I was not sure how important you all think this topic 
really is. I know projects, which frown upon such 
beautyfing commits. Hence I was waiting for an answer, 
whether such work is appreciated.

Anyway, last time I tried contributing to git, I was 
told to explicitely do it in the open. I did forget 
to CC the mailing list when asking Alex Riesen 
(2nd patch), whether it's all him.
I do see the benefit of the openess, when discussing code 
or documentation, but I wonder if you'd also like to see these
'Hello, are you the person having email x, y and z?' 
kind of mails  put on the mailing list as well. 
That would be 120 to go, which may be undesired?

So I'll start contacting the other people now.



 

  reply	other threads:[~2013-07-11 17:52 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-07-04 14:04 [PATCH 0/6] Corrections to the mailmap file Stefan Beller
2013-07-04 14:04 ` [PATCH 1/6] .mailmap: Multiple email addresses of Alejandro R. Sedeño Stefan Beller
2013-07-04 14:04 ` [PATCH 2/6] .mailmap: Multiple mail addresses of Alex Riesen Stefan Beller
2013-07-04 14:04 ` [PATCH 3/6] .mailmap: Multiple mail addresses of Paul Mackerras Stefan Beller
2013-07-04 14:04 ` [PATCH 4/6] .mailmap: Multiple mail addresses of Keith Cascio Stefan Beller
2013-07-04 14:04 ` [PATCH 5/6] .mailmap: Multiple mail addresses of Johannes Schindelin Stefan Beller
2013-07-04 14:04 ` [PATCH 6/6] .mailmap: Multiple mail addresses of Toby Allsopp Stefan Beller
2013-07-04 14:37 ` [PATCH 0/6] Corrections to the mailmap file Stefan Beller
2013-07-11 17:33 ` Junio C Hamano
2013-07-11 17:52   ` Stefan Beller [this message]
2013-07-11 18:16     ` Junio C Hamano
2013-07-12 11:49   ` [PATCH v2] " Stefan Beller
2013-07-12 11:49     ` [PATCH] .mailmap: Map email addresses to names Stefan Beller
2013-07-12 16:31       ` Junio C Hamano
2013-07-12 18:57       ` Jonathan Nieder
2013-07-12 19:02         ` Stefan Beller
2013-07-12 19:10           ` Stefan Beller
2013-07-12 16:27     ` [PATCH v2] Corrections to the mailmap file Junio C Hamano

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=51DEF0F1.5010204@googlemail.com \
    --to=stefanbeller@googlemail.com \
    --cc=git@vger.kernel.org \
    --cc=gitster@pobox.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.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.