<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Apr 16, 2020 at 12:02 PM Chris Tetreault <<a href="mailto:ctetreau@quicinc.com">ctetreau@quicinc.com</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">





<div lang="EN-US">
<div class="gmail-m_-5657094412268463651WordSection1">
<p class="MsoNormal">> you generally shouldn't approve patches if you aren't in a position to have committed the patch without review if it was your own patch<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">I think this might be an overly high bar. Even the most complicated patches need to be approved by somebody. However I think few would argue that the most complicated patches should be committed by anybody without review.</p></div></div></blockquote><div><br></div><div>Certainly - hence the "generally" and the other clauses in that sentence. I'd describe that as sort of "cross-review" - perhaps "reaching consensus", etc.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div lang="EN-US"><div class="gmail-m_-5657094412268463651WordSection1"><p class="MsoNormal"><u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal"><b>From:</b> llvm-dev <<a href="mailto:llvm-dev-bounces@lists.llvm.org" target="_blank">llvm-dev-bounces@lists.llvm.org</a>> <b>On Behalf Of
</b>David Blaikie via llvm-dev<br>
<b>Sent:</b> Wednesday, April 15, 2020 1:52 PM<br>
<b>To:</b> Andrew Kelley <<a href="mailto:andrew@ziglang.org" target="_blank">andrew@ziglang.org</a>><br>
<b>Cc:</b> LLVM Dev <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>><br>
<b>Subject:</b> [EXT] Re: [llvm-dev] whatever happened to this patch?<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<div>
<p class="MsoNormal">Generally an author should submit an approved patch. If they do not have commit access they should ask someone who does (usually the approver) to commit it on their behalf.<br>
<br>
(you generally shouldn't approve patches if you aren't in a position to have committed the patch without review if it was your own patch - so at the very least having commit access, but also being sufficiently familiar with the norms of the part of the project
 that you're confident committing without review (this doesn't fully cover all cases, some areas do more cross-review than others, etc))<u></u><u></u></p>
</div>
<p class="MsoNormal"><u></u> <u></u></p>
<div>
<div>
<p class="MsoNormal">On Wed, Apr 15, 2020 at 1:44 PM Andrew Kelley via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>> wrote:<u></u><u></u></p>
</div>
<blockquote style="border-top:none;border-right:none;border-bottom:none;border-left:1pt solid rgb(204,204,204);padding:0in 0in 0in 6pt;margin-left:4.8pt;margin-right:0in">
<p class="MsoNormal"><a href="https://reviews.llvm.org/D62475" target="_blank">https://reviews.llvm.org/D62475</a><br>
<br>
It looks like it never got merged, what was the missing step to get this<br>
landed?<br>
<br>
Regards,<br>
Andrew<br>
<br>
_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><u></u><u></u></p>
</blockquote>
</div>
</div>
</div>

</blockquote></div></div>