<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div class="h5"><div></div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote"><div>A couple more comments about the magic happening here ...</div><div><br></div><div>First, "git review" knows where to push based on a file checked in to the repo:</div><div><br></div><div> $ cat .gitreview </div><div>[gerrit]</div><div>host=<a href="http://review.openstack.org" target="_blank">review.openstack.org</a></div><div>port=29418</div><div>project=openstack/nova.git</div><div><br></div><div>"git review" also sets up a local commit hook that adds a "Change-Id" header to your commit message.  That Change-Id is what links multiple revisions of the same change together.  So, if you edit your change and push it again, as long as the Change-Id remains the same, gerrit treats it as the same review request and not a new one.</div><span class="HOEnZb"></span></div></div></div></blockquote></div><br></div><div class="gmail_extra">Sounds good to me. So it doesn't really matter from which repo you post a review so long as it's a clone of the original with that .gitreview file.<br><br></div><div class="gmail_extra">I have another question unrelated to reviews. Does your setup make it easy to mirror a repo? In a more complicated scenario, what if someone had a private fork but they wanted to get public commits to master mirrored to their repo? Would they have to treat the original repo as upstream and manually pull changes and rebase their private branch off of it?<br></div><div class="gmail_extra"><br clear="all"><br>-- <br><div dir="ltr"><pre style="white-space:pre-wrap;color:rgb(0,0,0)"><pre style="white-space:pre-wrap">Samuel Fortier-Galarneau
Digium, Inc. | Software Developer
445 Jan Davis Drive NW - Huntsville, AL 35806 - USA
Check us out at:  <a href="http://www.digium.com" target="_blank">www.digium.com</a>  & <a href="http://www.asterisk.org" target="_blank">www.asterisk.org</a></pre></pre></div>
</div></div>