<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Although I technically agree with Josh and Michael in that it would<br>seem that ARI is probably not the "proper" interface to send out<br>SIP messages, I find sometimes that it's useful to let people try<br>things out to see if my assumptions about the world are incorrect. <br>My personal opinion (not project lead opinion) is that I would be<br>ok with the module going in as long as it's disabled (doesn't build<br>or isn't active) by default and not core supported.  Anybody else<br>want to weigh in?</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">I have no complaint about making it an option as long as menuselect has it disabled by default.  Probably add the noload to modules.conf installed by 'make samples' with a comment about performance costs.</p><p style="white-space: pre-wrap; word-wrap: break-word;">The CI scripts enable all 'res' modules that can be built so I think we should make sure the testsuite noload's the module to avoid the performance hit.  Letting CI build the module would be best so we can get compile only CI coverage.</p><p style="white-space: pre-wrap; word-wrap: break-word;">I'm not sure about the current name of this module.  Maybe this is more of a res_stasis_* module?</p><p><a href="https://gerrit.asterisk.org/10715">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.asterisk.org/10715">change 10715</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/10715"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I1dedcc678265ebb74d35e7a2f58bc4b5084bcfe1 </div>
<div style="display:none"> Gerrit-Change-Number: 10715 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: sungtae kim <pchero21@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Alexei Gradinari <alex2grad@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Corey Farrell <git@cfware.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 (1000185) </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: Michael L. Young <elgueromexicano@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: sungtae kim <pchero21@gmail.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 28 Nov 2018 17:33:11 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-HasLabels: No </div>