<html><head><meta http-equiv="Content-Type" content="text/html; charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class="">I believed that it required two OMC approvals but was pointed to an earlier instance where only one was present and I flew with it without checking further.<div class="">My apologies for merging prematurely and I’ll back out the changes if any OMC member wants.</div><div class=""><br class=""></div><div class="">As for discussing this at the upcoming face to face, I agree wholeheartedly.</div><div class=""><br class=""></div><div class=""><br class=""></div><div class="">Pauli</div><div class=""><div class="">
<div dir="auto" style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class=""><div style="caret-color: rgb(0, 0, 0); color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none;">-- <br class="">Dr Paul Dale | Distinguished Architect | Cryptographic Foundations <br class="">Phone +61 7 3031 7217<br class="">Oracle Australia</div><div style="caret-color: rgb(0, 0, 0); color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none;"><br class=""></div><br class="Apple-interchange-newline"></div><br class="Apple-interchange-newline">
</div>
<div><br class=""><blockquote type="cite" class=""><div class="">On 4 Oct 2019, at 5:39 pm, Matt Caswell <<a href="mailto:matt@openssl.org" class="">matt@openssl.org</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div class=""><br class=""><br class="">On 04/10/2019 08:15, Dr. Matthias St. Pierre wrote:<br class=""><blockquote type="cite" class="">Dear OMC,<br class=""><br class="">while the process of merging and committing to openssl/openssl has been formalized,<br class="">no similar (official) rules for pull requests by non-OMC-member seem to apply to the<br class="">other two repositories openssl/tools and openssl/web. Probably it's because hardly<br class="">anybody outside the OMC else ever raises them? Or is it the other way around?<br class=""></blockquote><br class="">There are clear official rules. This vote was passed by the OMC over a year ago:<br class=""><br class="">topic: Openssl-web and tools repositories shall be under the same review<br class=""> policy as per the openssl repository where the reviewers are OMC members<br class=""><br class="">So it needs two approvals from an OMC member. It looks like recent commits<br class="">haven't obeyed those rules.<br class=""><br class=""><br class=""><blockquote type="cite" class="">I would like to raise the question whether it wouldn't be beneficial for all of us,<br class="">if we would apply the same rules (commit access for all committers, plus the well<br class="">known approval rules) to all of our repos. After all, the openssl/openssl repository<br class="">is the most valuable of the three and I see no reason why the others would need<br class="">more protection. In the case of the openssl/web repository which targets the<br class="">official website, you might want to consider a 2OMC approval rule, but even there<br class="">I don't see why the usual OMC veto rule wouldn't be sufficient.<br class=""></blockquote><br class="">There is a lot of merit in that. Certainly for tools. I've added it to the OMC<br class="">agenda for Nuremburg.<br class=""><br class="">Matt<br class=""><br class=""></div></div></blockquote></div><br class=""></div></body></html>