<div dir="ltr"><div dir="ltr"><br></div><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, May 24, 2019 at 7:34 PM Matt Caswell <<a href="mailto:matt@openssl.org">matt@openssl.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 24/05/2019 10:28, SHANE LONTIS wrote:<br>
> It doesn’t stop us both reviewing a PR. That doesn’t mean we both need to approve.<br>
<br>
Right...but in Matthias's version if you raise a PR, and then Pauli approves it,<br>
then you only then need to get a second committer approval. Otherwise you would<br>
need to get an OMC approval.<br></blockquote><div><br></div><div>It works that way in the original wording too - which is more simply stated IMHO. </div><div>You choose which approvals you combine. If there are three - select which ever two make the set you want to "combine" as such.</div><div><br></div><div>I also didn't see a need for a separate OMC approval if a committer submitted something and a same-organisation OMC member approved it - it just needs one more -  so the combination of approvals can be made from not-the-same-organisation.</div><div><br></div><div>Tim.</div><div><br></div><div> </div></div></div>