<p>Nico Huber has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/29471">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">MAINTAINERS: Clarify this is about active upstream development<br><br><vendor> seems to be confused about the meaning of our maintainers list.<br>I get the feeling some use it to organize corporate internal teams and<br>branches, adding names to the list that don't show up in Gerrit and even<br>if, often don't react to reviewing requests (within months). Maybe they<br>even don't know that this is about coreboot.org?<br><br>To clarify this:<br><br>  o Add an introductory paragraph mentioning development on coreboot.org.<br>  o Explicitly state that maintainers should be registered to Gerrit.<br>  o If a topic is tagged as `Supported`, expect that somebody reacts to<br>    review requests.<br><br>Change-Id: I9ee038dc5ee1f4993ba1d230ef6e737f20e2ff8a<br>Signed-off-by: Nico Huber <nico.huber@secunet.com><br>---<br>M MAINTAINERS<br>1 file changed, 12 insertions(+), 2 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/71/29471/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/MAINTAINERS b/MAINTAINERS</span><br><span>index b7031c5..46b588a 100644</span><br><span>--- a/MAINTAINERS</span><br><span>+++ b/MAINTAINERS</span><br><span>@@ -1,6 +1,14 @@</span><br><span> </span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">- List of maintainers and how to submit coreboot changes</span><br><span style="color: hsl(120, 100%, 40%);">+                    List of upstream coreboot maintainers</span><br><span style="color: hsl(120, 100%, 40%);">+                  and how to submit coreboot changes</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+This represents the list of maintainers that work on the upstream coreboot</span><br><span style="color: hsl(120, 100%, 40%);">+code base (on coreboot.org).  Maintainers are assigned to topics and when</span><br><span style="color: hsl(120, 100%, 40%);">+applicable to subtrees of the source tree. You'll find some subtrees that</span><br><span style="color: hsl(120, 100%, 40%);">+don't have a maintainer. If you are looking for reviewers for such a sub-</span><br><span style="color: hsl(120, 100%, 40%);">+tree, it's often a good choice to look at the git history who worked on it</span><br><span style="color: hsl(120, 100%, 40%);">+lastly.</span><br><span> </span><br><span> Please try to follow the guidelines below.  This will make things</span><br><span> easier on the maintainers.  Not all of these guidelines matter for every</span><br><span>@@ -65,6 +73,7 @@</span><br><span> Descriptions of section entries:</span><br><span> </span><br><span>     M: Maintainer: FullName <address@domain></span><br><span style="color: hsl(120, 100%, 40%);">+           Should be registered to Gerrit (https://review.coreboot.org/).</span><br><span>    R: Designated reviewer: FullName <address@domain></span><br><span>         These reviewers should be CCed on patches.</span><br><span>        L: Mailing list that is relevant to this area</span><br><span>@@ -73,7 +82,8 @@</span><br><span>    T: SCM tree type and location.</span><br><span>          Type is one of: git, hg, quilt, stgit, topgit</span><br><span>     S: Status, one of the following:</span><br><span style="color: hsl(0, 100%, 40%);">-           Supported:   Someone is actually paid to look after this.</span><br><span style="color: hsl(120, 100%, 40%);">+     Supported:   Someone is actually paid to look after this and can</span><br><span style="color: hsl(120, 100%, 40%);">+                   be expected to react to review requests.</span><br><span>        Maintained:  Someone actually looks after it.</span><br><span>        Odd Fixes:   It has a maintainer but they don't have time to do</span><br><span>                       much other than throw the odd patch in. See below..</span><br><span></span><br></pre><p>To view, visit <a href="https://review.coreboot.org/29471">change 29471</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://review.coreboot.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://review.coreboot.org/29471"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: coreboot </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>
<div style="display:none"> Gerrit-Change-Id: I9ee038dc5ee1f4993ba1d230ef6e737f20e2ff8a </div>
<div style="display:none"> Gerrit-Change-Number: 29471 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Nico Huber <nico.h@gmx.de> </div>