<p>Matthew Fredrickson <strong>posted comments</strong> on this change.</p><p><a href="https://gerrit.asterisk.org/8398">View Change</a></p><p>Patch set 2:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #ffd4d4;">Code-Review -1</span></p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">The option could be very interesting regardless.  Unfortunately,<br>project merge policy dictates a test needs to be added to put this<br>in a branch other than master.  You'll need a testsuite test to put<br>this in 13 or 15.  Sorry about the trouble :-(</p><p style="white-space: pre-wrap; word-wrap: break-word;">Matthew Fredrickson</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">After more thoughtful consideration, I've changed my mind.  I think that it's arguable that it's a bug and that should have worked before regardless.  I'd be ok with this going in release branches as long as we put something UPGRADE.txt saying that this behavior might change for some people.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Matthew Fredrickson</p><ul style="list-style: none; padding-left: 20px;"></ul><p>To view, visit <a href="https://gerrit.asterisk.org/8398">change 8398</a>. To unsubscribe, 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/8398"/><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-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I9842e1dc2f6e1c2c49dc33fe615255007d2f821e </div>
<div style="display:none"> Gerrit-Change-Number: 8398 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: Florian Floimair <f.floimair@commend.com> </div>
<div style="display:none"> Gerrit-Reviewer: Florian Floimair <f.floimair@commend.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-Comment-Date: Fri, 09 Mar 2018 15:27:38 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>