[Asterisk-code-review] Add missing OPTIONAL API and ARI dependences. (asterisk[master])

[Richard Mudgett]

Richard Mudgett has posted comments on this change. ( https://gerrit.asterisk.org/8007 )

Change subject: Add missing OPTIONAL_API and ARI dependences.
......................................................................


Patch Set 1: Code-Review-1

(1 comment)

https://gerrit.asterisk.org/#/c/8007/1/res/res_statsd.c
File res/res_statsd.c:

https://gerrit.asterisk.org/#/c/8007/1/res/res_statsd.c@384
PS1, Line 384: AST_MODULE_INFO(ASTERISK_GPL_KEY, AST_MODFLAG_GLOBAL_SYMBOLS, "Statsd client support",
             : 	.support_level = AST_MODULE_SUPPORT_EXTENDED,
             : 	.load = load_module,
             : 	.unload = unload_module,
             : 	.reload = reload_module,
Should revert this change.

The .load_pri is now the default of 128.  There are current optional and required users of this module with lower priorities.  A higher priority just makes it take more effort to find this dependency to start it before a load_pri of 128 would normally start it.



-- 
To view, visit https://gerrit.asterisk.org/8007
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia0c70571f5566784f63605e78e1ceccb4f79c606
Gerrit-Change-Number: 8007
Gerrit-PatchSet: 1
Gerrit-Owner: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: Jenkins2
Gerrit-Reviewer: Richard Mudgett <rmudgett at digium.com>
Gerrit-Comment-Date: Sun, 21 Jan 2018 19:49:46 +0000
Gerrit-HasComments: Yes
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20180121/5cd891dc/attachment.html>