<div class="gmail_quote">On Thu, Oct 14, 2010 at 3:11 PM, Bill Lorensen <span dir="ltr"><<a href="mailto:bill.lorensen@gmail.com">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;">
Here is what I've been doing in itk.<br>
<br>
I have an itk checkout that I track master with.<br>
When I review something, I create a branch just for the purpose of<br>
review. I checkout that branch.<br>
Then I pull the gerrit I want to review. I build, test and enter my<br>
comments into the review. When I'm done reviewing I delete the branch.<br>
<br>
So you are saying, I stay at master, then fetch I get this:<br>
[VTK(master)] git fetch <a href="http://review.source.kitware.com/p/VTK" target="_blank">http://review.source.kitware.com/p/VTK</a><br>
<div class="im">refs/changes/29/129/4 && git checkout FETCH_HEAD<br>
</div>From <a href="http://review.source.kitware.com/p/VTK" target="_blank">http://review.source.kitware.com/p/VTK</a><br>
 * branch            refs/changes/29/129/4 -> FETCH_HEAD<br>
Note: checking out 'FETCH_HEAD'.<br>
<br>
You are in 'detached HEAD' state. You can look around, make experimental<br>
changes and commit them, and you can discard any commits you make in this<br>
state without impacting any branches by performing another checkout.<br>
<br>
If you want to create a new branch to retain commits you create, you may<br>
do so (now or later) by using -b with the checkout command again. Example:<br>
<br>
  git checkout -b new_branch_name<br>
<br>
HEAD is now at bc7016e... ENH: Scripts to setup development environment.<br>
[VTK((no branch))]<br>
--------------------------------<br>
Now I do my build/test to verify the review.<br>
<br>
To go back to master and discard the fetched review code I<br>
<br>
git checkout master<br>
<br>
Or I can retain the fetched review code in a branch with:<br>
<br>
git checkout -b UsefulSetupScripts<br>
<div><div></div><div class="h5"><br></div></div></blockquote><div>That is how I use Gerrit, and is how I would recommend you review code. It means you have the tree in the state the submitter proposed, whereas your approach merges their changes into whatever your local master happens to be at. It also has the advantage that there is no local branch to delete unless you choose to create one for further review/pushing to the topic stage (for external contributions).</div>
<div><br></div><div>Hopefully that makes sense to you.</div><div><br></div><div>Marcus</div></div>