<p style="white-space: pre-wrap; word-wrap: break-word;">We had a bit of a discussion in asterisk-dev about allowing tests written in languages other than python. Historically this was allowed, but the consensus is now they should be strictly kept to python (if being submitted to the main code base) unless otherwise required (and explained why). However, if written in something other than python then there should be no external module dependencies that need to be installed for the test for a given language. This will greatly cut down on the number of potential dependencies required to execute the testsuite.</p><p style="white-space: pre-wrap; word-wrap: break-word;">For instance, this test requires the Text::CSV module to be installed in order to execute/pass.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Unfortunately to move forward with this, this test will either need to be:</p><p style="white-space: pre-wrap; word-wrap: break-word;">1. rewritten to just use python (the preferred option)<br>2. written to not be dependent on any external perl module</p><p style="white-space: pre-wrap; word-wrap: break-word;">Sorry about all this as I know this test was already accepted for submission and this is just an update to fix another problem.</p><p>Patch set 1:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #ffd4d4;">Code-Review -2</span></p><p><a href="https://gerrit.asterisk.org/9379">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.asterisk.org/9379">change 9379</a>. To unsubscribe, or for help writing mail filters, 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/9379"/><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: Ia73ef502f72bd75522d4b6656f6070807dbef349 </div>
<div style="display:none"> Gerrit-Change-Number: 9379 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Torrey Searle <tsearle@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Kevin Harwell <kharwell@digium.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 18 Jul 2018 17:43:32 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-HasLabels: Yes </div>