OK with me.<div><br></div><div>BTW, how did this error sneak through. I see that the continuous reported it yesterday, but the gerrit ronot builds for the patch were OK.</div><div><br></div><div>Bill</div><div><br><br><div class="gmail_quote">
On Fri, Nov 2, 2012 at 4:26 PM, Bradley Lowekamp <span dir="ltr"><<a href="mailto:blowekamp@mail.nih.gov" target="_blank">blowekamp@mail.nih.gov</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div style="word-wrap:break-word">Bill,<div><br></div><div>It's the end of the work day for myself. My patches compiled on my system and the one Mac that is responsive on CDash at home. They are rather trivial changes that have been approved by others. If they don't fix the problem at hand the dashboard is not going to be worse off. I am going ahead and merging them.</div>
<div><br></div><div>Brad</div><div><div class="h5"><div><br><div><div>On Nov 2, 2012, at 4:13 PM, Bill Lorensen <<a href="mailto:bill.lorensen@gmail.com" target="_blank">bill.lorensen@gmail.com</a>> wrote:</div><br>
<blockquote type="cite">Did you build and test this on a Visual Studio system? If so, I have no problems with merging it.<br><br><div class="gmail_quote">On Fri, Nov 2, 2012 at 3:34 PM, Bradley Lowekamp <span dir="ltr"><<a href="mailto:blowekamp@mail.nih.gov" target="_blank">blowekamp@mail.nih.gov</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word">I am not sure I see the point to that as it's expected that it will have a compilation error due to the unresolved mesh test.<div>
<br></div><div>I'd like to get the patches in to fix the dashboard I completed before 9am merged before I go home today so they will be fixed for tomorrow.</div><div><br></div><div>Brad</div><div><div>
<br><div><div>On Nov 2, 2012, at 3:21 PM, Bill Lorensen <<a href="mailto:bill.lorensen@gmail.com" target="_blank">bill.lorensen@gmail.com</a>> wrote:</div><br><blockquote type="cite">I think we should wait for a PC build to complete.<br>
<br><div class="gmail_quote">On Fri, Nov 2, 2012 at 1:30 PM, Bradley Lowekamp <span dir="ltr"><<a href="mailto:blowekamp@mail.nih.gov" target="_blank">blowekamp@mail.nih.gov</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word">Bill,<div><br></div><div>It appears that only the Mac factory is reporting on gerrit today.</div>
<div><br></div><div>I submitted 2 patches to address test failures and compilation errors which are occurring on the dashboard. </div><div><br></div><div>Error which is resolved by the patch you linked below, only occurs on non-windows builds with shared libraries. None of the robot builds are configured that way.</div>
<div><br></div><div>When do you propose that I can merge my patches?</div><div><br></div><div>Thanks,</div><div>Brad</div><div><br><div><div><div>On Nov 2, 2012, at 10:56 AM, Bill Lorensen <<a href="mailto:bill.lorensen@gmail.com" target="_blank">bill.lorensen@gmail.com</a>> wrote:</div>
<br></div><blockquote type="cite"><div>Folks,
<div><br></div><div>I see gerrit topics being merged before the robot builds complete.For example, this topic: <a href="http://review.source.kitware.com/#/c/8262/" target="_blank">http://review.source.kitware.com/#/c/8262/</a> was merged, yet the robot build shows errors on windows: <a href="http://open.cdash.org/index.php?project=Insight&filtercount=2&field1=buildname/string&compare1=63&value1=VTKPolyDataMeshIOTensorPixels2961-Ifb523cf-2&field2=buildstarttime/date&compare2=83&value2=2012-10-31" target="_blank">http://open.cdash.org/index.php?project=Insight&filtercount=2&field1=buildname/string&compare1=63&value1=VTKPolyDataMeshIOTensorPixels2961-Ifb523cf-2&field2=buildstarttime/date&compare2=83&value2=2012-10-31</a></div>
<div><br></div><div>I see this topic: <a href="http://review.source.kitware.com/#/c/8295/" target="_blank">http://review.source.kitware.com/#/c/8295/</a> has been approved, but the robots have not reported yet.</div><div>
<br></div><div>Now the nightly has errors and gerrit topics are showing the same errors. Compiler errors in the robot builds caused by previously merged topics are bad.</div>
<div><br></div><div>Bill</div></div>
_______________________________________________<br>Powered by <a href="http://www.kitware.com/" target="_blank">www.kitware.com</a><br><br>Visit other Kitware open-source projects at<br><a href="http://www.kitware.com/opensource/opensource.html" target="_blank">http://www.kitware.com/opensource/opensource.html</a><br>
<br>Kitware offers ITK Training Courses, for more information visit:<br><a href="http://kitware.com/products/protraining.php" target="_blank">http://kitware.com/products/protraining.php</a><br><br>Please keep messages on-topic and check the ITK FAQ at:<br>
<a href="http://www.itk.org/Wiki/ITK_FAQ" target="_blank">http://www.itk.org/Wiki/ITK_FAQ</a><br><br>Follow this link to subscribe/unsubscribe:<br><a href="http://www.itk.org/mailman/listinfo/insight-developers" target="_blank">http://www.itk.org/mailman/listinfo/insight-developers</a><br>
</blockquote></div><br></div></div></blockquote></div><br><br clear="all"><br>-- <br>Unpaid intern in BillsBasement at noware dot com<br><br>
</blockquote></div><br></div></div></div></blockquote></div><br><br clear="all"><br>-- <br>Unpaid intern in BillsBasement at noware dot com<br><br>
</blockquote></div><br></div></div></div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br>Unpaid intern in BillsBasement at noware dot com<br><br>
</div>