<p>Corey Farrell <strong>posted comments</strong> on this change.</p><p><a href="https://gerrit.asterisk.org/7873">View Change</a></p><p>Patch set 3:</p><p>(2 comments)</p><ul style="list-style: none; padding-left: 20px;"><li><p><a href="https://gerrit.asterisk.org/#/c/7873/3/main/loader.c">File main/loader.c:</a></p><ul style="list-style: none; padding-left: 20px;"><li><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/7873/3/main/loader.c@234">Patch Set #3, Line 234:</a> <code style="font-family:monospace,monospace"> * \param missing Vector to store name of any enhancer that is not running or declined.</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">I'm not sure if the help text is wrong, the code or I am just all around mi</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">This is intentional.  'missing' contains only modules which must be running before 'mod' can be started.  An enhancer is only required if it not declined.  Some modules decline load they are disabled by configuration, in that case the enhancer will never load.</p><p style="white-space: pre-wrap; word-wrap: break-word;">This doesn't make a difference with the current dependencies that are added (enhancers aren't widely used yet).  A potential future use case is if res_hep_pjsip had '.enhances = "res_hep,res_pjsip"' instead of '.requires'.  If res_hep is disabled by configuration res_hep_pjsip would decline.  chan_pjsip already requires res_pjsip, so the system would try starting res_hep_pjsip after res_hep / res_pjsip but before chan_pjsip.</p></li><li><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/7873/3/main/loader.c@1487">Patch Set #3, Line 1487:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><pre style="font-family: monospace,monospace; white-space: pre-wrap;">              if (module_deps_reference(mod, NULL)) {<br>                       ast_log(LOG_WARNING, "Module '%s' could not be loaded due to missing dependencies.\n", resource_name);<br>                      unload_dynamic_module(mod);<br><br>                 return required ? AST_MODULE_LOAD_FAILURE : AST_MODULE_LOAD_DECLINE;<br>          }<br></pre></blockquote></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Not sure if this would be the best place to do it, but do you think it woul</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Actually start_resource already does that, so this block can just be removed.  I'm not sure why I left this here, might have been to allow me to deal with the required argument but that's unneeded.  required is never true when resource_heap == NULL.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/7873">change 7873</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/7873"/><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: I9be08d1dd331aceadc1dcba00b804d71360b2fbb </div>
<div style="display:none"> Gerrit-Change-Number: 7873 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: Corey Farrell <git@cfware.com> </div>
<div style="display:none"> Gerrit-Reviewer: Corey Farrell <git@cfware.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Kevin Harwell <kharwell@digium.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 11 Jan 2018 23:55:53 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>