Re: [PATCH] Update Boris Brezillon email address

From: Kamil Konieczny
Date: Fri Feb 16 2018 - 10:35:39 EST


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.

--
Best regards,
Kamil Konieczny
Samsung R&D Institute Poland