<p>Corey Farrell <strong>posted comments</strong> on this change.</p><p><a href="https://gerrit.asterisk.org/8516">View Change</a></p><p>Patch set 2:</p><p style="white-space: pre-wrap; word-wrap: break-word;">This will still need to be rebased when 8507 is merged.  The added packages from 8507 are included here but this will need to modified to delete the added packages from install_prereq.  I still wanted to post it now to get thoughts on the updated way of adding packages.</p><p style="white-space: pre-wrap; word-wrap: break-word;">I feel this way is better as it will allow logic to be used if needed.  For example checking $ID and/or $VERSION_ID to decide if certain packages should be included.</p><ul style="list-style: none; padding-left: 20px;"></ul><p>To view, visit <a href="https://gerrit.asterisk.org/8516">change 8516</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/8516"/><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: I16a26b19047a8b3da3947c7953cd104241b7472d </div>
<div style="display:none"> Gerrit-Change-Number: 8516 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: Corey Farrell <git@cfware.com> </div>
<div style="display:none"> Gerrit-Reviewer: Alexander Traud <pabstraud@compuserve.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-Comment-Date: Wed, 14 Mar 2018 11:58:10 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>