[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 0/3] mailmap: Clean up
From: |
Alex Bennée |
Subject: |
Re: [Qemu-devel] [PATCH 0/3] mailmap: Clean up |
Date: |
Sat, 24 Aug 2019 08:30:49 +0100 |
User-agent: |
mu4e 1.3.4; emacs 27.0.50 |
Aleksandar Markovic <address@hidden> writes:
> 23.08.2019. 08.13, "Markus Armbruster" <address@hidden> је написао/ла:
>>
>> Philippe Mathieu-Daudé <address@hidden> writes:
>>
>> > Trivial cleanup of .mailmap to have a nice 'git shortlog' output.
>> >
>> > Philippe Mathieu-Daudé (3):
>> > mailmap: Reorder by sections
>> > mailmap: Update philmd email address
>> > mailmap: Add many entries to improve 'git shortlog' statistics
>> >
>> > .mailmap | 123 +++++++++++++++++++++++++++++++++++++++++++++++++++----
>> > 1 file changed, 115 insertions(+), 8 deletions(-)
>>
>> Series
>> Reviewed-by: Markus Armbruster <address@hidden>
>>
>> However, it increases the difference to contrib/gitdm/aliases.
>
> Alex' initial gitdm effort, as I understood it, was not meant to cover all
> history from 2007 or so, but just to give reasonable statistics for 2018
> (amd future years).
>
> In that light, .mailmap and gitdm aliases do not need to be equivalent.
>
> But perhaps Alex would now want gitdm to be used for all QEMU history? Is
> this desirable?
It would be of interest historically but not something I'd want to spend
a lot of time adding code churn for.
>
> Aleksandar
>
>> I'm just
>> as guilty; my recent "[PATCH 2/2] contrib/gitdm: Add address@hidden
>> to group-map-redhat" updates only that. and not .mailmap.
>>
>> Perhaps we want to keep the two in sync manually. We should then add
>> suitable comments to each file.
>>
>> Could we instead teach gitdm to use .mailmap, and ditch
>> contrib/gitdm/aliases?
>>
>> aliases' format is documented in gitdm's README. Each line maps a
>> non-canonical e-mail address to a canonical one.
>>
>> .mailmap's format is documented in git-shortlog(1). It can do a bit
>> more. Even the common part differs: it has two addresses in different
>> order *boggle*.
>>
--
Alex Bennée