<font color='black' size='2' face='arial'>We also need to start enforcing a rule that <font size="2">you CANNOT approve with only one <font size="2">reviewer unless the Gerrit CDash@home builds are totally green.<br>
<br>
<font size="2">Does <font size="2">anybody object to starting that immediately, now that we are getting<font size="2"> </font>closer to green eveywhere?</font></font><br>
<br>
<font size="2">Obviously, there will be exceptional cases,<font size="2"> but 99% of the gerrit topics should be totally green before mergin<font size="2">g<font size="2">.<font size="2"> One of the exceptions I can think of right <font size="2">away is to fix a<font size="2">n important dashb<font size="2">oard error elsewhere that has not shown up on the CDa<font size="2">sh@home builds. In such a case, I would still think it wise to <font size="2">wait for green gerrit builds, but a<font size="2">t a minimum, there should be multiple people who build a<font size="2">nd test it themselves<font size="2">.</font></font></font></font></font></font></font></font></font></font></font></font></font><br>
<br>
<br>
<font size="2">D<br>
<br>
</font></font></font><font color="black" face="arial" size="2">

<div> <br>

</div>



<div> <br>

</div>



<div> <br>

</div>



<div style="font-family:arial,helvetica;font-size:10pt;color:black">-----Original Message-----<br>

From: Bill Lorensen <bill.lorensen@gmail.com><br>

To: Marcus D. Hanwell <marcus.hanwell@kitware.com><br>

Cc: VTK Developers <vtk-developers@vtk.org><br>

Sent: Wed, Feb 13, 2013 11:49 am<br>

Subject: [vtk-developers] Gerrit request build<br>

<br>









<div id="AOLMsgPart_0_6aa9daac-999e-4f69-ae42-9567bb2fabca" style="margin: 0px;font-family: Tahoma, Verdana, Arial, Sans-Serif;font-size: 12px;color: #000;background-color: #fff;">

<pre style="font-size: 9pt;"><tt>Marcus,

I notice that "request build" works at the change level, but not the
topic level.

As we get more outside contributors, we insiders need to make sure a
topic compiles and passes tests before we approve and submit a topic.

Just today, an external contributors topic was approved and merged
without a gerrit build. Turns out there was a syntax error.

Bill
_______________________________________________
Powered by <a href="http://www.kitware.com" target="_blank">www.kitware.com</a>

Visit other Kitware open-source projects at <a href="http://www.kitware.com/opensource/opensource.html" target="_blank">http://www.kitware.com/opensource/opensource.html</a>

Follow this link to subscribe/unsubscribe:
<a href="http://www.vtk.org/mailman/listinfo/vtk-developers" target="_blank">http://www.vtk.org/mailman/listinfo/vtk-developers</a>

</tt></pre>
</div>

 <!-- end of AOLMsgPart_0_6aa9daac-999e-4f69-ae42-9567bb2fabca -->=<font size="2">==</font></div>
</font></font>