----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3408/#review11447 -----------------------------------------------------------
Ship it! Ship It! - Matt Jordan On March 30, 2014, 2:05 p.m., Corey Farrell wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/3408/ > ----------------------------------------------------------- > > (Updated March 30, 2014, 2:05 p.m.) > > > Review request for Asterisk Developers. > > > Bugs: ASTERISK-23559 > https://issues.asterisk.org/jira/browse/ASTERISK-23559 > > > Repository: Asterisk > > > Description > ------- > > ASTERISK-23391 caused a regression in Asterisk 11. defaultlanguage is used as > a fall-back when no channel is available. This variable is listed in > options.h, allowing compile to succeed. This caused a run-time symbol failure > because defaultlanguage is not listed in main/asterisk.exports.in. > > This fix adds the variable to exports for Asterisk 11 only. I'm not against > renaming the variable to ast_defaultlanguage in 11 if that is preferred. The > function wasn't in Asterisk 1.8 so no change was made there, Asterisk 12 > renames the variable to ast_defaultlanguage so it was not a problem in 12+. > > > Diffs > ----- > > /branches/11/utils/extconf.c 411572 > /branches/11/main/channel.c 411572 > /branches/11/main/asterisk.c 411572 > /branches/11/include/asterisk/options.h 411572 > /branches/11/apps/app_voicemail.c 411572 > > Diff: https://reviewboard.asterisk.org/r/3408/diff/ > > > Testing > ------- > > Verified this corrects the symbol failure. > > > Thanks, > > Corey Farrell > >
-- _____________________________________________________________________ -- 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
