> On Aug. 5, 2013, 10:45 a.m., wdoekes wrote: > > Can't reproduce. > > > > My asterisk-trunk skips [directories](!). > > I broke a path, and it starts only when I added "(!)", which leads me to > > believe that it doesn't read it then.
Yeah, I can't reproduce this anymore. *shrug* I'll just close for now - Russell ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/2684/#review9325 ----------------------------------------------------------- On July 18, 2013, 9:44 a.m., Russell Bryant wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/2684/ > ----------------------------------------------------------- > > (Updated July 18, 2013, 9:44 a.m.) > > > Review request for Asterisk Developers. > > > Repository: Asterisk > > > Description > ------- > > While working on a deployment, I had to change the [directories] section of > asterisk.conf from the defaults. That worked. Later I noticed that the > directories section was defined as a template-only section like so: > > [directories](!) > > which means my changes should *not* have taken effect. This one line change > fixes that. > > As a side note, while looking at this, I noticed multiple cases of comparing > against a category name like this throughout the file, which seems wrong: > > from ast_category_delete() > > if (cat->name == category) { > > from ast_variable_browse() > > if (config->last_browse && (config->last_browse->name == category)) { > > etc. > > > Diffs > ----- > > /trunk/main/config.c 394685 > > Diff: https://reviewboard.asterisk.org/r/2684/diff/ > > > Testing > ------- > > > Thanks, > > Russell Bryant > >
-- _____________________________________________________________________ -- 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