1. Patchew
  2. QEMU
  3. View series

[PATCH] mailmap: Remove PhilMD redirect entries

Philippe Mathieu-Daudé posted 1 patch 1 week, 2 days ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20260520195206.40679-1-philmd@linaro.org
.mailmap | 3 ---
1 file changed, 3 deletions(-)
[PATCH] mailmap: Remove PhilMD redirect entries
Posted by Philippe Mathieu-Daudé 1 week, 2 days ago 
Our ./scripts/get_maintainer.pl uses git .mailmap as a fallback:

  $ ./scripts/get_maintainer.pl -h
  usage: ./scripts/get_maintainer.pl [options] patchfile
         ./scripts/get_maintainer.pl [options] -f file|directory

    --git-fallback => use git when no exact MAINTAINERS pattern (default: 1)
    --git-since => git history to use (default: 1-year-ago)
    --mailmap => use .mailmap file (default: 1)
    ...

  Default options:
    [--email --nogit --git-fallback --m --r --n --l --multiline
     --pattern-depth=0 --remove-duplicates --rolestats]

(see the '--git-fallback' used as default option).

Per gitmailmap(5) man page:

  ... the file .mailmap [...] is used to map author and
  committer names and email addresses to canonical real
  names and email addresses.

I added my entries in the "replace old addresses by a more recent
one" section for this script use case, but we have been doing a
good cleanup at having all files covered in MAINTAINERS, so with
retrospective these entries aren't needed anymore (besides, using
them retroactively credits all my contributions to the latest
employer).

Simply remove my mappings of that "old -> recent" section.

Reported-by: Markus Armbruster <armbru@redhat.com>
Suggested-by: Alex Bennée <alex.bennee@linaro.org>
Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
---
 .mailmap | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/.mailmap b/.mailmap
index ce569d83f4a..82ff20b8ddb 100644
--- a/.mailmap
+++ b/.mailmap
@@ -104,9 +104,6 @@ Paul Burton <paulburton@kernel.org> <paul.burton@mips.com>
 Paul Burton <paulburton@kernel.org> <paul.burton@imgtec.com>
 Paul Burton <paulburton@kernel.org> <paul@archlinuxmips.org>
 Paul Burton <paulburton@kernel.org> <pburton@wavecomp.com>
-Philippe Mathieu-Daudé <philmd@linaro.org> <f4bug@amsat.org>
-Philippe Mathieu-Daudé <philmd@linaro.org> <philmd@redhat.com>
-Philippe Mathieu-Daudé <philmd@linaro.org> <philmd@fungible.com>
 Pierrick Bouvier <pierrick.bouvier@oss.qualcomm.com> <pierrick.bouvier@linaro.org>
 Roman Bolshakov <rbolshakov@ddn.com> <r.bolshakov@yadro.com>
 Sriram Yagnaraman <sriram.yagnaraman@ericsson.com> <sriram.yagnaraman@est.tech>
-- 
2.53.0
Re: [PATCH] mailmap: Remove PhilMD redirect entries
Posted by Manos Pitsidianakis 1 week, 2 days ago 
On Wed, May 20, 2026 at 10:52 PM Philippe Mathieu-Daudé
<philmd@linaro.org> wrote:
>
> Our ./scripts/get_maintainer.pl uses git .mailmap as a fallback:
>
>   $ ./scripts/get_maintainer.pl -h
>   usage: ./scripts/get_maintainer.pl [options] patchfile
>          ./scripts/get_maintainer.pl [options] -f file|directory
>
>     --git-fallback => use git when no exact MAINTAINERS pattern (default: 1)
>     --git-since => git history to use (default: 1-year-ago)
>     --mailmap => use .mailmap file (default: 1)
>     ...
>
>   Default options:
>     [--email --nogit --git-fallback --m --r --n --l --multiline
>      --pattern-depth=0 --remove-duplicates --rolestats]
>
> (see the '--git-fallback' used as default option).
>
> Per gitmailmap(5) man page:
>
>   ... the file .mailmap [...] is used to map author and
>   committer names and email addresses to canonical real
>   names and email addresses.
>
> I added my entries in the "replace old addresses by a more recent
> one" section for this script use case, but we have been doing a
> good cleanup at having all files covered in MAINTAINERS, so with
> retrospective these entries aren't needed anymore (besides, using
> them retroactively credits all my contributions to the latest
> employer).
>


Hey Philippe,

As I posted here
https://lore.kernel.org/qemu-devel/3104a00e-4c13-4bf7-95a0-1f2840bec50d@linaro.org/T/#t

> People who wish to use git log for employer attribution reasons must
> use --no-mailmap.

mailmap is for updating names/emails, not attribution. I do not think
your patch is necessary.

> Simply remove my mappings of that "old -> recent" section.
>
> Reported-by: Markus Armbruster <armbru@redhat.com>
> Suggested-by: Alex Bennée <alex.bennee@linaro.org>
> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> ---
>  .mailmap | 3 ---
>  1 file changed, 3 deletions(-)
>
> diff --git a/.mailmap b/.mailmap
> index ce569d83f4a..82ff20b8ddb 100644
> --- a/.mailmap
> +++ b/.mailmap
> @@ -104,9 +104,6 @@ Paul Burton <paulburton@kernel.org> <paul.burton@mips.com>
>  Paul Burton <paulburton@kernel.org> <paul.burton@imgtec.com>
>  Paul Burton <paulburton@kernel.org> <paul@archlinuxmips.org>
>  Paul Burton <paulburton@kernel.org> <pburton@wavecomp.com>
> -Philippe Mathieu-Daudé <philmd@linaro.org> <f4bug@amsat.org>
> -Philippe Mathieu-Daudé <philmd@linaro.org> <philmd@redhat.com>
> -Philippe Mathieu-Daudé <philmd@linaro.org> <philmd@fungible.com>
>  Pierrick Bouvier <pierrick.bouvier@oss.qualcomm.com> <pierrick.bouvier@linaro.org>
>  Roman Bolshakov <rbolshakov@ddn.com> <r.bolshakov@yadro.com>
>  Sriram Yagnaraman <sriram.yagnaraman@ericsson.com> <sriram.yagnaraman@est.tech>
> --
> 2.53.0
>
>

-- 
Manos Pitsidianakis
Emulation and Virtualization Engineer at Linaro Ltd
Re: [PATCH] mailmap: Remove PhilMD redirect entries
Posted by Daniel P. Berrangé 1 week, 2 days ago 
On Thu, May 21, 2026 at 09:20:49AM +0300, Manos Pitsidianakis wrote:
> On Wed, May 20, 2026 at 10:52 PM Philippe Mathieu-Daudé
> <philmd@linaro.org> wrote:
> >
> > Our ./scripts/get_maintainer.pl uses git .mailmap as a fallback:
> >
> >   $ ./scripts/get_maintainer.pl -h
> >   usage: ./scripts/get_maintainer.pl [options] patchfile
> >          ./scripts/get_maintainer.pl [options] -f file|directory
> >
> >     --git-fallback => use git when no exact MAINTAINERS pattern (default: 1)
> >     --git-since => git history to use (default: 1-year-ago)
> >     --mailmap => use .mailmap file (default: 1)
> >     ...
> >
> >   Default options:
> >     [--email --nogit --git-fallback --m --r --n --l --multiline
> >      --pattern-depth=0 --remove-duplicates --rolestats]
> >
> > (see the '--git-fallback' used as default option).
> >
> > Per gitmailmap(5) man page:
> >
> >   ... the file .mailmap [...] is used to map author and
> >   committer names and email addresses to canonical real
> >   names and email addresses.
> >
> > I added my entries in the "replace old addresses by a more recent
> > one" section for this script use case, but we have been doing a
> > good cleanup at having all files covered in MAINTAINERS, so with
> > retrospective these entries aren't needed anymore (besides, using
> > them retroactively credits all my contributions to the latest
> > employer).
> >
> 
> 
> Hey Philippe,
> 
> As I posted here
> https://lore.kernel.org/qemu-devel/3104a00e-4c13-4bf7-95a0-1f2840bec50d@linaro.org/T/#t
> 
> > People who wish to use git log for employer attribution reasons must
> > use --no-mailmap.
> 
> mailmap is for updating names/emails, not attribution. I do not think
> your patch is necessary.

Attribution of committers to employers is something that is very
much a loose heuristic that only works for relatively recent commits.

Beyond .mailmap, the data we have in contrib/gitdm only ever reflects
our current point-in-time understanding of employers. For cases where
we map personal email addresses to employers, we are not keeping a
historical record as people change jobs, so all historical commits
from a personal email address get associated with the newest employer.

IOW if you want to attribute accurately, you need to look at the state
of contrib/gitdm & .mailmap, as they were when the commit you want to
attribute was merged.


With regards,
Daniel
-- 
|: https://berrange.com       ~~        https://hachyderm.io/@berrange :|
|: https://libvirt.org          ~~          https://entangle-photo.org :|
|: https://pixelfed.art/berrange   ~~    https://fstop138.berrange.com :|
Re: [PATCH] mailmap: Remove PhilMD redirect entries
Posted by Philippe Mathieu-Daudé 1 week, 2 days ago 
On 21/5/26 08:20, Manos Pitsidianakis wrote:
> On Wed, May 20, 2026 at 10:52 PM Philippe Mathieu-Daudé
> <philmd@linaro.org> wrote:
>>
>> Our ./scripts/get_maintainer.pl uses git .mailmap as a fallback:
>>
>>    $ ./scripts/get_maintainer.pl -h
>>    usage: ./scripts/get_maintainer.pl [options] patchfile
>>           ./scripts/get_maintainer.pl [options] -f file|directory
>>
>>      --git-fallback => use git when no exact MAINTAINERS pattern (default: 1)
>>      --git-since => git history to use (default: 1-year-ago)
>>      --mailmap => use .mailmap file (default: 1)
>>      ...
>>
>>    Default options:
>>      [--email --nogit --git-fallback --m --r --n --l --multiline
>>       --pattern-depth=0 --remove-duplicates --rolestats]
>>
>> (see the '--git-fallback' used as default option).
>>
>> Per gitmailmap(5) man page:
>>
>>    ... the file .mailmap [...] is used to map author and
>>    committer names and email addresses to canonical real
>>    names and email addresses.
>>
>> I added my entries in the "replace old addresses by a more recent
>> one" section for this script use case, but we have been doing a
>> good cleanup at having all files covered in MAINTAINERS, so with
>> retrospective these entries aren't needed anymore (besides, using
>> them retroactively credits all my contributions to the latest
>> employer).
>>
> 
> 
> Hey Philippe,
> 
> As I posted here
> https://lore.kernel.org/qemu-devel/3104a00e-4c13-4bf7-95a0-1f2840bec50d@linaro.org/T/#t
> 
>> People who wish to use git log for employer attribution reasons must
>> use --no-mailmap.
> 
> mailmap is for updating names/emails, not attribution. I do not think
> your patch is necessary.

Great, thank you for mentioning the --no-mailmap option.

Markus, Alex, does that option work for you?

> 
>> Simply remove my mappings of that "old -> recent" section.
>>
>> Reported-by: Markus Armbruster <armbru@redhat.com>
>> Suggested-by: Alex Bennée <alex.bennee@linaro.org>
>> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
>> ---
>>   .mailmap | 3 ---
>>   1 file changed, 3 deletions(-)
>>
>> diff --git a/.mailmap b/.mailmap
>> index ce569d83f4a..82ff20b8ddb 100644
>> --- a/.mailmap
>> +++ b/.mailmap
>> @@ -104,9 +104,6 @@ Paul Burton <paulburton@kernel.org> <paul.burton@mips.com>
>>   Paul Burton <paulburton@kernel.org> <paul.burton@imgtec.com>
>>   Paul Burton <paulburton@kernel.org> <paul@archlinuxmips.org>
>>   Paul Burton <paulburton@kernel.org> <pburton@wavecomp.com>
>> -Philippe Mathieu-Daudé <philmd@linaro.org> <f4bug@amsat.org>
>> -Philippe Mathieu-Daudé <philmd@linaro.org> <philmd@redhat.com>
>> -Philippe Mathieu-Daudé <philmd@linaro.org> <philmd@fungible.com>
>>   Pierrick Bouvier <pierrick.bouvier@oss.qualcomm.com> <pierrick.bouvier@linaro.org>
>>   Roman Bolshakov <rbolshakov@ddn.com> <r.bolshakov@yadro.com>
>>   Sriram Yagnaraman <sriram.yagnaraman@ericsson.com> <sriram.yagnaraman@est.tech>
>> --
>> 2.53.0
>>
>>
>

Patchew 2.1-devel-ea86937

© 2016 - 2026 Red Hat, Inc.