----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3060/ -----------------------------------------------------------
(Updated Dec. 9, 2013, 4:52 p.m.) Review request for Asterisk Developers and Matt Jordan. Changes ------- Moved the append to modules.conf code to happen right after the default configs are installed. This way it only appends the base default conf file and if it needs to be overriden it can be in the specific test that needs that behaviour. Bugs: ASTERISK-22887 https://issues.asterisk.org/jira/browse/ASTERISK-22887 Repository: testsuite Description ------- While chan_sip and chan_pjsip can be loaded and used at the same time it was causing problems in the testsuite because their defaults try to bind to the same host:port. So for the testsuite made is so any test that depends on res_pjsip will "noload" chan_sip. Instead of manipulating every current res_pjsip test (through their own modules.conf or config setting), this fix checks to see if the test requires/depends on res_pjsip and if it does it appends a noload => chan_sip.so to the copied modules.conf for that test. This functionality is configured via the newly added conflicts.txt file. Any new conflicts that may arise hopefully can resuse the mechanism. Diffs (updated) ----- asterisk/trunk/lib/python/asterisk/asterisk.py 4415 asterisk/trunk/lib/python/asterisk/TestConfig.py 4415 asterisk/trunk/lib/python/asterisk/TestCase.py 4415 asterisk/trunk/configs/modules.conf PRE-CREATION asterisk/trunk/configs/conflicts.txt PRE-CREATION Diff: https://reviewboard.asterisk.org/r/3060/diff/ Testing ------- Thanks, Kevin Harwell
-- _____________________________________________________________________ -- 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