<p> Attention is currently required from: N A, Sarah Autumn. </p>
<p><a href="https://gerrit.asterisk.org/c/asterisk/+/17948">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="null">Patchset:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/17948?tab=comments">Patch Set #4:</a> </p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">it requires separate modifications to /etc/dahdi/system.conf and /etc/asterisk/chan_dahdi.conf (not to mention an upcoming patch to the DAHDI kernel driver),</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Admittedly I know very little about DAHDI, but I +1'd this as the code is simply enough and looked good to me. However, given the above it might be this code will need to wait to be merged until the other dependent patches have been merged into their respective repo's.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Or perhaps version checks could be put in place? At the very least maybe a mentioning of version dependencies somewhere? Maybe in the CHANGES section and/or the commit log.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Unless of course you're thinking the DAHDI changes may be not go in for a very long time, if at all and ppl will just manually apply patches themselves. Then I'm not sure. Maybe a mention/link to the other issues?</p><p style="white-space: pre-wrap; word-wrap: break-word;">Maybe someone else has some other ideas...</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/17948">change 17948</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/+/17948"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 18 </div>
<div style="display:none"> Gerrit-Change-Id: Ibcef6e21020ab6e64f16f4761ec1e0e7fce9c260 </div>
<div style="display:none"> Gerrit-Change-Number: 17948 </div>
<div style="display:none"> Gerrit-PatchSet: 5 </div>
<div style="display:none"> Gerrit-Owner: Sarah Autumn <sarah@endlesstemple.org> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-Reviewer: Kevin Harwell <kharwell@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: N A <mail@interlinked.x10host.com> </div>
<div style="display:none"> Gerrit-Attention: N A <mail@interlinked.x10host.com> </div>
<div style="display:none"> Gerrit-Attention: Sarah Autumn <sarah@endlesstemple.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 23 Feb 2022 22:16:04 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Comment-In-Reply-To: N A <mail@interlinked.x10host.com> </div>
<div style="display:none"> Comment-In-Reply-To: Sarah Autumn <sarah@endlesstemple.org> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>