<p>Corey Farrell <strong>posted comments</strong> on this change.</p><p><a href="https://gerrit.asterisk.org/8673">View Change</a></p><p>Patch set 1:</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">What exactly is blocking this?<br>Gerrit 8646 is far away from adding Python 3 support, see the PJSUA<br>part. Even if it does, the tool Python must be found differently.<br>For example, Ubuntu 18.04 LTS comes with Python 3 on default, but<br>does not have a ‘python’ anymore; except you install Python 2.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">8646 makes the build system completely compatible with python2 and python3 tools which are not used to build the PJSUA component (see below for comments about PYTHONDEV).  The python tool detected by configure.ac is only used by 'make full' and 'make ari-stubs'.  It would be nice if you could give 8646 a review as it actually fixes the issue with the current version of python (3.x).  We should not go back and forth with the configure script, get it right the first time.  IMO the order of preference for python tools should be: python, python3, python2, python3.6, python2.7.  This will make us use the default per OS (/usr/bin/python), or if none is default it will try python3 (current) followed by python2 (legacy).</p><p style="white-space: pre-wrap; word-wrap: break-word;">As for moving forward with PYTHONDEV v3, see ASTERISK-27787.  This includes patches that add pjsua2 (compatible with py2 and py3) but for this to be useful the testsuite will have to switch from pjsua to pjsua2 python modules (they're not source compatible).  So I have no problem with your PYTHONDEV changes as nobody has started work on using pjsua2.  But this has nothing to do with what version /usr/bin/python gives us as we do not use 'python setup.py' to build the pjsua python bindings.</p><ul style="list-style: none; padding-left: 20px;"></ul><p>To view, visit <a href="https://gerrit.asterisk.org/8673">change 8673</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/8673"/><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: I29e694dc7e92510dc26b15895b55f78f67146b2c </div>
<div style="display:none"> Gerrit-Change-Number: 8673 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Alexander Traud <pabstraud@compuserve.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, 18 Apr 2018 13:53:42 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>