<p><a href="https://gerrit.asterisk.org/9883">View Change</a></p><p>2 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.asterisk.org/#/c/9883/5//COMMIT_MSG">Commit Message:</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/9883/5//COMMIT_MSG@7">Patch Set #5, Line 7:</a> <code style="font-family:monospace,monospace">pbx_config.c: Some modules not be loaded if they initially decline to load</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Change to:</p><p style="white-space: pre-wrap; word-wrap: break-word;">pbx_config.c: Fix reloading module if initially declined to load.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/#/c/9883/5//COMMIT_MSG@9">Patch Set #5, Line 9:</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;">Added exit if extensions.conf file not available<br>on loading pbx_config<br></pre></blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">This statement is not true and should be removed from the commit message.  The second part of the sentence is what was done in the patch.</p><p style="white-space: pre-wrap; word-wrap: break-word;">The commit message is not intended to be a history of the patch itself as that is part of what Gerrit does in the review process.  The commit message is to describe the problem the patch is fixing and an overview of the fix.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/9883">change 9883</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/9883"/><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: Ib00665106043b1be5148ffa7a477396038915854 </div>
<div style="display:none"> Gerrit-Change-Number: 9883 </div>
<div style="display:none"> Gerrit-PatchSet: 5 </div>
<div style="display:none"> Gerrit-Owner: Chris Savinovich <csavinovich@digium.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: Richard Mudgett <rmudgett@digium.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 17 Aug 2018 23:44:02 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>