<div dir="ltr"><div dir="ltr">On Thu, Sep 1, 2022 at 10:14 AM <<a href="mailto:asterisk@phreaknet.org">asterisk@phreaknet.org</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 9/1/2022 7:53 AM, Joshua C. Colp wrote:<br>
> On Thu, Sep 1, 2022 at 8:07 AM <<a href="mailto:asterisk@phreaknet.org" target="_blank">asterisk@phreaknet.org</a> <br>
> <mailto:<a href="mailto:asterisk@phreaknet.org" target="_blank">asterisk@phreaknet.org</a>>> wrote:<br>
><br>
>     On 8/31/2022 7:47 PM, Michael Bradeen wrote:<br>
>     > Hello again!<br>
>     ><br>
>     > The Python3 release of the testsuite has been merged. The wiki has<br>
>     > been updated and will continue to update as things change.<br>
>     ><br>
>     > As indicated before, with this change running the testsuite on<br>
>     Python2<br>
>     > is no longer supported.  Please check the wiki for any questions<br>
>     and<br>
>     > reply on the forum or this email with questions that come up!<br>
>     Thanks for the update Mike, and all the work involved in this,<br>
><br>
>         I was able to get the runInVenv.sh script to finally run last<br>
>     night,<br>
>     after installing a few additional Debian packages (python3-dev and<br>
>     python3-venv)<br>
>     (I wonder if an install_prereqs.sh for the test suite might be<br>
>     useful at<br>
>     some point, but that's for a different day)<br>
><br>
><br>
> There already is one in the same place as the Asterisk one, <br>
> contrib/scripts/install_prereq<br>
Ah, thanks for pointing that out ;)<br>
<br>
That script fails for me because it assumes sudo is installed, and the <br>
test system I ran it on didn't have sudo. I'm not sure if that's a <br>
reasonable assumption to make, either. I think it's more reasonable to <br>
have the install process run install_prereq itself as sudo, as opposed <br>
to making sudo calls within the script. The regular Asterisk <br>
install_prereq script doesn't have this sudo dependency.<br>
<br>
I imagine this type of thing is typically run as root or sudo anyways, <br>
and if the script needs to ensure it's being run as root (even though <br>
this is the user's responsibility), it could  check the UID and abort if <br>
it's not 0.<br>
Any objection to removing the sudo calls from install_prereq? It's <br>
unnecessary and causes breakage.<br></blockquote><div><br></div><div>Making it work equivalent to the Asterisk one is fine. </div></div><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div style="font-family:tahoma,sans-serif"><font color="#073763">Joshua C. Colp</font></div><div style="font-family:tahoma,sans-serif"><font color="#073763">Asterisk Project Lead</font></div><div style="font-family:tahoma,sans-serif"><font color="#073763">Sangoma Technologies</font></div><div style="font-family:tahoma,sans-serif"><font color="#073763">Check us out at <a href="http://www.sangoma.com" target="_blank">www.sangoma.com</a> and <a href="http://www.asterisk.org" target="_blank">www.asterisk.org</a></font><br></div></div></div></div></div></div></div></div></div></div></div>