On 1/18/20 2:27 PM, Markus Armbruster wrote:
Philippe Mathieu-Daudé <phi...@redhat.com> writes:

On 1/18/20 8:10 AM, Markus Armbruster wrote:
Philippe Mathieu-Daudé <phi...@redhat.com> 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é <phi...@redhat.com>
---
   .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 <aa...@os.amperecomputing.com>
   Alexey Gerasimenko <x19...@gmail.com>
   Alex Ivanov <v...@aleksoft.net>
   Andreas Färber <afaer...@suse.de>
+Andreas Färber <andreas.faer...@web.de>
+Andreas Färber <andreas.faer...@web.de> <andreas.faerber>
+Arei Gonglei <arei.gong...@huawei.com>
+Arei Gonglei <arei.gong...@huawei.com> <root@ceth6.(none)>

I can't find this one in git-log.

I was first surprised:

$ git log -1 3b08098b40
commit 3b08098b409c0fb28f85436ba1adeb1d401ec8f7
Author:     Gonglei <arei.gong...@huawei.com>
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 <arei.gong...@huawei.com>
     Reviewed-by: Markus Armbruster <arm...@redhat.com>

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.

OK will do, thank for the review!

[...]
   Shin'ichiro Kawasaki <kawas...@juno.dti.ne.jp>
   Shin'ichiro Kawasaki <shinichiro.kawas...@wdc.com>
+Siwei Zhuang <siwei.zhu...@data61.csiro.au>

It's actually spelled siwei.zhu...@data61.csiro.au in git-log.
Shouldn't matter.

Correct:

$ git shortlog -e 6478dd745d~..6478dd745d
Zhuang, Siwei (Data61, Kensington NSW) <siwei.zhu...@data61.csiro.au> (1):
       hw/riscv: Add optional symbol callback ptr to riscv_load_kernel()

$ git shortlog -e 6478dd745d~..6478dd745d
Siwei Zhuang <siwei.zhu...@data61.csiro.au> (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 <arm...@redhat.com>

[...]



Reply via email to