<div dir="ltr"><span style="font-size:12.8px">Renato,</span><div style="font-size:12.8px"><br></div><div style="font-size:12.8px">I'm not speaking on ways to circumvent the review system, but rather on when enough is enough.</div><div style="font-size:12.8px"><br></div><div style="font-size:12.8px">Do you expect whole back-ends to receive the same level of code review as individual incremental patches to existing ones? If yes, by who? In case of existing back-ends, there are maintainers who understand the code and able to do a proper code review; who can do the same for new back-ends?</div><div style="font-size:12.8px"><br></div><div style="font-size:12.8px">Can we put some formal expectations into your guidelines?</div><div style="font-size:12.8px"><br></div><div style="font-size:12.8px">Yours,</div><div style="font-size:12.8px">Andrey</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jul 29, 2016 at 3:38 PM, Renato Golin <span dir="ltr"><<a href="mailto:renato.golin@linaro.org" target="_blank">renato.golin@linaro.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 29 July 2016 at 12:09, Andrey Bokhanko <<a href="mailto:andreybokhanko@gmail.com">andreybokhanko@gmail.com</a>> wrote:<br>
> What if a new back-end ticks all the right boxes but simply not being<br>
> reviewed by established members of the community?<br>
<br>
</span>This case is clear. The code only goes in after proper review.<br>
<br>
Interests vary from case to case, and normally, pinging on the mailing<br>
list and on IRC is the only way to go. Waiting forever doesn't work,<br>
and asking a friend maintainer to "push it for you" or just pushing<br>
yourself is very much *never* appropriate.<br>
<br>
This is one of the few clear cases I can think of that would warrant a<br>
permanent commit ban for all involved.<br>
<br>
Upstream takes time, we all have to live with it. But the alternative<br>
is worse. :)<br>
<br>
cheers,<br>
--renato<br>
</blockquote></div><br></div>