2019-08-05 12:16:55

by Andrea Parri

[permalink] [raw]
Subject: [PATCH] MAINTAINERS: Update e-mail address for Andrea Parri

My @amarulasolutions.com address stopped working this July, so update
to my @gmail.com address where you'll still be able to reach me.

Signed-off-by: Andrea Parri <[email protected]>
Cc: Alan Stern <[email protected]>
Cc: Will Deacon <[email protected]>
Cc: Peter Zijlstra <[email protected]>
Cc: Boqun Feng <[email protected]>
Cc: Nicholas Piggin <[email protected]>
Cc: David Howells <[email protected]>
Cc: Jade Alglave <[email protected]>
Cc: Luc Maranget <[email protected]>
Cc: "Paul E. McKenney" <[email protected]>
Cc: Akira Yokosawa <[email protected]>
Cc: Daniel Lustig <[email protected]>
---
MAINTAINERS | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index 6426db5198f05..527317026492f 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -9326,7 +9326,7 @@ F: drivers/misc/lkdtm/*

LINUX KERNEL MEMORY CONSISTENCY MODEL (LKMM)
M: Alan Stern <[email protected]>
-M: Andrea Parri <[email protected]>
+M: Andrea Parri <[email protected]>
M: Will Deacon <[email protected]>
M: Peter Zijlstra <[email protected]>
M: Boqun Feng <[email protected]>
--
2.17.1


2019-08-05 15:02:21

by Akira Yokosawa

[permalink] [raw]
Subject: Re: [PATCH] MAINTAINERS: Update e-mail address for Andrea Parri

On Mon, 5 Aug 2019 14:15:17 +0200, Andrea Parri wrote:
> My @amarulasolutions.com address stopped working this July, so update
> to my @gmail.com address where you'll still be able to reach me.
>
> Signed-off-by: Andrea Parri <[email protected]>
> Cc: Alan Stern <[email protected]>
> Cc: Will Deacon <[email protected]>
> Cc: Peter Zijlstra <[email protected]>
> Cc: Boqun Feng <[email protected]>
> Cc: Nicholas Piggin <[email protected]>
> Cc: David Howells <[email protected]>
> Cc: Jade Alglave <[email protected]>
> Cc: Luc Maranget <[email protected]>
> Cc: "Paul E. McKenney" <[email protected]>
> Cc: Akira Yokosawa <[email protected]>
> Cc: Daniel Lustig <[email protected]>
> ---
> MAINTAINERS | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)

Hi Andrea,

Why don't you also add an entry in .mailmap as Will did in commit
c584b1202f2d ("MAINTAINERS: Update my email address to use @kernel.org")?

Thanks, Akira

>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 6426db5198f05..527317026492f 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -9326,7 +9326,7 @@ F: drivers/misc/lkdtm/*
>
> LINUX KERNEL MEMORY CONSISTENCY MODEL (LKMM)
> M: Alan Stern <[email protected]>
> -M: Andrea Parri <[email protected]>
> +M: Andrea Parri <[email protected]>
> M: Will Deacon <[email protected]>
> M: Peter Zijlstra <[email protected]>
> M: Boqun Feng <[email protected]>
>

2019-08-05 15:18:22

by Andrea Parri

[permalink] [raw]
Subject: Re: [PATCH] MAINTAINERS: Update e-mail address for Andrea Parri

> Why don't you also add an entry in .mailmap as Will did in commit
> c584b1202f2d ("MAINTAINERS: Update my email address to use @kernel.org")?

I considered it but could not understand its purpose... Maybe you can
explain it to me? ;-) (can resend with this change if needed/desired).

Thanks,
Andrea

2019-08-05 15:24:35

by James Bottomley

[permalink] [raw]
Subject: Re: [PATCH] MAINTAINERS: Update e-mail address for Andrea Parri

On Mon, 2019-08-05 at 17:15 +0200, Andrea Parri wrote:
> > Why don't you also add an entry in .mailmap as Will did in commit
> > c584b1202f2d ("MAINTAINERS: Update my email address to use
> > @kernel.org")?
>
> I considered it but could not understand its purpose... Maybe you
> can explain it to me? ;-) (can resend with this change if
> needed/desired).

man git-shortlog gives you the gory detail, but its use is to "coalesce
together commits by the same person in the shortlog, where their name
and/or email address was spelled differently." The usual way this
happens is that people have the name that appears in the From field
with and without initials.

James

2019-08-05 16:22:00

by Andrea Parri

[permalink] [raw]
Subject: Re: [PATCH] MAINTAINERS: Update e-mail address for Andrea Parri

On Mon, Aug 05, 2019 at 08:23:38AM -0700, James Bottomley wrote:
> On Mon, 2019-08-05 at 17:15 +0200, Andrea Parri wrote:
> > > Why don't you also add an entry in .mailmap as Will did in commit
> > > c584b1202f2d ("MAINTAINERS: Update my email address to use
> > > @kernel.org")?
> >
> > I considered it but could not understand its purpose... Maybe you
> > can explain it to me? ;-) (can resend with this change if
> > needed/desired).
>
> man git-shortlog gives you the gory detail, but its use is to "coalesce
> together commits by the same person in the shortlog, where their name
> and/or email address was spelled differently." The usual way this
> happens is that people have the name that appears in the From field
> with and without initials.

Thanks for the remarks, James. Given this, I'm okay with the submitted
version (i.e., no change to .mailmap).

Andrea

2019-08-05 17:46:05

by Paul E. McKenney

[permalink] [raw]
Subject: Re: [PATCH] MAINTAINERS: Update e-mail address for Andrea Parri

On Mon, Aug 05, 2019 at 08:23:38AM -0700, James Bottomley wrote:
> On Mon, 2019-08-05 at 17:15 +0200, Andrea Parri wrote:
> > > Why don't you also add an entry in .mailmap as Will did in commit
> > > c584b1202f2d ("MAINTAINERS: Update my email address to use
> > > @kernel.org")?
> >
> > I considered it but could not understand its purpose... Maybe you
> > can explain it to me? ;-) (can resend with this change if
> > needed/desired).
>
> man git-shortlog gives you the gory detail, but its use is to "coalesce
> together commits by the same person in the shortlog, where their name
> and/or email address was spelled differently." The usual way this
> happens is that people have the name that appears in the From field
> with and without initials.

New one on me, thank you! So I should have a line in .mailmap like this?

Paul E. McKenney <[email protected]> <[email protected]> <[email protected]>

Thanx, Paul

2019-08-05 18:10:28

by James Bottomley

[permalink] [raw]
Subject: Re: [PATCH] MAINTAINERS: Update e-mail address for Andrea Parri

On Mon, 2019-08-05 at 10:43 -0700, Paul E. McKenney wrote:
> On Mon, Aug 05, 2019 at 08:23:38AM -0700, James Bottomley wrote:
> > On Mon, 2019-08-05 at 17:15 +0200, Andrea Parri wrote:
> > > > Why don't you also add an entry in .mailmap as Will did in
> > > > commit
> > > > c584b1202f2d ("MAINTAINERS: Update my email address to use
> > > > @kernel.org")?
> > >
> > > I considered it but could not understand its purpose... Maybe
> > > you can explain it to me? ;-) (can resend with this change if
> > > needed/desired).
> >
> > man git-shortlog gives you the gory detail, but its use is to
> > "coalesce together commits by the same person in the shortlog,
> > where their name and/or email address was spelled
> > differently." The usual way this happens is that people have the
> > name that appears in the From field with and without initials.
>
> New one on me, thank you! So I should have a line in .mailmap like
> this?
>
> Paul E. McKenney <[email protected]> <[email protected]
> rg> <[email protected]>

Well, you could, but there's no need. As long as your email has 'Paul
E. McKenney' as the text prefix, git-shortlog will do the correct
aggregation without any need for a .mailmap entry. However, if, say,
your linaro email had been

Paul McKenney <[email protected]>

Then you would need one because git-shortlog would think 'Paul
McKenney' and 'Paul E. McKenney' were two different people.

James

2019-08-05 22:19:52

by Paul E. McKenney

[permalink] [raw]
Subject: Re: [PATCH] MAINTAINERS: Update e-mail address for Andrea Parri

On Mon, Aug 05, 2019 at 11:09:28AM -0700, James Bottomley wrote:
> On Mon, 2019-08-05 at 10:43 -0700, Paul E. McKenney wrote:
> > On Mon, Aug 05, 2019 at 08:23:38AM -0700, James Bottomley wrote:
> > > On Mon, 2019-08-05 at 17:15 +0200, Andrea Parri wrote:
> > > > > Why don't you also add an entry in .mailmap as Will did in
> > > > > commit
> > > > > c584b1202f2d ("MAINTAINERS: Update my email address to use
> > > > > @kernel.org")?
> > > >
> > > > I considered it but could not understand its purpose... Maybe
> > > > you can explain it to me? ;-) (can resend with this change if
> > > > needed/desired).
> > >
> > > man git-shortlog gives you the gory detail, but its use is to
> > > "coalesce together commits by the same person in the shortlog,
> > > where their name and/or email address was spelled
> > > differently." The usual way this happens is that people have the
> > > name that appears in the From field with and without initials.
> >
> > New one on me, thank you! So I should have a line in .mailmap like
> > this?
> >
> > Paul E. McKenney <[email protected]> <[email protected]
> > rg> <[email protected]>
>
> Well, you could, but there's no need. As long as your email has 'Paul
> E. McKenney' as the text prefix, git-shortlog will do the correct
> aggregation without any need for a .mailmap entry. However, if, say,
> your linaro email had been
>
> Paul McKenney <[email protected]>
>
> Then you would need one because git-shortlog would think 'Paul
> McKenney' and 'Paul E. McKenney' were two different people.

Thank you for the explication!

Thanx, Paul

2019-08-07 16:22:27

by Paul E. McKenney

[permalink] [raw]
Subject: Re: [PATCH] MAINTAINERS: Update e-mail address for Andrea Parri

On Mon, Aug 05, 2019 at 02:15:17PM +0200, Andrea Parri wrote:
> My @amarulasolutions.com address stopped working this July, so update
> to my @gmail.com address where you'll still be able to reach me.
>
> Signed-off-by: Andrea Parri <[email protected]>

Queued for the v5.4 merge window, thank you!

Thanx, Paul

> Cc: Alan Stern <[email protected]>
> Cc: Will Deacon <[email protected]>
> Cc: Peter Zijlstra <[email protected]>
> Cc: Boqun Feng <[email protected]>
> Cc: Nicholas Piggin <[email protected]>
> Cc: David Howells <[email protected]>
> Cc: Jade Alglave <[email protected]>
> Cc: Luc Maranget <[email protected]>
> Cc: "Paul E. McKenney" <[email protected]>
> Cc: Akira Yokosawa <[email protected]>
> Cc: Daniel Lustig <[email protected]>
> ---
> MAINTAINERS | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 6426db5198f05..527317026492f 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -9326,7 +9326,7 @@ F: drivers/misc/lkdtm/*
>
> LINUX KERNEL MEMORY CONSISTENCY MODEL (LKMM)
> M: Alan Stern <[email protected]>
> -M: Andrea Parri <[email protected]>
> +M: Andrea Parri <[email protected]>
> M: Will Deacon <[email protected]>
> M: Peter Zijlstra <[email protected]>
> M: Boqun Feng <[email protected]>
> --
> 2.17.1
>