> On Jan. 24, 2014, 12:46 p.m., Paul Belanger wrote: > > Okay, so I see what you are saying. I just ran the scripts and they are > > broken. So, with that in mind I think we need two changesets, one that > > fixes the current alembic migration paths. And the seconds which adds your > > logic. > > > > Lastly, this we need an automate testcase for this, as it is obvious this > > code was checked in broken from the start.
"Lastly, this we need an automate testcase for this, as it is obvious this code was checked in broken from the start." That'd be nice, but it's a bit more work than I'm going to sign up for at this point. If someone would like to contribute that as an automated test, however, it would be hugely appreciated. - Matt ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3148/#review10685 ----------------------------------------------------------- On Jan. 24, 2014, 11:46 a.m., Kevin Harwell wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/3148/ > ----------------------------------------------------------- > > (Updated Jan. 24, 2014, 11:46 a.m.) > > > Review request for Asterisk Developers. > > > Bugs: ASTERISK-23038 > https://issues.asterisk.org/jira/browse/ASTERISK-23038 > > > Repository: Asterisk > > > Description > ------- > > Added a "debug" configuration option for res_pjsip that when set to "yes" > enables SIP messages to be logged. It is specified under the "system" type. > > Also updated the alembic 12.1 script to include this option as well as a few > others that were missing. Also updated the "_adding_extensions" script in > order to make the "id" column on the table a primary key because mysql needed > it to be as such. > > > Diffs > ----- > > branches/12/res/res_pjsip_logger.c 406340 > branches/12/res/res_pjsip/config_global.c 406340 > branches/12/res/res_pjsip.c 406340 > branches/12/include/asterisk/res_pjsip.h 406340 > > branches/12/contrib/ast-db-manage/config/versions/581a4264e537_adding_extensions.py > 406340 > > branches/12/contrib/ast-db-manage/config/versions/2fc7930b41b3_add_pjsip_endpoint_options_for_12_1.py > 406340 > branches/12/configs/pjsip.conf.sample 406340 > > Diff: https://reviewboard.asterisk.org/r/3148/diff/ > > > Testing > ------- > > Set the "debug" option in the pjsip.conf file and observed SIP debug messages > on the console. Also, tested the modified alembic scripts against postgres > and mysql database servers. > > > 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
