Re: [PATCH] Update Boris Brezillon email address

[Boris Brezillon <boris.brezillon@bootlin.com>]

On Fri, 16 Feb 2018 16:35:27 +0100
Kamil Konieczny <k.konieczny@partner.samsung.com> wrote:

> On 16.02.2018 15:54, Boris Brezillon wrote:
> > Adding back all the people that were Cc-ed on the initial email.
> >=20
> > On Fri, 16 Feb 2018 15:18:21 +0100
> > Kamil Konieczny <k.konieczny@partner.samsung.com> wrote:
> >  =20
> >> On 16.02.2018 15:00, Boris Brezillon wrote: =20
> >>> On Fri, 16 Feb 2018 12:21:53 +0100
> >>> Kamil Konieczny <k.konieczny@partner.samsung.com> wrote:
> >>>    =20
> >>>> On 16.02.2018 12:16, Boris Brezillon wrote:   =20
> >>>>> On Fri, 16 Feb 2018 12:07:42 +0100
> >>>>> Kamil Konieczny <k.konieczny@partner.samsung.com> wrote:
> >>>>>      =20
> >>>>>> On 16.02.2018 11:44, Boris Brezillon wrote:     =20
> >>>>>>> Free Electrons is now Bootlin.
> >>>>>>>
> >>>>>>> Signed-off-by: Boris Brezillon <boris.brezillon@bootlin.com>
> >>>>>>> ---
> >>>>>>> Note that I'm planning to take this patch through the MTD tree.
> >>>>>>> ---
> >>>>>>>  .mailmap    |  7 ++++---
> >>>>>>>  MAINTAINERS | 10 +++++-----
> >>>>>>>  2 files changed, 9 insertions(+), 8 deletions(-)
> >>>>>>>
> >>>>>>> diff --git a/.mailmap b/.mailmap
> >>>>>>> index e18cab73e209..50c1d6bf36b2 100644
> >>>>>>> --- a/.mailmap
> >>>>>>> +++ b/.mailmap
> >>>>>>> @@ -33,9 +33,10 @@ Axel Lin <axel.lin@gmail.com>
> >>>>>>>  Ben Gardner <bgardner@wabtec.com>
> >>>>>>>  Ben M Cahill <ben.m.cahill@intel.com>
> >>>>>>>  Bj=C3=B6rn Steinbrink <B.Steinbrink@gmx.de>
> >>>>>>> -Boris Brezillon <boris.brezillon@free-electrons.com>
> >>>>>>> -Boris Brezillon <boris.brezillon@free-electrons.com> <b.brezillo=
n.dev@gmail.com>
> >>>>>>> -Boris Brezillon <boris.brezillon@free-electrons.com> <b.brezillo=
n@overkiz.com>
> >>>>>>> +Boris Brezillon <boris.brezillon@bootlin.com>
> >>>>>>> +Boris Brezillon <boris.brezillon@bootlin.com> <boris.brezillon@f=
ree-electrons.com>       =20
> >>>>>>
> >>>>>> ------------------------------------------------------------------=
 ??????????????????     =20
> >>>>>
> >>>>> Is there a problem with this line?
> >>>>>      =20
> >>>>
> >>>> I think you want to delete all 'free-electrons', yet here you add th=
is   =20
> >>>
> >>> I'm clearly not a git expert, but according to man git-shortlog [1], =
the
> >>> format I'm using here allows mailmap to replace the old email by the
> >>> new one, which is exactly what I want.
> >>>
> >>> [1]https://git-scm.com/docs/git-shortlog
> >>>    =20
> >>
> >> Ah, I see, but why do you send patch for .mailmap ?
> >> it is your private file, not global one, I do not want mine .mailmap
> >> to be replaced by yours :) =20
> >=20
> > No, it's not private, it's part of the git tree.
> >  =20
> >> But beside this, you can use sed script
> >>
> >> sed -i -e 's/free-electrons/bootlin/g' .mailmap
> >>
> >> Please just resend this with patch for MAINTAINER's only
> >> =20
> >=20
> > .mailmap is versioned for a good reason, and I don't see why I couldn't
> > update entries for my emails, especially since those entries already
> > existed before this patch. =20
>=20
> I am newbie, it looks you are right, sorry for questions.
>=20

No problem, but when you're not sure about something you should not act
like you know how things should be done.

Regards,

Boris

--=20
Boris Brezillon, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
http://bootlin.com