<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Patch Set 2:</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Patch Set 2:</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Patch Set 2:</p><p style="white-space: pre-wrap; word-wrap: break-word;">(3 comments)</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Patch Set 2: Code-Review-2</p><p style="white-space: pre-wrap; word-wrap: break-word;">This review can't be cleanly cherry-picked to the other branches.  You'll need to cherry-pick to 16 locally and figure out the conflicts.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">I'm a little confused. You ask to cherry-pick manually but this was already done for 16, 18 and master branches? I also noticed those builds are unstable but I'm not sure what that means... Jenkins shows two failed pjsip tests and one failed AMI connection. I'm not sure how those are related to a chan_sip change? What should I do?</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">It was unclear that this was a backported change from the other branches. There's no need to worry about those, then.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Thanks, I just noticed that too. (I got confused because I mixed up app_followme changes with chan_sip changes, sorry). It appears Tzafrir already submitted this change as https://gerrit.asterisk.org/c/asterisk/+/2843/2 but we have something different in our build. This is why this change is colliding with 16 and forwards. Starting to see a pattern here. Trying to decide out whether to backport his change or abandon this change.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Ok, decided to backport Tzafrir's patch. His patch has been merged since Asterisk 14 so there's no need to cherry-pick for 16 and others.</p><p><a href="https://gerrit.asterisk.org/c/asterisk/+/14696">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/14696">change 14696</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/c/asterisk/+/14696"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 13 </div>
<div style="display:none"> Gerrit-Change-Id: Ic15a2bac4f16d0fce7a1b5e7b375f9bafee37aa4 </div>
<div style="display:none"> Gerrit-Change-Number: 14696 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: Dennis <dennis.buteyn@xorcom.com> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-Reviewer: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Joshua Colp <jcolp@sangoma.com> </div>
<div style="display:none"> Gerrit-Reviewer: Kevin Harwell <kharwell@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Tzafrir Cohen <tzafrir.cohen@xorcom.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 19 Aug 2020 11:30:31 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>