<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Oct 13, 2014 at 10:15 AM, Matthew Jordan <span dir="ltr"><<a href="mailto:mjordan@digium.com" target="_blank" onclick="window.open('https://mail.google.com/mail/?view=cm&tf=1&to=mjordan@digium.com&cc=&bcc=&su=&body=','_blank','location=yes,menubar=yes,resizable=yes,width=800,height=600');return false;">mjordan@digium.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">On Mon, Oct 13, 2014 at 12:58 AM, Tzafrir Cohen<br>
<<a href="mailto:tzafrir.cohen@xorcom.com" onclick="window.open('https://mail.google.com/mail/?view=cm&tf=1&to=tzafrir.cohen@xorcom.com&cc=&bcc=&su=&body=','_blank','location=yes,menubar=yes,resizable=yes,width=800,height=600');return false;">tzafrir.cohen@xorcom.com</a>> wrote:<br>
> Hi<br>
><br>
> I finally got the asterisk-testsuite accpted into Debian (I know, I need<br>
> to get it updated).<br>
><br>
> And just as it entered Unstable, it already got 3 bug reports on build<br>
> failures: <a href="http://bugs.debian.org/src:asterisk-testsuite" target="_blank">http://bugs.debian.org/src:asterisk-testsuite</a><br>
><br>
> Notable of them is: <a href="https://bugs.debian.org/764906" target="_blank">https://bugs.debian.org/764906</a><br>
><br>
> Basically asttest fails to build on non-linux platforms due to the use<br>
> of <linux/limits.h> (for using e.g. PATH_MAX.<br>
><br>
> What should I do on other platforms?<br>
><br>
> * Try to rewrite the code to avoid using PATH_MAX (for instance, HURD<br>
>   does not have this limit), or:<br>
><br>
> * If the platform is not Linux, don't build asttest and don't run Lua<br>
>   tests (is it used for Python ones as well?)<br>
><br>
<br>
While I'm okay with either option, I think it is important to note<br>
that we (a) almost never write tests in Lua any more (in fact, I'd say<br>
we actively discourage it), and (b) have been migrating the tests from<br>
Lua to Python for some time.<br>
<br>
I don't think you lose much coverage by going with your second option.<br>
<span class=""><font color="#888888"><br></font></span></blockquote><div><br></div><div>Just FYI...  asttest is still used by the following tests:</div><div><br></div><div>tests/channels/SIP/rfc2833_dtmf_detect</div><div>tests/channels/SIP/handle_response_refer</div><div>tests/channels/SIP/tcpauthtimeout<br></div><div>tests/channels/SIP/sip_channel_params<br></div><div>tests/channels/SIP/tcpauthlimit<br></div><div>tests/apps/queues/macro_gosub_test</div><div>tests/apps/queues/ringinuse_and_pause</div><div>tests/apps/queues/ringinuse_and_pause<br></div><div>tests/apps/queues/queue_baseline</div><div>tests/apps/queues/position_priority_maxlen</div><div>tests/apps/queues/wrapup_time</div><div>tests/apps/queues/wrapup_time<br></div><div>tests/fax/sip/local_channel_t38_queryoption</div><div>tests/cdr/app_queue</div><div>tests/cdr/app_dial_G_flag<br></div><div>tests/manager/response-time</div><div>tests/manager/authlimit<br></div><div>tests/manager/authtimeout</div><div>tests/manager/action-events-response<br></div><div>tests/span-alarms<br></div><div> <br></div><div>You're also going to have to do some work on it to get it to even compile on Lua 5.2.</div><div><br></div><div><br></div></div></div></div>