On Fri, Jun 29, 2018 at 10:42 AM Junio C Hamano <gits...@pobox.com> wrote: > > Stefan Beller <sbel...@google.com> writes: > > > There are multiple author idents who have different email addresses, but > > the same name; assume they are the same person, as the world of open source > > is actually rather small. > > Thanks for an interesting experiment. As with 2/3, I suspect that > most of the contents in the patch may be correct, but I'd rather see > these confirmed by those whose names appear in the patch. > > IOW, I would not feel comfortable applying a patch, unless it looks > like this (just taking a single person as a random example), > and I do not mind many such individual patches:
right, I did not quite expect this patch to be applied, though it looked like such a sweet shortcut. I might just email all of them and ask for them to update their .mailmap entries. Thanks, Stefan > Subject: .mailmap: unify the same Ben Peart cc'd Ben, so we could start here with a patch. :) > These multiple author identities in our history are actually the > same person. Map them to the latest address. > > Signed-off-by: Stefan Beller <sbel...@google.com> > Acked-by: Ben Peart <ben.pe...@microsoft.com> > --- > > diff --git a/.mailmap b/.mailmap > index ff96ef7401f..2607846582a 100644 > --- a/.mailmap > +++ b/.mailmap > @@ -5,54 +5,86 @@ > Amos Waterland <a...@debian.org> <a...@us.ibm.com> > Ben Walton <bdwal...@gmail.com> <bwal...@artsci.utoronto.ca> > Benoit Sigoure <tsuna...@gmail.com> <ts...@lrde.epita.fr> > +Ben Peart <ben.pe...@microsoft.com> Ben Peart <benpe...@microsoft.com> > +Ben Peart <ben.pe...@microsoft.com> Ben Peart <peart...@gmail.com> > Bernt Hansen <be...@norang.ca> <be...@alumni.uwaterloo.ca> > Brandon Casey <draf...@gmail.com> <ca...@nrlssc.navy.mil> > > It may be even better if we can arraange the author of the patch to > be the one who is involved, with "Helped-by: Stefan". ok, I'll think how I can help but not write the code. Thanks, Stefan