[asterisk-dev] [Code Review] Change per-file debug and verbose levels to be per-module, the way users expect them to work.

[Kevin Fleming]

> On 2010-03-19 11:02:15, Russell Bryant wrote:
> > Nice work.  :-)  Should we make this allow the ".so" extension?

Well, we could. For 1.8, I'm kind of thinking we can just drop support in CLI commands for '.so' extensions completely, since they haven't been required for some time now (although we could keep it for modules.conf to avoid breaking existing config files). If we agree that's reasonable, then I'd rather not modify this patch to add support for them.


- Kevin


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/574/#review1708
-----------------------------------------------------------


On 2010-03-22 11:33:34, Kevin Fleming wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/574/
> -----------------------------------------------------------
> 
> (Updated 2010-03-22 11:33:34)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Summary
> -------
> 
> 'core set debug' and 'core set verbose' can optionally change the level for a specific filename; however, this is actually for a specific source file name, not the module that source file is included in. With examples like chan_sip, chan_iax2, chan_misdn and others consisting of multiple source files, this will not lead to the behavior that users expect. If they want to set the debug level for chan_sip, they want it set for all of chan_sip, and not to have to also set it for reqresp_parser and other files that comprise the chan_sip module.
> 
> This patch changes this functionality to be module-name based instead of file-name based.
> 
> To make this work, some Makefile modifications were required to ensure that the AST_MODULE definition is present in each object file produced for each module as well. 
> 
> 
> Diffs
> -----
> 
>   /trunk/Makefile.moddir_rules 253530 
>   /trunk/channels/Makefile 253530 
>   /trunk/codecs/Makefile 253530 
>   /trunk/include/asterisk/logger.h 253530 
>   /trunk/include/asterisk/options.h 253530 
>   /trunk/main/Makefile 253530 
>   /trunk/main/cli.c 253530 
>   /trunk/pbx/Makefile 253530 
>   /trunk/res/Makefile 253530 
> 
> Diff: https://reviewboard.asterisk.org/r/574/diff
> 
> 
> Testing
> -------
> 
> Compile testing, and basic CLI sanity checking.
> 
> 
> Thanks,
> 
> Kevin
> 
>