<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Apr 13, 2015 at 6:52 PM, Matthew Jordan <span dir="ltr"><<a href="mailto:mjordan@digium.com" target="_blank">mjordan@digium.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">For *right now*, we are going to try cherry-picking the changes to the<br>
affected branches when the change is first up for review. This is<br>
clearly a pretty big change in process, as the act of merging into<br>
other branches was (a) always done by those with commit access and (b)<br>
never reviewed. There's at least two good reasons to give this a try:<br>
<br>
(1) There isn't anyone with "commit" access. Anyone can post a review<br>
up to Gerrit. The plus side is that there are far fewer barriers to<br>
getting a patch into Asterisk. The downside is that there isn't a<br>
select group of people who have been trusted to do the merges. The<br>
only way to ensure that patches actually do get merged into all the<br>
branches is to require people to put the patches up with the initial<br>
review.<br>
<br>
(2) Gerrit really, really wants to review things. That's a good thing:<br>
we've had plenty of bad merges take out branches in the past - either<br>
from compilation issues, subtle bugs that creep in due to API<br>
compatibility problems, etc. We've had even more that get merged<br>
upstream and fail to take advantage of APIs that exist in later<br>
versions of Asterisk. Reviews will help to catch that.<br>
<br>
This is a trial. We'll re-evaluate how things are working at the end<br>
of the week.<br>
<br>
Instructions for cherry-picking changes are on a draft wiki page here:<br>
<br>
<a href="https://wiki.asterisk.org/wiki/display/AST/Git+Usage" target="_blank">https://wiki.asterisk.org/wiki/display/AST/Git+Usage</a></blockquote><div><br></div><div>One thing that helps tracking backports is using the same Change-Id for the same patch applied to multiple branches.  You can use that Id to check the status of a single patch across all branches.  For example, I saw this patch that's proposed for both master and 13:</div><div><br></div><div><a href="https://gerrit.asterisk.org/#/q/Id0ce0528e58014da1324856ea537e7765466044a">https://gerrit.asterisk.org/#/q/Id0ce0528e58014da1324856ea537e7765466044a</a></div><div><br></div><div>-- </div><div>Russell Bryant </div></div></div></div>