<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Sep 18, 2014 at 3:01 PM, Samuel Galarneau <span dir="ltr"><<a href="mailto:sgalarneau@digium.com" target="_blank">sgalarneau@digium.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><span class=""><div class="gmail_extra"><div class="gmail_quote"><div><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></span></div></div></div></blockquote></div><br></div></span><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?</div></div></blockquote><div><br></div><div>Mirroring, sure.  I don't remember exactly how we do it, but all OpenStack repos are mirrored to github for convenience, for example.</div><div><br></div><div>Regarding private branches, git generally makes that kind of thing ***MUCH*** easier than svn.  You can easily track the exact commits that are applied on top of upstream.  You could either periodically rebase your changes on top of upstream (re-applying the commits, rewriting history but a cleaner history), or periodically merge from upstream.  In either case, I personally wouldn't automate it.  You want some sanity checking around that stuff.  Conflicts happen.  Really, I think this is going to be MUCH better no matter what specific infrastructure you go with.</div><div><br></div><div>-- </div><div>Russell Bryant</div></div></div></div>