Re: Deadname rewriting

From: CB Bailey <cb@hashpling.org>
Date: Jun 21, 2019, 22:16 UTC
Message-ID: <20190621221650.hoet5juwjhjcw5mh@hashpling.org>
In-Reply-To: <877e9e8vxt.fsf@evledraar.gmail.com>

On Fri, Jun 21, 2019 at 11:34:06PM +0200, Ævar Arnfjörð Bjarmason wrote:

Show 15 quoted lines

> >> This topic was discussed at the last git contributor summit (brought up
> >> by CB Bailey) resulting in this patch, which I see didn't make it in &
> >> needs to be resurrected again:
> >> https://public-inbox.org/git/20181212171052.13415-1-cb@hashpling.org/
> >
> > Thanks for the link.
> >
> > I didn't know about config options for mailmap.file and log.mailmap
> > before. These do make this option much more useful, especially when we
> > can insert default settings for them into /etc/gitconfig across the
> > company.
> 
> Right, and to the extent that we don't --use-mailmap by default I think
> that's mainly because nobody's cared enough to advocate for it. I think
> it would be a sensible default.

That was this patch:

https://public-inbox.org/git/20181213120940.26477-1-cb@hashpling.org/

There were no objections so I was going to re-propose it but I haven't got around to this for a number of reasons, many of which are not Git related. Ideally, I wanted to fix all of the known issues with mailmap such as some behaviors of shortlog fixed with the shortlog patch above.

I also noticed some more artifacts that I would like to be fixed. In particular the RFC 822 style "trailers" should be rewritten by default.

Having something like this pop up is not likely to be acceptable in a project which uses trailers:

commit abcd...
Author: Bob <bob@...>

    important commit message

    Signed-off-by: Alice <alice@...?

Obviously it's virtually impossible to account for everything such as someone referencing Bob by their deadname in the free text body of a historical commit.

CB