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-next>] [day] [month] [year] [list]

Date:   Fri, 16 Feb 2018 15:54:55 +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>,
        Boris Brezillon <boris.brezillon@...tlin.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

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.

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