[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] mailmap: Add more entries to sanitize 'git log' output
From: |
Markus Armbruster |
Subject: |
Re: [PATCH] mailmap: Add more entries to sanitize 'git log' output |
Date: |
Sat, 18 Jan 2020 14:27:32 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.3 (gnu/linux) |
Philippe Mathieu-Daudé <address@hidden> writes:
> On 1/18/20 8:10 AM, Markus Armbruster wrote:
>> Philippe Mathieu-Daudé <address@hidden> writes:
>>
>>> Most of these developers have the Signed-off-by tag properly
>>> written, but not the author/commiter name. Fix this.
>>> Also we incorrectly wrote Arei Gonglei name, update and reorder.
>>>
>>> git-log does not use this file by default until you specify the
>>> --use-mailmap flag:
>>>
>>> $ git log --use-mailmap
>>>
>>> Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
>>> ---
>>> .mailmap | 29 ++++++++++++++++++++++++++++-
>>> 1 file changed, 28 insertions(+), 1 deletion(-)
>>>
>>> diff --git a/.mailmap b/.mailmap
>>> index 3816e4effe..1a859d9e65 100644
>>> --- a/.mailmap
>>> +++ b/.mailmap
>>> @@ -56,6 +56,10 @@ Aaron Lindsay <address@hidden>
>>> Alexey Gerasimenko <address@hidden>
>>> Alex Ivanov <address@hidden>
>>> Andreas Färber <address@hidden>
>>> +Andreas Färber <address@hidden>
>>> +Andreas Färber <address@hidden> <andreas.faerber>
>>> +Arei Gonglei <address@hidden>
>>> +Arei Gonglei <address@hidden> <root@ceth6.(none)>
>>
>> I can't find this one in git-log.
>
> I was first surprised:
>
> $ git log -1 3b08098b40
> commit 3b08098b409c0fb28f85436ba1adeb1d401ec8f7
> Author: Gonglei <address@hidden>
> AuthorDate: Wed Dec 3 18:25:46 2014 +0000
> Commit: root <root@ceth6.(none)>
> CommitDate: Mon Dec 22 14:39:21 2014 +0800
>
> bootdevice: add validate check for qemu_boot_set()
>
> Signed-off-by: Gonglei <address@hidden>
> Reviewed-by: Markus Armbruster <address@hidden>
>
> After thinking for a bit I remembered I once changed my default format:
>
> $ git config format.pretty
> fuller
Aha! Could've thought of this myself...
> I suppose I should add this in the commit description.
Can't hurt.
[...]
>>> Shin'ichiro Kawasaki <address@hidden>
>>> Shin'ichiro Kawasaki <address@hidden>
>>> +Siwei Zhuang <address@hidden>
>>
>> It's actually spelled address@hidden in git-log.
>> Shouldn't matter.
>
> Correct:
>
> $ git shortlog -e 6478dd745d~..6478dd745d
> Zhuang, Siwei (Data61, Kensington NSW) <address@hidden> (1):
> hw/riscv: Add optional symbol callback ptr to riscv_load_kernel()
>
> $ git shortlog -e 6478dd745d~..6478dd745d
> Siwei Zhuang <address@hidden> (1):
> hw/riscv: Add optional symbol callback ptr to riscv_load_kernel()
Suggest to adjust case if you respin.
> Thanks for caring checking all entries!
Reviewed-by: Markus Armbruster <address@hidden>
[...]