Re: [PATCH] Update Boris Brezillon email address

[Boris Brezillon]

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

> On 16.02.2018 15:54, Boris Brezillon wrote:
> > Adding back all the people that were Cc-ed on the initial email.
> > 
> > On Fri, 16 Feb 2018 15:18:21 +0100
> > Kamil Konieczny <k.konieczny@xxxxxxxxxxxxxxxxxxx> wrote:
> >   
> >> On 16.02.2018 15:00, Boris Brezillon wrote:  
> >>> On Fri, 16 Feb 2018 12:21:53 +0100
> >>> Kamil Konieczny <k.konieczny@xxxxxxxxxxxxxxxxxxx> wrote:
> >>>     
> >>>> On 16.02.2018 12:16, Boris Brezillon wrote:    
> >>>>> On Fri, 16 Feb 2018 12:07:42 +0100
> >>>>> Kamil Konieczny <k.konieczny@xxxxxxxxxxxxxxxxxxx> wrote:
> >>>>>       
> >>>>>> On 16.02.2018 11:44, Boris Brezillon wrote:      
> >>>>>>> Free Electrons is now Bootlin.
> >>>>>>>
> >>>>>>> Signed-off-by: Boris Brezillon <boris.brezillon@xxxxxxxxxxx>
> >>>>>>> ---
> >>>>>>> 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@xxxxxxxxx>
> >>>>>>>  Ben Gardner <bgardner@xxxxxxxxxx>
> >>>>>>>  Ben M Cahill <ben.m.cahill@xxxxxxxxx>
> >>>>>>>  BjÃrn Steinbrink <B.Steinbrink@xxxxxx>
> >>>>>>> -Boris Brezillon <boris.brezillon@xxxxxxxxxxxxxxxxxx>
> >>>>>>> -Boris Brezillon <boris.brezillon@xxxxxxxxxxxxxxxxxx> <b.brezillon.dev@xxxxxxxxx>
> >>>>>>> -Boris Brezillon <boris.brezillon@xxxxxxxxxxxxxxxxxx> <b.brezillon@xxxxxxxxxxx>
> >>>>>>> +Boris Brezillon <boris.brezillon@xxxxxxxxxxx>
> >>>>>>> +Boris Brezillon <boris.brezillon@xxxxxxxxxxx> <boris.brezillon@xxxxxxxxxxxxxxxxxx>        
> >>>>>>
> >>>>>> ------------------------------------------------------------------ ??????????????????      
> >>>>>
> >>>>> Is there a problem with this line?
> >>>>>       
> >>>>
> >>>> I think you want to delete all 'free-electrons', yet here you add this    
> >>>
> >>> 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
> >>>     
> >>
> >> 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 :)  
> > 
> > No, it's not private, it's part of the git tree.
> >   
> >> 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
> >>  
> > 
> > .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.  
> 
> I am newbie, it looks you are right, sorry for questions.
> 

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

Regards,

Boris

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