[Asterisk-code-review] install prereq: Split into multiple scripts and data. (asterisk[master])

Corey Farrell asteriskteam at digium.com
Wed Mar 14 06:58:10 CDT 2018


Corey Farrell has posted comments on this change. ( https://gerrit.asterisk.org/8516 )

Change subject: install_prereq: Split into multiple scripts and data.
......................................................................


Patch Set 2:

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.

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.


-- 
To view, visit https://gerrit.asterisk.org/8516
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I16a26b19047a8b3da3947c7953cd104241b7472d
Gerrit-Change-Number: 8516
Gerrit-PatchSet: 2
Gerrit-Owner: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: Alexander Traud <pabstraud at compuserve.com>
Gerrit-Reviewer: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: Jenkins2
Gerrit-Comment-Date: Wed, 14 Mar 2018 11:58:10 +0000
Gerrit-HasComments: No
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20180314/f9b896cf/attachment.html>


More information about the asterisk-code-review mailing list