Time to add a Git .mailmap?

Started by Daniel Gustafssonabout 1 year ago11 messages
#1Daniel Gustafsson
daniel@yesql.se
1 attachment(s)

When looking at our Git tree for a recent conference presentation I happened to
notice that we have recently gained duplicate names in the shortlog. Not sure
if we care enough to fix that with a .mailmap, but if we do the attached diff
makes sure that all commits are accounted for a single committer entry.

--
Daniel Gustafsson

Attachments:

mailmap.diffapplication/octet-stream; name=mailmap.diff; x-unix-mode=0644Download
diff --git a/.mailmap b/.mailmap
new file mode 100644
index 0000000000..22d95d3484
--- /dev/null
+++ b/.mailmap
@@ -0,0 +1 @@
+Álvaro Herrera <alvherre@alvh.no-ip.org>
#2Nathan Bossart
nathandbossart@gmail.com
In reply to: Daniel Gustafsson (#1)
Re: Time to add a Git .mailmap?

On Thu, Oct 31, 2024 at 11:37:13AM +0100, Daniel Gustafsson wrote:

When looking at our Git tree for a recent conference presentation I happened to
notice that we have recently gained duplicate names in the shortlog. Not sure
if we care enough to fix that with a .mailmap, but if we do the attached diff
makes sure that all commits are accounted for a single committer entry.

Seems reasonable to me.

--
nathan

#3Michael Paquier
michael@paquier.xyz
In reply to: Nathan Bossart (#2)
Re: Time to add a Git .mailmap?

On Thu, Oct 31, 2024 at 02:45:02PM -0500, Nathan Bossart wrote:

On Thu, Oct 31, 2024 at 11:37:13AM +0100, Daniel Gustafsson wrote:

When looking at our Git tree for a recent conference presentation I happened to
notice that we have recently gained duplicate names in the shortlog. Not sure
if we care enough to fix that with a .mailmap, but if we do the attached diff
makes sure that all commits are accounted for a single committer entry.

Seems reasonable to me.

+1.
--
Michael
#4Alvaro Herrera
alvherre@alvh.no-ip.org
In reply to: Daniel Gustafsson (#1)
Re: Time to add a Git .mailmap?

On 2024-Oct-31, Daniel Gustafsson wrote:

When looking at our Git tree for a recent conference presentation I happened to
notice that we have recently gained duplicate names in the shortlog. Not sure
if we care enough to fix that with a .mailmap, but if we do the attached diff
makes sure that all commits are accounted for a single committer entry.

LGTM. I'd also add this line while at it:

Peter Eisentraut <peter@eisentraut.org> <peter_e@gmx.net>

This takes care of all the duplicate "identities" in the history AFAICT.

--
Álvaro Herrera Breisgau, Deutschland — https://www.EnterpriseDB.com/
"No me acuerdo, pero no es cierto. No es cierto, y si fuera cierto,
no me acuerdo." (Augusto Pinochet a una corte de justicia)

#5Peter Eisentraut
peter@eisentraut.org
In reply to: Alvaro Herrera (#4)
Re: Time to add a Git .mailmap?

On 01.11.24 12:53, Alvaro Herrera wrote:

On 2024-Oct-31, Daniel Gustafsson wrote:

When looking at our Git tree for a recent conference presentation I happened to
notice that we have recently gained duplicate names in the shortlog. Not sure
if we care enough to fix that with a .mailmap, but if we do the attached diff
makes sure that all commits are accounted for a single committer entry.

LGTM. I'd also add this line while at it:

Peter Eisentraut <peter@eisentraut.org> <peter_e@gmx.net>

This takes care of all the duplicate "identities" in the history AFAICT.

I'm not sure if this is a good use of the mailmap feature. If someone
commits under <peter@companyfoo.com> for a while and then later as
<peter@companybar.com>, and the mailmap maps everything to the most
recent one, that seems kind of misleading or unfair? The examples on
the gitmailmap man page all indicate that this feature is to correct
accidental variations or obvious mistakes, but not to unify everything
to the extent that it alters the historical record.

#6Daniel Gustafsson
daniel@yesql.se
In reply to: Peter Eisentraut (#5)
Re: Time to add a Git .mailmap?

On 1 Nov 2024, at 13:53, Peter Eisentraut <peter@eisentraut.org> wrote:

On 01.11.24 12:53, Alvaro Herrera wrote:

On 2024-Oct-31, Daniel Gustafsson wrote:

When looking at our Git tree for a recent conference presentation I happened to
notice that we have recently gained duplicate names in the shortlog. Not sure
if we care enough to fix that with a .mailmap, but if we do the attached diff
makes sure that all commits are accounted for a single committer entry.

LGTM. I'd also add this line while at it:
Peter Eisentraut <peter@eisentraut.org> <peter_e@gmx.net>
This takes care of all the duplicate "identities" in the history AFAICT.

I'm not sure if this is a good use of the mailmap feature. If someone commits under <peter@companyfoo.com> for a while and then later as <peter@companybar.com>, and the mailmap maps everything to the most recent one, that seems kind of misleading or unfair? The examples on the gitmailmap man page all indicate that this feature is to correct accidental variations or obvious mistakes, but not to unify everything to the extent that it alters the historical record.

I agree with this and propose to leave it at the originally proposed mailmap
contents.

--
Daniel Gustafsson

#7Alvaro Herrera
alvherre@alvh.no-ip.org
In reply to: Peter Eisentraut (#5)
Re: Time to add a Git .mailmap?

On 2024-Nov-01, Peter Eisentraut wrote:

LGTM. I'd also add this line while at it:

Peter Eisentraut <peter@eisentraut.org> <peter_e@gmx.net>

This takes care of all the duplicate "identities" in the history AFAICT.

I'm not sure if this is a good use of the mailmap feature. If someone
commits under <peter@companyfoo.com> for a while and then later as
<peter@companybar.com>, and the mailmap maps everything to the most recent
one, that seems kind of misleading or unfair?

While I would agree with this line of thinking if the situation were as
you describe, it should be obvious that it isn't; nobody here uses or
has ever used a work email as committer address[1]AFAIK gmx.net is a ISP-supplied address, not a work address.[2]scrappy@hub.org and simon@2ndQuadrant.com might be considered work addresses, but they aren't really. Nevertheless,
since this argument is about _your_ personal identity not mine, I'm not
going to stand against you on it.

Therefore I +1 Daniel's original proposal with thanks, and BTW I'm not
sorry for changing my name to use the hard-won ' accent on it :-)

The examples on the gitmailmap man page all indicate that this feature
is to correct accidental variations or obvious mistakes, but not to
unify everything to the extent that it alters the historical record.

I don't think these examples are normative. There's plenty of evidence
that people look for ways to attribute contributions to individuals
rather than email-based identities. See for example

https://gitlab.com/gitlab-org/gitlab/-/issues/14909
https://github.com/microsoft/vscode/blob/main/.mailmap
https://github.com/gradle/gradle/blob/master/.mailmap

[1]: AFAIK gmx.net is a ISP-supplied address, not a work address.
[2]: scrappy@hub.org and simon@2ndQuadrant.com might be considered work addresses, but they aren't really
considered work addresses, but they aren't really

--
Álvaro Herrera PostgreSQL Developer — https://www.EnterpriseDB.com/
"E pur si muove" (Galileo Galilei)

#8Daniel Gustafsson
daniel@yesql.se
In reply to: Alvaro Herrera (#7)
Re: Time to add a Git .mailmap?

On 5 Nov 2024, at 10:33, Alvaro Herrera <alvherre@alvh.no-ip.org> wrote:

Therefore I +1 Daniel's original proposal with thanks, and BTW I'm not
sorry for changing my name to use the hard-won ' accent on it :-)

Done.

--
Daniel Gustafsson

#9Michael Paquier
michael@paquier.xyz
In reply to: Alvaro Herrera (#7)
Re: Time to add a Git .mailmap?

On Tue, Nov 05, 2024 at 10:33:13AM +0100, Alvaro Herrera wrote:

While I would agree with this line of thinking if the situation were as
you describe, it should be obvious that it isn't; nobody here uses or
has ever used a work email as committer address[1][2]. Nevertheless,
since this argument is about _your_ personal identity not mine, I'm not
going to stand against you on it.

Therefore I +1 Daniel's original proposal with thanks, and BTW I'm not
sorry for changing my name to use the hard-won ' accent on it :-)

And now I'm going to ask how you figured out about the ë in my name,
because it's right ;)

I've dropped it from the commit logs because of keyboard laziness,
mostly.
--
Michael

#10Alvaro Herrera
alvherre@alvh.no-ip.org
In reply to: Michael Paquier (#9)
Re: Time to add a Git .mailmap?

On 2024-Nov-06, Michael Paquier wrote:

And now I'm going to ask how you figured out about the ë in my name,
because it's right ;)

Hah, I got it from Fabien Coelho,
/messages/by-id/alpine.DEB.2.10.1512240729160.17411@sto
and then searched around to confirm that it was correct.

Funnily enough, I'm the only committer that has ever used "Michaël", but
I'm not the only one to have used the mistaken "Paquiër". Go figure.

--
Álvaro Herrera PostgreSQL Developer — https://www.EnterpriseDB.com/
"That sort of implies that there are Emacs keystrokes which aren't obscure.
I've been using it daily for 2 years now and have yet to discover any key
sequence which makes any sense." (Paul Thomas)

#11Michael Paquier
michael@paquier.xyz
In reply to: Alvaro Herrera (#10)
Re: Time to add a Git .mailmap?

On Wed, Nov 06, 2024 at 11:36:42AM +0100, Alvaro Herrera wrote:

Funnily enough, I'm the only committer that has ever used "Michaël", but
I'm not the only one to have used the mistaken "Paquiër". Go figure.

I haven't noticed this one. That's new :DD
--
Michael