Re: [PATCH] mailmap: consistently normalize brian m. carlson's name

From: brian m. carlson <sandals@crustytoothpaste.net>
Date: Sep 17, 2018, 22:21 UTC
Message-ID: <20180917222158.GY432229@genre.crustytoothpaste.net>
In-Reply-To: <20180917181800.GA140909@aiede.svl.corp.google.com>

On Mon, Sep 17, 2018 at 11:18:00AM -0700, Jonathan Nieder wrote:

Show 23 quoted lines

> v2.18.0-rc0~70^2 (mailmap: update brian m. carlson's email address,
> 2018-05-08) changed the mailmap to map
> 
>   sandals@crustytoothpaste.ath.cx
>    -> brian m. carlson <sandals@crustytoothpaste.net>
> 
> instead of
> 
>   sandals@crustytoothpaste.net
>     -> brian m. carlson <sandals@crustytoothpaste.ath.cx>
> 
> That means the mapping
> 
>   Brian M. Carlson <sandals@crustytoothpaste.ath.cx>
>     -> brian m. carlson <sandals@crustytoothpaste.net>
> 
> is redundant, so we can remove it.  More importantly, it means that
> the identity "Brian M. Carlson <sandals@crustytoothpaste.net>" used in
> some commits is not normalized any more.  Add a mapping for it.
> 
> Noticed while updating Debian's Git packaging, which uses "git
> shortlog --no-merges" to produce a list of changes in each version,
> grouped by author's (normalized) name.

I think this commit message makes sense. I apparently still fail to understand how the .mailmap format works, so I can't tell you if the patch is correct.

-- 
brian m. carlson: Houston, Texas, US
OpenPGP: https://keybase.io/bk2204