* [PATCH] mailmap: Remove PhilMD redirect entries
@ 2026-05-20 19:52 Philippe Mathieu-Daudé
2026-05-21 6:20 ` Manos Pitsidianakis
0 siblings, 1 reply; 4+ messages in thread
From: Philippe Mathieu-Daudé @ 2026-05-20 19:52 UTC (permalink / raw)
To: qemu-devel
Cc: Philippe Mathieu-Daudé, Markus Armbruster, Alex Bennée
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
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] mailmap: Remove PhilMD redirect entries
2026-05-20 19:52 [PATCH] mailmap: Remove PhilMD redirect entries Philippe Mathieu-Daudé
@ 2026-05-21 6:20 ` Manos Pitsidianakis
2026-05-21 6:29 ` Philippe Mathieu-Daudé
2026-05-21 7:51 ` Daniel P. Berrangé
0 siblings, 2 replies; 4+ messages in thread
From: Manos Pitsidianakis @ 2026-05-21 6:20 UTC (permalink / raw)
To: Philippe Mathieu-Daudé
Cc: qemu-devel, Markus Armbruster, Alex Bennée
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
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] mailmap: Remove PhilMD redirect entries
2026-05-21 6:20 ` Manos Pitsidianakis
@ 2026-05-21 6:29 ` Philippe Mathieu-Daudé
2026-05-21 7:51 ` Daniel P. Berrangé
1 sibling, 0 replies; 4+ messages in thread
From: Philippe Mathieu-Daudé @ 2026-05-21 6:29 UTC (permalink / raw)
To: Manos Pitsidianakis; +Cc: qemu-devel, Markus Armbruster, Alex Bennée
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
>>
>>
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] mailmap: Remove PhilMD redirect entries
2026-05-21 6:20 ` Manos Pitsidianakis
2026-05-21 6:29 ` Philippe Mathieu-Daudé
@ 2026-05-21 7:51 ` Daniel P. Berrangé
1 sibling, 0 replies; 4+ messages in thread
From: Daniel P. Berrangé @ 2026-05-21 7:51 UTC (permalink / raw)
To: Manos Pitsidianakis
Cc: Philippe Mathieu-Daudé, qemu-devel, Markus Armbruster,
Alex Bennée
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 :|
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2026-05-21 7:52 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-05-20 19:52 [PATCH] mailmap: Remove PhilMD redirect entries Philippe Mathieu-Daudé
2026-05-21 6:20 ` Manos Pitsidianakis
2026-05-21 6:29 ` Philippe Mathieu-Daudé
2026-05-21 7:51 ` Daniel P. Berrangé
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.