That does help clear things up. The gerrit code 'git gerrit-push' that was added doesn't work though. Git claims that its not a command that it supports. I did find some instructions at this link that worked <a href="http://public.kitware.com/Wiki/ITK/Gerrit" target="_blank">http://public.kitware.com/Wiki/ITK/Gerrit</a>. When doing the git remote command I changed it to VTK instead of ITK and then followed the rest of it. I did mess up on the commenting and I'll have to see if I can clean that up. So the code is now up for review. <br>


<br><div class="gmail_quote">On Tue, Feb 22, 2011 at 2:23 PM, Bill Lorensen <span dir="ltr"><<a href="mailto:bill.lorensen@gmail.com" target="_blank">bill.lorensen@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


Of the two methods, I would recommend using Gerrit. It is more likely<br>
to be reviewed by more people.<br>
<font color="#888888"><br>
Bill<br>
</font><div><div></div><div><br>
On Tue, Feb 22, 2011 at 1:59 PM, Marcus D. Hanwell<br>
<<a href="mailto:marcus.hanwell@kitware.com" target="_blank">marcus.hanwell@kitware.com</a>> wrote:<br>
> On Tue, Feb 22, 2011 at 1:48 PM, David Doria <<a href="mailto:daviddoria@gmail.com" target="_blank">daviddoria@gmail.com</a>> wrote:<br>
>> On Tue, Feb 22, 2011 at 1:29 PM, Glenn Faken <<a href="mailto:glenn.faken@gmail.com" target="_blank">glenn.faken@gmail.com</a>> wrote:<br>
>>> Part of our course is to try and get any code that we fix merged in with<br>
>>> the official release. If not that is ok but it would be like extra brownie<br>
>>> points if we could. Would using those other sites be a good first step and<br>
>>> then move on to official? From looking at the git turturial it looked like<br>
>>> when you make a topic it was kind of like a side off shoot that I could mess<br>
>>> around with and then when I push it to git it would then evaluated. If the<br>
>>> code didn't mess anything up then it would be moved onto the next stage till<br>
>>> it was apart of the main code. I guess the way I read it made it seem like<br>
>>> its my own playground until somebody approved the changes.<br>
>><br>
>> Glenn,<br>
>><br>
>> The "official" entry process is through Gerrit as of late. You have to<br>
>> push to Gerrit and have a couple of people approve the changes. Only<br>
>> then can you push to master.<br>
>><br>
> That is not correct, if you do not have push access (which we don't<br>
> give straight away) then either sharing topics on Github/Gitorious or<br>
> pushing your commits to Gerrit is the only way to get your changes<br>
> looked at. Then emailing the development list asking for a review is a<br>
> good way to go. If you use Gerrit then some of us, such as myself, get<br>
> emailed whenever a VTK commit is pushed.<br>
><br>
> Once it is approved then one of us would need to merge it. If you<br>
> continued to contribute changes, and the quality was good, then we<br>
> would look at giving greater privileges to directly merge your changes<br>
> using the stage. Is that clearer?<br>
><br>
> <a href="http://www.vtk.org/Wiki/VTK/Git/Simple" target="_blank">http://www.vtk.org/Wiki/VTK/Git/Simple</a><br>
><br>
> I added a brief section about basic use of Gerrit to push a topic<br>
> branch (taken from ParaView's page).<br>
><br>
> Marcus<br>
> --<br>
> Marcus D. Hanwell, Ph.D.<br>
> R&D Engineer, Kitware Inc.<br>
> (518) 881-4937<br>
</div></div><div><div></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 <a href="http://www.kitware.com/opensource/opensource.html" target="_blank">http://www.kitware.com/opensource/opensource.html</a><br>
><br>
> Follow this link to subscribe/unsubscribe:<br>
> <a href="http://www.vtk.org/mailman/listinfo/vtk-developers" target="_blank">http://www.vtk.org/mailman/listinfo/vtk-developers</a><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 <a href="http://www.kitware.com/opensource/opensource.html" target="_blank">http://www.kitware.com/opensource/opensource.html</a><br>
<br>
Follow this link to subscribe/unsubscribe:<br>
<a href="http://www.vtk.org/mailman/listinfo/vtk-developers" target="_blank">http://www.vtk.org/mailman/listinfo/vtk-developers</a><br>
<br>
</div></div></blockquote></div><br><br clear="all"><br>-- <br><br>Glenn Faken<br>