[asterisk-dev] [Code Review] Separate verbose level for logging

Matt Jordan reviewboard at asterisk.org
Thu Dec 22 10:30:20 CST 2011


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

Ship it!


Thanks for tracking down the issue with the previous version - looks good

- Matt


On Dec. 21, 2011, 2:14 p.m., Tilghman Lesher wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/1599/
> -----------------------------------------------------------
> 
> (Updated Dec. 21, 2011, 2:14 p.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Summary
> -------
> 
> One problem that many admins have today is that to have maximum verbosity going to the logs, for later debugging of trouble calls, the console becomes virtually unusable, because the verbosity level is global.  This patch attempts to solve that problem, allowing each verbose recipient to have its own filter as to what it wants to receive.
> 
> 
> Diffs
> -----
> 
>   /trunk/CHANGES 348792 
>   /trunk/UPGRADE.txt 348792 
>   /trunk/configs/logger.conf.sample 348792 
>   /trunk/include/asterisk/logger.h 348792 
>   /trunk/main/asterisk.c 348792 
>   /trunk/main/cli.c 348792 
>   /trunk/main/config.c 348792 
>   /trunk/main/loader.c 348792 
>   /trunk/main/logger.c 348792 
>   /trunk/main/stdtime/localtime.c 348792 
>   /trunk/utils/ael_main.c 348792 
>   /trunk/utils/hashtest.c 348792 
>   /trunk/utils/hashtest2.c 348792 
>   /trunk/utils/refcounter.c 348792 
> 
> Diff: https://reviewboard.asterisk.org/r/1599/diff
> 
> 
> Testing
> -------
> 
> Verified logging with files, consoles with different verbosity levels, etc.
> 
> 
> Thanks,
> 
> Tilghman
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20111222/283954f0/attachment.htm>


More information about the asterisk-dev mailing list