<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Only putting the change in 16/master sounds fair to me.</p><p style="white-space: pre-wrap; word-wrap: break-word;">I'd be happy to make the change but I'm not sure how to proceed.<br>Should the same note be made in both UPGRADE.txt and CHANGES?</p><p style="white-space: pre-wrap; word-wrap: break-word;">And how to please Gerrit? Start with a cherry-pick to 16, amend<br>with an updated UPGRADE.txt then abandon this original version?</p></blockquote><ul><li>The same note should be put into UPGRADE.txt and CHANGES in the appropriate AMI section for 16.0.0 and 16 respectively.</li><li>The 16 branch version can be cherry-picked to 16.0 in gerrit.</li><li>The master cannot be cherry-picked in gerrit because UPGRADE.txt will conflict.  That conflict is easily resolved as there wouldn't be any change to UPGRADE.txt for master.</li></ul><p style="white-space: pre-wrap; word-wrap: break-word;">FYI:  The ASTERISK issue should be on a line by itself.  The "This fixes" part is redundant.</p><p style="white-space: pre-wrap; word-wrap: break-word;">It would be easier to do the changes locally than in gerrit.</p><ul><li>Create a new branch off of origin/16 (git checkout -b v16_patch --track origin/16)</li><li>cherry-pick your patch to the new branch</li><li>Amend the patch to add the UPGRADE.txt and CHANGES note in the appropriate section for 16.0.0 and 16 respectively.  (git add -u; git commit --amend)</li><li>Create a new branch off of origin/master (git checkout -b master_patch --track origin/master)</li><li>cherry-pick the change from your local 16 branch (git cherry-pick v16_patch)</li><li>Resolve the conflict by removing the change to UPGRADE.txt</li><li>Put both reviews up to gerrit</li><li>Cherry-pick the 16 branch version to 16.0</li><li>Abandon the 15 branch version</li></ul><p style="white-space: pre-wrap; word-wrap: break-word;">See.  Nothing to it. :)</p><p><a href="https://gerrit.asterisk.org/10037">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.asterisk.org/10037">change 10037</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.asterisk.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.asterisk.org/10037"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 15 </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I45b3119f30e4dbc17b49831f2b1a4f2c1beadafe </div>
<div style="display:none"> Gerrit-Change-Number: 10037 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: lvl <digium@lvlconsultancy.nl> </div>
<div style="display:none"> Gerrit-Reviewer: Corey Farrell <git@cfware.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Joshua Colp <jcolp@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Matthew Fredrickson <creslin@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Richard Mudgett <rmudgett@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: lvl <digium@lvlconsultancy.nl> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 06 Sep 2018 22:13:40 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-HasLabels: No </div>