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

[Corey Farrell]

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 3:

> Greetings!
 > 
 > Might I suggest using something like bindep[1] to express your OS
 > dependencies in a single location, eg: bindep.txt.  Then, your
 > automation could be writing around it for each distro. As an
 > example, here is how we support dependencies with zuul[2]. Here is
 > an ansible role you could also use[3]
 > 
 > [1] https://docs.openstack.org/infra/bindep/readme.html
 > [2] http://git.openstack.org/cgit/openstack-infra/zuul/tree/bindep.txt
 > [3] http://git.openstack.org/cgit/openstack-infra/zuul-jobs/tree/roles/bindep

The point of install_prereq is for end-users to be able to run it on target machines to install what is required to build Asterisk.  It looks like end-users would have to install bindep before bindep.txt could be used?  What I mean is I'm not seeing anything about generating shell script to be used by end-users.  Kind of like how you don't need to install autoconf to run ./configure, autoconf is only required to generate the configure script from configure.ac.


-- 
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: 3
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: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Jenkins2
Gerrit-Reviewer: Paul Belanger <pabelanger at gmail.com>
Gerrit-Comment-Date: Wed, 21 Mar 2018 23:16:59 +0000
Gerrit-HasComments: No
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20180321/ab12603a/attachment-0001.html>