<p dir="ltr">+1 Looks good to me, but someone else must approve.</p>
<div class="gmail_quote">On Feb 13, 2014 12:18 PM, "Bradley Lowekamp" <<a href="mailto:blowekamp@mail.nih.gov">blowekamp@mail.nih.gov</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Matt,<br>
<br>
The patch looks good to me:<br>
<a href="https://github.com/blowekamp/ITK/commit/49294b0683e773dfe3adc9240a5646d55db2ae05" target="_blank">https://github.com/blowekamp/ITK/commit/49294b0683e773dfe3adc9240a5646d55db2ae05</a><br>
<br>
It should be ready to merge.<br>
<br>
Brad<br>
<br>
On Feb 10, 2014, at 11:27 AM, Matt McCormick <<a href="mailto:matt.mccormick@kitware.com">matt.mccormick@kitware.com</a>> wrote:<br>
<br>
> Hi Brad,<br>
><br>
> Thanks for working on the patch! Since the dashboard branch does not<br>
> see much activity, I can push it manually.  It looks good now to me.<br>
> Please let us know when you think it is ready, and it will be pushed.<br>
><br>
> Thanks,<br>
> Matt<br>
><br>
> On Mon, Feb 10, 2014 at 11:16 AM, Bradley Lowekamp<br>
> <<a href="mailto:blowekamp@mail.nih.gov">blowekamp@mail.nih.gov</a>> wrote:<br>
>> Hello,<br>
>><br>
>> I have a patch for the dashboard brach:<br>
>> <a href="https://github.com/blowekamp/ITK/commit/66f94e57865626d9b54491196b57c859b7d2daf3" target="_blank">https://github.com/blowekamp/ITK/commit/66f94e57865626d9b54491196b57c859b7d2daf3</a><br>
>><br>
>> I tried pushing to gerrit:<br>
>><br>
>> git push gerrit HEAD:refs/for/dashboard<br>
>> Counting objects: 83, done.<br>
>> Delta compression using up to 80 threads.<br>
>> Compressing objects: 100% (40/40), done.<br>
>> Writing objects: 100% (72/72), 16.07 KiB | 0 bytes/s, done.<br>
>> Total 72 (delta 42), reused 54 (delta 32)<br>
>> remote: Resolving deltas: 100% (42/42)<br>
>> To blowekamp@review.source.kitware.com:ITK<br>
>> ! [remote rejected] HEAD -> refs/for/dashboard (branch dashboard not found)<br>
>> error: failed to push some refs to 'blowekamp@review.source.kitware.com:ITK'<br>
>><br>
>> Thanks,<br>
>> Brad<br>
>><br>
<br>
_______________________________________________<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>
_______________________________________________<br>
Community mailing list<br>
<a href="mailto:Community@itk.org">Community@itk.org</a><br>
<a href="http://public.kitware.com/cgi-bin/mailman/listinfo/community" target="_blank">http://public.kitware.com/cgi-bin/mailman/listinfo/community</a><br>
</blockquote></div>