[asterisk-dev] [Code Review] 2781: config_options: Drop missing config file ERROR statement to a WARNING
Matt Jordan
reviewboard at asterisk.org
Wed Aug 21 14:42:04 CDT 2013
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/2781/
-----------------------------------------------------------
Review request for Asterisk Developers.
Repository: Asterisk
Description
-------
One liner, but it is worth discussing.
Currently, if config_options doesn't detect a config file for a module, it emits an ERROR.
That seems a bit excessive, particularly for config files processed by the Asterisk core. It is not an error to be missing acl.conf - you just won't have ACLs. That feels like a WARNING at best to me.
Objections?
Diffs
-----
/trunk/main/config_options.c 397306
Diff: https://reviewboard.asterisk.org/r/2781/diff/
Testing
-------
Thanks,
Matt Jordan
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20130821/1eb379a0/attachment.htm>
More information about the asterisk-dev
mailing list