<p dir="ltr">My vote is people with permission to commit/push are allowed to approve.</p>
<p dir="ltr">I'm not 100% set on that clause for GeoMoose, it was part of MapServer's RFC 7.2.</p>
<p dir="ltr">Basically it implies you work in your own repo and only push to official once it has been tested/reviewed.  Usually, this is good, but I could see it hampering development of 3.0 where we know it will be broken for awhile and we want multiple people to help fix it in a coordinated fashion.</p>

<div class="gmail_quote">On May 30, 2013 11:52 AM, "Eli Adam" <<a href="mailto:eadam@co.lincoln.or.us">eadam@co.lincoln.or.us</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
RFC-4 looks good overall to me. On this:<br>
<br>
Use GitHub pull requests rather than directly committing to the<br>
branches as a practical way of testing proposed changes before<br>
inclusion.<br>
<br>
It may be worth clarifying if you can approve your own pull requests.<br>
We can discuss these details in greater depth at a PSC meeting.<br>
<br>
Eli<br>
<br>
On Tue, May 28, 2013 at 3:51 PM, Jim Klassen <<a href="mailto:klassen.js@gmail.com">klassen.js@gmail.com</a>> wrote:<br>
> PSC Members and interested GeoMoose community members,<br>
><br>
> We are looking at formalizing our committer guidelines.<br>
><br>
> At the same time moving new development in the project from Subversion to<br>
> GitHub to allow us to more easily manage the code base and in particular<br>
> make it easier to merge changes between the official GeoMoose version and<br>
> branches that are being customized for specific projects.<br>
><br>
> As the history in Subversion is unfortunately messy with respect to branches<br>
> and tags, the current thinking is that to avoid a very messy import process<br>
> that existing branches including 2.6.x will continue to be maintained in<br>
> Subversion, while newer versions (e.g 2.8 or 3.0) will be developed in Git.<br>
><br>
> Please review RFC-4 and reply with your comments.<br>
><br>
> <a href="http://www.geomoose.org/trunk/rfc/rfc-4.html" target="_blank">http://www.geomoose.org/trunk/rfc/rfc-4.html</a><br>
><br>
> Jim Klassen<br>
><br>
> _______________________________________________<br>
> Geomoose-users mailing list<br>
> <a href="mailto:Geomoose-users@lists.osgeo.org">Geomoose-users@lists.osgeo.org</a><br>
> <a href="http://lists.osgeo.org/mailman/listinfo/geomoose-users" target="_blank">http://lists.osgeo.org/mailman/listinfo/geomoose-users</a><br>
><br>
</blockquote></div>