-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4083/
-----------------------------------------------------------
(Updated Oct. 15, 2014, 11:53 a.m.)
Review request for Asterisk Developers.
Changes
-------
Addressed issues: instead of replace now using an explicit comparison in a
utility function.
Repository: testsuite
Description
-------
When a core dump occurs, if the test path started with t, e, or s, those
characters would get stripped out from the path created to store the backtrace
file. This is due to incorrect usage of lstrip, which removes all instances of
any of the provided characters from the start of the string regardless of the
order:
>>> print 'tests/something'.lstrip('tests/')
omething
This patch changes lstrip to replace in several places, where the intent was
clearly to remove only an exact string from another.
Diffs (updated)
-----
/asterisk/trunk/runtests.py 5730
/asterisk/trunk/lib/python/asterisk/sippversion.py 5730
Diff: https://reviewboard.asterisk.org/r/4083/diff/
Testing
-------
Thanks,
Scott Griepentrog
--
_____________________________________________________________________
-- 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
