MAINTAINERS: Clarify the policy WRT the Write After Approval list

Message ID alpine.DEB.2.20.2110271324290.13542@tpp.orcam.me.uk
State Committed
Commit c79399c7e128a3eaa61597f5991e909224fc9808
Headers
Series MAINTAINERS: Clarify the policy WRT the Write After Approval list |

Commit Message

Maciej W. Rozycki Oct. 27, 2021, 12:32 p.m. UTC
  * MAINTAINERS: Clarify the policy WRT the Write After Approval
	list.
---
On Tue, 26 Oct 2021, Jeff Law wrote:

> > >   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.

 Thanks.  It did not occur to me that we had such a policy in place even 
before said commit, and now that I can see it is the case it seems to me 
like it has been a recurring problem with people not being aware of it, 
and I can hardly imagine anyone running the test suite for a MAINTAINERS 
file update.

 So while I maintain my concerns about the policy itself, how about this 
change, so that at least it's written down somewhere other than mailing 
list archives only?

  Maciej
---
 MAINTAINERS | 3 +++
 1 file changed, 3 insertions(+)
  

Comments

Maciej W. Rozycki Nov. 3, 2021, 2 p.m. UTC | #1
On Wed, 27 Oct 2021, Maciej W. Rozycki wrote:

> 	* MAINTAINERS: Clarify the policy WRT the Write After Approval
> 	list.

 Ping for:

<https://gcc.gnu.org/pipermail/gcc-patches/2021-October/582690.html>

  Maciej
  
Jeff Law Nov. 3, 2021, 3:44 p.m. UTC | #2
On 11/3/2021 8:00 AM, Maciej W. Rozycki wrote:
> On Wed, 27 Oct 2021, Maciej W. Rozycki wrote:
>
>> 	* MAINTAINERS: Clarify the policy WRT the Write After Approval
>> 	list.
>   Ping for:
>
> <https://gcc.gnu.org/pipermail/gcc-patches/2021-October/582690.html>
OK
jeff
  
Maciej W. Rozycki Nov. 3, 2021, 5:08 p.m. UTC | #3
On Wed, 3 Nov 2021, Jeff Law wrote:

> > > 	* MAINTAINERS: Clarify the policy WRT the Write After Approval
> > > 	list.
> >   Ping for:
> > 
> > <https://gcc.gnu.org/pipermail/gcc-patches/2021-October/582690.html>
> OK

 Applied now, thanks!

  Maciej
  

Patch

diff --git a/MAINTAINERS b/MAINTAINERS
index fe56b2f647e..1471f53d30b 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -15,6 +15,9 @@  To report problems in GCC, please visit:
 
   http://gcc.gnu.org/bugs/
 
+Note: when adding someone to a more specific section please remove any
+corresponding entry from the Write After Approval list.
+
 Maintainers
 ===========