-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/3198/
-----------------------------------------------------------
(Updated Feb. 12, 2014, 8:40 a.m.)
Status
------
This change has been marked as submitted.
Review request for Asterisk Developers.
Repository: testsuite
Description
-------
While testing a patch to the testsuite, I noticed this ugliness:
rm: cannot remove ‘/var/run/asterisk/asterisk.ctl’: Permission denied
rm: cannot remove ‘/var/run/asterisk/asterisk.pid’: Permission denied
Which is caused by:
# XXX TODO Hard coded path, gross.
os.system("rm -f /var/run/asterisk/asterisk.ctl")
os.system("rm -f /var/run/asterisk/asterisk.pid")
The hardcoded path is ugly, but not problematic. The stale files cleanup is
just a friendly feature and not necessary.
The problem, however, was that I wasn't 'root', and some test cases are bound
to fail soon because there is still a running asterisk out there.
This patch fixes so the test suite dies if there is a running asterisk which
cannot be killed.
Diffs
-----
/asterisk/trunk/runtests.py 4675
Diff: https://reviewboard.asterisk.org/r/3198/diff/
Testing
-------
Yes. If there is an asterisk running that cannot be killed, the suite dies.
Thanks,
wdoekes
--
_____________________________________________________________________
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
To UNSUBSCRIBE or update options visit:
http://lists.digium.com/mailman/listinfo/asterisk-dev
