<div dir="ltr">The default behavior, rebasing automatically, is the sane default to avoid having developers run into unexpected merge conflicts on new patch submissions.<div><br></div><div>But if git-review can check to see if a review already exists in gerrit *before* doing the local rebase, I'd be in favor of it skipping the rebase by default if the review already exists. Require developers to rebase existing patches manually. (This is my way of saying I can't think of a good answer to your question.)</div><div><br></div><div><div>While we're on the topic, it's also worth noting that --no-rebase becomes critically important when a patch in the review sequence has already been approved, because the entire series will be rebased, potentially pulling patches out of the gate, clearing the Workflow+1 bit, and resetting the gate (probably unnecessarily). A tweak to the default behavior would help avoid this scenario.</div><div><br></div><div>-Dolph<br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Dec 30, 2014 at 8:46 AM, David Kranz <span dir="ltr"><<a href="mailto:dkranz@redhat.com" target="_blank">dkranz@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Many times when I review a revision of an existing patch, I can't see just the change from the previous version due to other rebases. The git-review documentation mentions this issue and suggests using -R to make life easier for reviewers when submitting new revisions. Can some one explain when we should *not* use -R after doing 'git commit --amend'? Or is using -R just something that should be done but many folks don't know about it?<br>
<br>
-David<br>
<br>
>From git-review doc:<br>
<br>
-R, --no-rebase<br>
Do not automatically perform a rebase before submitting the<br>
change to Gerrit.<br>
<br>
When submitting a change for review, you will usually want it to<br>
be based on the tip of upstream branch in order to avoid possible<br>
conflicts. When amending a change and rebasing the new patchset,<br>
the Gerrit web interface will show a difference between the two<br>
patchsets which contains all commits in between. This may confuse<br>
many reviewers that would expect to see a much simpler differ‐<br>
ence.<br>
<br>
<br>
______________________________<u></u>_________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org" target="_blank">OpenStack-dev@lists.openstack.<u></u>org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/<u></u>cgi-bin/mailman/listinfo/<u></u>openstack-dev</a><br>
</blockquote></div><br></div></div></div></div>