linux-kernel - Re: [PATCH] Update Boris Brezillon email address

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:   Fri, 16 Feb 2018 16:51:47 +0100
From:   Boris Brezillon <boris.brezillon@...tlin.com>
To:     Kamil Konieczny <k.konieczny@...tner.samsung.com>
Cc:     David Woodhouse <dwmw2@...radead.org>,
        Brian Norris <computersforpeace@...il.com>,
        Marek Vasut <marek.vasut@...il.com>,
        Richard Weinberger <richard@....at>,
        Cyrille Pitchen <cyrille.pitchen@...ev4u.fr>,
        linux-mtd@...ts.infradead.org,
        Herbert Xu <herbert@...dor.apana.org.au>,
        "David S. Miller" <davem@...emloft.net>,
        linux-crypto@...r.kernel.org,
        Mike Turquette <mturquette@...libre.com>,
        Stephen Boyd <sboyd@...eaurora.org>, linux-clk@...r.kernel.org,
        David Airlie <airlied@...ux.ie>,
        Daniel Vetter <daniel@...ll.ch>,
        dri-devel@...ts.freedesktop.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] Update Boris Brezillon email address

On Fri, 16 Feb 2018 16:35:27 +0100
Kamil Konieczny <k.konieczny@...tner.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.
> > 
> > On Fri, 16 Feb 2018 15:18:21 +0100
> > Kamil Konieczny <k.konieczny@...tner.samsung.com> wrote:
> >   
> >> On 16.02.2018 15:00, Boris Brezillon wrote:  
> >>> On Fri, 16 Feb 2018 12:21:53 +0100
> >>> Kamil Konieczny <k.konieczny@...tner.samsung.com> wrote:
> >>>     
> >>>> On 16.02.2018 12:16, Boris Brezillon wrote:    
> >>>>> On Fri, 16 Feb 2018 12:07:42 +0100
> >>>>> Kamil Konieczny <k.konieczny@...tner.samsung.com> wrote:
> >>>>>       
> >>>>>> On 16.02.2018 11:44, Boris Brezillon wrote:      
> >>>>>>> Free Electrons is now Bootlin.
> >>>>>>>
> >>>>>>> Signed-off-by: Boris Brezillon <boris.brezillon@...tlin.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@...il.com>
> >>>>>>>  Ben Gardner <bgardner@...tec.com>
> >>>>>>>  Ben M Cahill <ben.m.cahill@...el.com>
> >>>>>>>  Björn Steinbrink <B.Steinbrink@....de>
> >>>>>>> -Boris Brezillon <boris.brezillon@...e-electrons.com>
> >>>>>>> -Boris Brezillon <boris.brezillon@...e-electrons.com> <b.brezillon.dev@...il.com>
> >>>>>>> -Boris Brezillon <boris.brezillon@...e-electrons.com> <b.brezillon@...rkiz.com>
> >>>>>>> +Boris Brezillon <boris.brezillon@...tlin.com>
> >>>>>>> +Boris Brezillon <boris.brezillon@...tlin.com> <boris.brezillon@...e-electrons.com>        
> >>>>>>
> >>>>>> ------------------------------------------------------------------ ??????????????????      
> >>>>>
> >>>>> 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