[Asterisk-code-review] BuildSystem: Go for Python X.Y explicitly. (asterisk[master])

Alexander Traud asteriskteam at digium.com
Thu Apr 19 04:12:32 CDT 2018


Hello Jenkins2, Corey Farrell, 

I'd like you to reexamine a change. Please visit

    https://gerrit.asterisk.org/8673

to look at the new patch set (#2).

Change subject: BuildSystem: Go for Python X.Y explicitly.
......................................................................

BuildSystem: Go for Python X.Y explicitly.

With recent changes, the scripts used in the build system work with Python 3.6
now but remain compatible with Python 2.7. In such a case, PEP 394 recommends
to go for just 'python'. However, several platforms do not have 'python', for
example NetBSD and DragonFly BSD. Ubuntu 18.04 LTS does not come with 'python'
on default either. One approach would be to go for 'python3' and then fallback
to 'python2'. Again, NetBSD and DragonFly BSD do not offer this. Furthermore
with such an approach, Asterisk might use an untested Python version.

Until the Python community comes up with a more streamlined user experienced
(not only for the Shebang but also the tools env and pkg-config), Asterisk looks
for the still supported versions explicitly. Whenever a newer Python version is
released, it should be tested and then added to that list.

When it comes to the Test Framework, Asterisk still supports just Python 2, see
ASTERISK-27787. Therefore, to avoid Python 3 on those platforms which link to
that version on default already (Arch Linux and Gentoo Linux), pick Python 2.7
explicitly.

ASTERISK-27717

Change-Id: I29e694dc7e92510dc26b15895b55f78f67146b2c
---
M configure
M configure.ac
M contrib/scripts/install_prereq
3 files changed, 21 insertions(+), 16 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/73/8673/2
-- 
To view, visit https://gerrit.asterisk.org/8673
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I29e694dc7e92510dc26b15895b55f78f67146b2c
Gerrit-Change-Number: 8673
Gerrit-PatchSet: 2
Gerrit-Owner: Alexander Traud <pabstraud at compuserve.com>
Gerrit-Reviewer: Alexander Traud <pabstraud at compuserve.com>
Gerrit-Reviewer: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: Jenkins2
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20180419/8bb780ce/attachment.html>


More information about the asterisk-code-review mailing list