[committed] MAINTAINERS: Add myself as a VAX port maintainer

Message ID alpine.DEB.2.20.2110261622190.13542@tpp.orcam.me.uk
State Committed
Commit 28006282028b548238cee4601117f21c498dc2b7
Headers
Series [committed] MAINTAINERS: Add myself as a VAX port maintainer |

Commit Message

Maciej W. Rozycki Oct. 26, 2021, 3:26 p.m. UTC
  * MAINTAINERS (CPU Port Maintainers): Add myself as a VAX port
	maintainer.
---
 MAINTAINERS | 1 +
 1 file changed, 1 insertion(+)
  

Comments

Martin Liška Oct. 26, 2021, 3:34 p.m. UTC | #1
On 10/26/21 17:26, Maciej W. Rozycki wrote:
> 	* MAINTAINERS (CPU Port Maintainers): Add myself as a VAX port
> 	maintainer.

Congratulations!

Please remove your name from Write After Approval:

$ make check -k RUNTESTFLAGS="maintainers.exp"
...
Running /home/marxin/Programming/gcc/gcc/testsuite/gcc.src/maintainers.exp ...
Redundant in write approval: Maciej W. Rozycki
FAIL: maintainers-verify.sh

Cheers,
Martin

> ---
>   MAINTAINERS | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index b22f930583a..901a6305184 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -117,6 +117,7 @@ tilegx port		Walter Lee		<walt@tilera.com>
>   tilepro port		Walter Lee		<walt@tilera.com>
>   v850 port		Nick Clifton		<nickc@redhat.com>
>   vax port		Matt Thomas		<matt@3am-software.com>
> +vax port		Maciej W. Rozycki	<macro@embecosm.com>
>   visium port		Eric Botcazou		<ebotcazou@libertysurf.fr>
>   x86-64 port		Jan Hubicka		<hubicka@ucw.cz>
>   xstormy16 port		Nick Clifton		<nickc@redhat.com>
>
  
Maciej W. Rozycki Oct. 26, 2021, 5:51 p.m. UTC | #2
On Tue, 26 Oct 2021, Martin Liška wrote:

> Please remove your name from Write After Approval:
> 
> $ make check -k RUNTESTFLAGS="maintainers.exp"
> ...
> Running /home/marxin/Programming/gcc/gcc/testsuite/gcc.src/maintainers.exp ...
> Redundant in write approval: Maciej W. Rozycki
> FAIL: maintainers-verify.sh

 Hmm, that seems like useless policy to me; the "Write After Approval" 
section used to be an exhaustive, alphabetically sorted list of people 
with commit rights, and it still is with our sister projects binutils and 
GDB.  By removing entries for people listed elsewhere (which may be across 
several places anyway) you require one to look for information scattered 
around the file.  And contributors to remember adding themselves back as 
they step down from maintainer posts.

 Besides, my e-mail addresses listed are different in the two sections, 
and that's been deliberate.  Also what about people carrying identical 
full names?  I insist on using my middle initial for a reason.

 It seems like there's been hardly any discussion about this matter around 
the time this stuff was added with commit bddcac9d1c32 ("[contrib] Add 
contrib/maintainers-verify.sh").  What was the actual motivation behind 
that change?

  Maciej
  
Jakub Jelinek Oct. 26, 2021, 7:06 p.m. UTC | #3
On Tue, Oct 26, 2021 at 06:51:09PM +0100, Maciej W. Rozycki wrote:
> On Tue, 26 Oct 2021, Martin Liška wrote:
> 
> > Please remove your name from Write After Approval:
> > 
> > $ make check -k RUNTESTFLAGS="maintainers.exp"
> > ...
> > Running /home/marxin/Programming/gcc/gcc/testsuite/gcc.src/maintainers.exp ...
> > Redundant in write approval: Maciej W. Rozycki
> > FAIL: maintainers-verify.sh
> 
>  Hmm, that seems like useless policy to me; the "Write After Approval" 
> section used to be an exhaustive, alphabetically sorted list of people 
> with commit rights, and it still is with our sister projects binutils and 
> GDB.  By removing entries for people listed elsewhere (which may be across 
> several places anyway) you require one to look for information scattered 
> around the file.  And contributors to remember adding themselves back as 
> they step down from maintainer posts.
> 
>  Besides, my e-mail addresses listed are different in the two sections, 
> and that's been deliberate.  Also what about people carrying identical 
> full names?  I insist on using my middle initial for a reason.
> 
>  It seems like there's been hardly any discussion about this matter around 
> the time this stuff was added with commit bddcac9d1c32 ("[contrib] Add 
> contrib/maintainers-verify.sh").  What was the actual motivation behind 
> that change?

That was only addition of a script and testcase to verify what has been done
in MAINTAINERS since forever.
Just look at all the commits to remove redundant entries from Write After
Approval, e.g.
https://gcc.gnu.org/legacy-ml/gcc-patches/2003-05/msg00366.html
All maintainers or reviewers (global or specific) have write after approval
rights for areas they don't maintain.

	Jakub
  
Jeff Law Oct. 26, 2021, 8:28 p.m. UTC | #4
On 10/26/2021 1:06 PM, Jakub Jelinek via Gcc-patches wrote:
> On Tue, Oct 26, 2021 at 06:51:09PM +0100, Maciej W. Rozycki wrote:
>> On Tue, 26 Oct 2021, Martin Liška wrote:
>>
>>> Please remove your name from Write After Approval:
>>>
>>> $ make check -k RUNTESTFLAGS="maintainers.exp"
>>> ...
>>> Running /home/marxin/Programming/gcc/gcc/testsuite/gcc.src/maintainers.exp ...
>>> Redundant in write approval: Maciej W. Rozycki
>>> FAIL: maintainers-verify.sh
>>   Hmm, that seems like useless policy to me; the "Write After Approval"
>> section used to be an exhaustive, alphabetically sorted list of people
>> with commit rights, and it still is with our sister projects binutils and
>> GDB.  By removing entries for people listed elsewhere (which may be across
>> several places anyway) you require one to look for information scattered
>> around the file.  And contributors to remember adding themselves back as
>> they step down from maintainer posts.
>>
>>   Besides, my e-mail addresses listed are different in the two sections,
>> and that's been deliberate.  Also what about people carrying identical
>> full names?  I insist on using my middle initial for a reason.
>>
>>   It seems like there's been hardly any discussion about this matter around
>> the time this stuff was added with commit bddcac9d1c32 ("[contrib] Add
>> contrib/maintainers-verify.sh").  What was the actual motivation behind
>> that change?
> That was only addition of a script and testcase to verify what has been done
> in MAINTAINERS since forever.
> Just look at all the commits to remove redundant entries from Write After
> Approval, e.g.
> https://gcc.gnu.org/legacy-ml/gcc-patches/2003-05/msg00366.html
> All maintainers or reviewers (global or specific) have write after approval
> rights for areas they don't maintain.
I went ahead and fixed Maciej's entries in the obvious way.

jeff
  

Patch

diff --git a/MAINTAINERS b/MAINTAINERS
index b22f930583a..901a6305184 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -117,6 +117,7 @@  tilegx port		Walter Lee		<walt@tilera.com>
 tilepro port		Walter Lee		<walt@tilera.com>
 v850 port		Nick Clifton		<nickc@redhat.com>
 vax port		Matt Thomas		<matt@3am-software.com>
+vax port		Maciej W. Rozycki	<macro@embecosm.com>
 visium port		Eric Botcazou		<ebotcazou@libertysurf.fr>
 x86-64 port		Jan Hubicka		<hubicka@ucw.cz>
 xstormy16 port		Nick Clifton		<nickc@redhat.com>