<div dir="ltr">This is getting to be an old thread, but the "magic" for adding multiple changes is as follows:<div><br></div><div>* use your working branch</div><div>* make changes</div><div>* git commit -a --amend</div><div>* save...</div><div>* git review</div><div><br></div><div>This way, the same branch, with the initial commit, will be preserved along with the same change ID.</div><div><br></div><div>If you want to build upon an initial review, then you can do:</div><div><br></div><div>* git checkout -b intial_changes</div><div>* ...make changes and git review...</div><div>* git checkout -b secondary_changes<br>* ...make changes, git commit, git review...</div><div><br></div><div>This will provide a second review, based upon the first one.</div><div><br></div><div>If you need to change the first review, then that's fine. Follow the initial procedure with making changes, and then git commit -a --amend, along with git review.</div><div><br></div><div>If upstream (master) changes, and you need to rebase, that's fine too.</div><div><br></div><div>* git checkout master</div><div>* git fetch upstream (assuming you've run a git remote add upstream <path to upstream Asterisk master>)</div><div>* git rebase upstream/master</div><div>* git review</div><div><br></div><div>Then if you need to rebase the secondary patch, after rebasing your initial_changes branch, just rebase against origin/initial_changes, and run your git review again.</div><div><br></div><div>This way, you can chain changes, and not have to duplicate work. As the reviews happen and you make changes, you just keep committing against the same change ID (which happens when you git commit -a --amend; git review), and eventually things get merged down from the initial branch to your secondary, tertiary, etc branches.</div><div><br></div><div>Hope that helps. Let me know if you need any additional help...</div><div><br></div><div>Leif.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Aug 23, 2016 at 5:45 AM, Joshua Colp <span dir="ltr"><<a href="mailto:jcolp@digium.com" target="_blank">jcolp@digium.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">Dennis Guse wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hey,<br>
<br>
we adjusted the patch set (fftw3 is now a ifdef-ed).<br>
Precisely, these two commits have been modified (basically adding HAVE_FFTW3)<br>
<br>
<a href="https://gerrit.asterisk.org/#/c/3522/" rel="noreferrer" target="_blank">https://gerrit.asterisk.org/#/<wbr>c/3522/</a><br>
<a href="https://gerrit.asterisk.org/#/c/3524/" rel="noreferrer" target="_blank">https://gerrit.asterisk.org/#/<wbr>c/3524/</a><br>
<br>
Two questions:<br>
1. It seems that we were working outside the commit tree by adjusting<br>
3522 and 3524 - we pulled each one down using _git review -d XXXX_,<br>
applied the modifications, and submitted using _git review_<br>
Is it necessary to also "re-commit" the subsequent patches (or does<br>
gerrit this automatically)?<br>
And if so, how?<br>
At the moment Gerrit does not try to rebuild the subsequent patches<br>
(i.e., 3523 and 3525).<br>
</blockquote>
<br></span>
Gerrit does not automatically do this, it's up to you to rebase the patches so that a review is rebased on the latest version of the one it is dependent on. You can either do this from the web interface using the Rebase button (if it will cleanly rebase) or you have to build up a patch series on your system. This can be done by creating a local branch and pulling down each patch into it using (git review -x <review>), making any changes needed, committing, and doing git review again. This provides the dependency information to Gerrit it needs.<span class=""><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
2. How can we withdraw the OPUS patch from the patch set?<br>
<a href="https://gerrit.asterisk.org/#/c/3526/" rel="noreferrer" target="_blank">https://gerrit.asterisk.org/#/<wbr>c/3526/</a><br>
</blockquote>
<br></span>
You can click "Abandon" from the web interface and it will abandon that specific review.<br>
<br>
Cheers,<span class="im HOEnZb"><br>
<br>
-- <br>
Joshua Colp<br>
Digium, Inc. | Senior Software Developer<br>
445 Jan Davis Drive NW - Huntsville, AL 35806 - US<br>
Check us out at: <a href="http://www.digium.com" rel="noreferrer" target="_blank">www.digium.com</a> & <a href="http://www.asterisk.org" rel="noreferrer" target="_blank">www.asterisk.org</a><br>
<br>
<br></span><div class="HOEnZb"><div class="h5">
-- <br>
______________________________<wbr>______________________________<wbr>_________<br>
-- Bandwidth and Colocation Provided by <a href="http://www.api-digital.com" rel="noreferrer" target="_blank">http://www.api-digital.com</a> --<br>
<br>
asterisk-dev mailing list<br>
To UNSUBSCRIBE or update options visit:<br>
  <a href="http://lists.digium.com/mailman/listinfo/asterisk-dev" rel="noreferrer" target="_blank">http://lists.digium.com/mailma<wbr>n/listinfo/asterisk-dev</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature">Leif Madsen<br><a href="http://www.oreilly.com/catalog/asterisk" target="_blank">http://www.oreilly.com/catalog/asterisk</a><br><a href="http://www.leifmadsen.com" target="_blank">http://www.leifmadsen.com</a></div>
</div>