<p>Corey Farrell <strong>posted comments</strong> on this change.</p><p><a href="https://gerrit.asterisk.org/8854">View Change</a></p><p>Patch set 1:</p><p style="white-space: pre-wrap; word-wrap: break-word;">1) This will be rebased once 8691 and 8829 are merged.<br>2) The Jenkins failure will be resolved by merge of 8830.</p><p style="white-space: pre-wrap; word-wrap: break-word;">This is being posted now in the hopes that someone with spare system resources can run the full testsuite with python2.  I've spot-checked a couple of tests, they still work but I would be surprised if nothing is broken by this.</p><p style="white-space: pre-wrap; word-wrap: break-word;">./self_test can be run to execute the unit tests (they all pass for me under python2 and python3).  Note that test_sip_dialog_test_condition is disabled not due to the python2 conversion, sip_dialog_test_condition is already broken in the current code.  The test was renamed with a .txt extension to suppress pep8 warnings from that file.  lib/python/asterisk/self_test/*.py passes pep8 and flake8.</p><p style="white-space: pre-wrap; word-wrap: break-word;">The hope is that this can be merged pretty much as is once it's rebased and verified python2 is not broken, with additional python3 issues to be addressed in follow-up reviews.</p><ul style="list-style: none; padding-left: 20px;"></ul><p>To view, visit <a href="https://gerrit.asterisk.org/8854">change 8854</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/8854"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: testsuite </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: If76c2d3e11e4ab4552d0df7841287c8bb2de7918 </div>
<div style="display:none"> Gerrit-Change-Number: 8854 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Corey Farrell <git@cfware.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: Fri, 20 Apr 2018 22:35:01 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>