diff options
author | Jenkins2 <jenkins2@gerrit.asterisk.org> | 2018-01-26 17:40:56 -0600 |
---|---|---|
committer | Gerrit Code Review <gerrit2@gerrit.digium.api> | 2018-01-26 17:40:56 -0600 |
commit | da5859246db94d9852323b1357401559f162d7ba (patch) | |
tree | 808ec2d895c205b1d45e9f9d610f4a20b077cd04 | |
parent | 7fbf1e32caedf26c9b24b3ba04cae433dd9451b3 (diff) | |
parent | 679fa5fb3445870ad29953c606ab5e26f44e4e22 (diff) |
Merge "Add missing OPTIONAL_API and ARI dependences."
33 files changed, 85 insertions, 27 deletions
diff --git a/apps/app_mixmonitor.c b/apps/app_mixmonitor.c index bb47cfc34..af2949209 100644 --- a/apps/app_mixmonitor.c +++ b/apps/app_mixmonitor.c @@ -1556,4 +1556,9 @@ static int load_module(void) return res; } -AST_MODULE_INFO_STANDARD(ASTERISK_GPL_KEY, "Mixed Audio Monitoring Application"); +AST_MODULE_INFO(ASTERISK_GPL_KEY, AST_MODFLAG_DEFAULT, "Mixed Audio Monitoring Application", + .support_level = AST_MODULE_SUPPORT_CORE, + .load = load_module, + .unload = unload_module, + .optional_modules = "func_periodic_hook", +); diff --git a/apps/app_statsd.c b/apps/app_statsd.c index 54f9511a8..a4ca233ea 100644 --- a/apps/app_statsd.c +++ b/apps/app_statsd.c @@ -420,4 +420,9 @@ static int load_module(void) return ast_register_application_xml(app, statsd_exec); } -AST_MODULE_INFO_STANDARD_EXTENDED(ASTERISK_GPL_KEY, "StatsD Dialplan Application"); +AST_MODULE_INFO(ASTERISK_GPL_KEY, AST_MODFLAG_DEFAULT, "StatsD Dialplan Application", + .support_level = AST_MODULE_SUPPORT_EXTENDED, + .load = load_module, + .unload = unload_module, + .requires = "res_statsd", +); diff --git a/bridges/bridge_builtin_features.c b/bridges/bridge_builtin_features.c index e6cacbf7e..e9efb916c 100644 --- a/bridges/bridge_builtin_features.c +++ b/bridges/bridge_builtin_features.c @@ -519,4 +519,9 @@ static int load_module(void) return AST_MODULE_LOAD_SUCCESS; } -AST_MODULE_INFO_STANDARD(ASTERISK_GPL_KEY, "Built in bridging features"); +AST_MODULE_INFO(ASTERISK_GPL_KEY, AST_MODFLAG_DEFAULT, "Built in bridging features", + .support_level = AST_MODULE_SUPPORT_CORE, + .load = load_module, + .unload = unload_module, + .optional_modules = "res_monitor", +); diff --git a/res/ari/resource_bridges.c b/res/ari/resource_bridges.c index a320c2987..9808288e9 100644 --- a/res/ari/resource_bridges.c +++ b/res/ari/resource_bridges.c @@ -24,8 +24,6 @@ */ /*** MODULEINFO - <depend type="module">res_stasis_recording</depend> - <depend type="module">res_stasis_playback</depend> <support_level>core</support_level> ***/ diff --git a/res/ari/resource_channels.c b/res/ari/resource_channels.c index 6b4f26c25..cdf0f8820 100644 --- a/res/ari/resource_channels.c +++ b/res/ari/resource_channels.c @@ -24,10 +24,6 @@ */ /*** MODULEINFO - <depend type="module">res_stasis_answer</depend> - <depend type="module">res_stasis_playback</depend> - <depend type="module">res_stasis_recording</depend> - <depend type="module">res_stasis_snoop</depend> <support_level>core</support_level> ***/ diff --git a/res/ari/resource_device_states.c b/res/ari/resource_device_states.c index 020d9301f..c442fa51b 100644 --- a/res/ari/resource_device_states.c +++ b/res/ari/resource_device_states.c @@ -24,7 +24,6 @@ */ /*** MODULEINFO - <depend type="module">res_stasis_device_state</depend> <support_level>core</support_level> ***/ diff --git a/res/ari/resource_events.c b/res/ari/resource_events.c index d81067f15..5a8e89858 100644 --- a/res/ari/resource_events.c +++ b/res/ari/resource_events.c @@ -24,7 +24,6 @@ */ /*** MODULEINFO - <depend type="module">res_http_websocket</depend> <support_level>core</support_level> ***/ diff --git a/res/ari/resource_mailboxes.c b/res/ari/resource_mailboxes.c index 3577e6515..ae0f24978 100644 --- a/res/ari/resource_mailboxes.c +++ b/res/ari/resource_mailboxes.c @@ -24,7 +24,6 @@ */ /*** MODULEINFO - <depend type="module">res_stasis_mailbox</depend> <support_level>core</support_level> ***/ diff --git a/res/ari/resource_playbacks.c b/res/ari/resource_playbacks.c index 57f424c01..fc05a8d03 100644 --- a/res/ari/resource_playbacks.c +++ b/res/ari/resource_playbacks.c @@ -24,7 +24,6 @@ */ /*** MODULEINFO - <depend type="module">res_stasis_playback</depend> <support_level>core</support_level> ***/ diff --git a/res/ari/resource_recordings.c b/res/ari/resource_recordings.c index 39ef65ce3..090f26370 100644 --- a/res/ari/resource_recordings.c +++ b/res/ari/resource_recordings.c @@ -24,7 +24,6 @@ */ /*** MODULEINFO - <depend type="module">res_stasis_recording</depend> <support_level>core</support_level> ***/ diff --git a/res/res_ari_bridges.c b/res/res_ari_bridges.c index 34b9d898c..93cf7c49b 100644 --- a/res/res_ari_bridges.c +++ b/res/res_ari_bridges.c @@ -35,6 +35,8 @@ <depend type="module">res_ari</depend> <depend type="module">res_ari_model</depend> <depend type="module">res_stasis</depend> + <depend type="module">res_stasis_recording</depend> + <depend type="module">res_stasis_playback</depend> <support_level>core</support_level> ***/ @@ -1579,5 +1581,5 @@ AST_MODULE_INFO(ASTERISK_GPL_KEY, AST_MODFLAG_DEFAULT, "RESTful API module - Bri .support_level = AST_MODULE_SUPPORT_CORE, .load = load_module, .unload = unload_module, - .requires = "res_ari,res_ari_model,res_stasis", + .requires = "res_ari,res_ari_model,res_stasis,res_stasis_recording,res_stasis_playback", ); diff --git a/res/res_ari_channels.c b/res/res_ari_channels.c index 7862c1006..7ca7e2e4d 100644 --- a/res/res_ari_channels.c +++ b/res/res_ari_channels.c @@ -35,6 +35,10 @@ <depend type="module">res_ari</depend> <depend type="module">res_ari_model</depend> <depend type="module">res_stasis</depend> + <depend type="module">res_stasis_answer</depend> + <depend type="module">res_stasis_playback</depend> + <depend type="module">res_stasis_recording</depend> + <depend type="module">res_stasis_snoop</depend> <support_level>core</support_level> ***/ @@ -2869,5 +2873,5 @@ AST_MODULE_INFO(ASTERISK_GPL_KEY, AST_MODFLAG_DEFAULT, "RESTful API module - Cha .support_level = AST_MODULE_SUPPORT_CORE, .load = load_module, .unload = unload_module, - .requires = "res_ari,res_ari_model,res_stasis", + .requires = "res_ari,res_ari_model,res_stasis,res_stasis_answer,res_stasis_playback,res_stasis_recording,res_stasis_snoop", ); diff --git a/res/res_ari_device_states.c b/res/res_ari_device_states.c index ec8890b1e..04479e6ac 100644 --- a/res/res_ari_device_states.c +++ b/res/res_ari_device_states.c @@ -35,6 +35,7 @@ <depend type="module">res_ari</depend> <depend type="module">res_ari_model</depend> <depend type="module">res_stasis</depend> + <depend type="module">res_stasis_device_state</depend> <support_level>core</support_level> ***/ @@ -349,5 +350,5 @@ AST_MODULE_INFO(ASTERISK_GPL_KEY, AST_MODFLAG_DEFAULT, "RESTful API module - Dev .support_level = AST_MODULE_SUPPORT_CORE, .load = load_module, .unload = unload_module, - .requires = "res_ari,res_ari_model,res_stasis", + .requires = "res_ari,res_ari_model,res_stasis,res_stasis_device_state", ); diff --git a/res/res_ari_events.c b/res/res_ari_events.c index f750a5454..06eb77a38 100644 --- a/res/res_ari_events.c +++ b/res/res_ari_events.c @@ -35,6 +35,7 @@ <depend type="module">res_ari</depend> <depend type="module">res_ari_model</depend> <depend type="module">res_stasis</depend> + <depend type="module">res_http_websocket</depend> <support_level>core</support_level> ***/ @@ -473,5 +474,5 @@ AST_MODULE_INFO(ASTERISK_GPL_KEY, AST_MODFLAG_DEFAULT, "RESTful API module - Web .support_level = AST_MODULE_SUPPORT_CORE, .load = load_module, .unload = unload_module, - .requires = "res_ari,res_ari_model,res_stasis", + .requires = "res_ari,res_ari_model,res_stasis,res_http_websocket", ); diff --git a/res/res_ari_mailboxes.c b/res/res_ari_mailboxes.c index 6469f936e..123664e46 100644 --- a/res/res_ari_mailboxes.c +++ b/res/res_ari_mailboxes.c @@ -35,6 +35,7 @@ <depend type="module">res_ari</depend> <depend type="module">res_ari_model</depend> <depend type="module">res_stasis</depend> + <depend type="module">res_stasis_mailbox</depend> <support_level>core</support_level> ***/ @@ -355,5 +356,5 @@ AST_MODULE_INFO(ASTERISK_GPL_KEY, AST_MODFLAG_DEFAULT, "RESTful API module - Mai .support_level = AST_MODULE_SUPPORT_CORE, .load = load_module, .unload = unload_module, - .requires = "res_ari,res_ari_model,res_stasis", + .requires = "res_ari,res_ari_model,res_stasis,res_stasis_mailbox", ); diff --git a/res/res_ari_playbacks.c b/res/res_ari_playbacks.c index 0148a7400..547b39e39 100644 --- a/res/res_ari_playbacks.c +++ b/res/res_ari_playbacks.c @@ -35,6 +35,7 @@ <depend type="module">res_ari</depend> <depend type="module">res_ari_model</depend> <depend type="module">res_stasis</depend> + <depend type="module">res_stasis_playback</depend> <support_level>core</support_level> ***/ @@ -307,5 +308,5 @@ AST_MODULE_INFO(ASTERISK_GPL_KEY, AST_MODFLAG_DEFAULT, "RESTful API module - Pla .support_level = AST_MODULE_SUPPORT_CORE, .load = load_module, .unload = unload_module, - .requires = "res_ari,res_ari_model,res_stasis", + .requires = "res_ari,res_ari_model,res_stasis,res_stasis_playback", ); diff --git a/res/res_ari_recordings.c b/res/res_ari_recordings.c index 531ff6574..45c789c2d 100644 --- a/res/res_ari_recordings.c +++ b/res/res_ari_recordings.c @@ -35,6 +35,7 @@ <depend type="module">res_ari</depend> <depend type="module">res_ari_model</depend> <depend type="module">res_stasis</depend> + <depend type="module">res_stasis_recording</depend> <support_level>core</support_level> ***/ @@ -891,5 +892,5 @@ AST_MODULE_INFO(ASTERISK_GPL_KEY, AST_MODFLAG_DEFAULT, "RESTful API module - Rec .support_level = AST_MODULE_SUPPORT_CORE, .load = load_module, .unload = unload_module, - .requires = "res_ari,res_ari_model,res_stasis", + .requires = "res_ari,res_ari_model,res_stasis,res_stasis_recording", ); diff --git a/res/res_monitor.c b/res/res_monitor.c index 9dcbdbe62..e2b5fb252 100644 --- a/res/res_monitor.c +++ b/res/res_monitor.c @@ -1013,4 +1013,5 @@ AST_MODULE_INFO(ASTERISK_GPL_KEY, AST_MODFLAG_GLOBAL_SYMBOLS | AST_MODFLAG_LOAD_ .load = load_module, .unload = unload_module, .load_pri = AST_MODPRI_CHANNEL_DEPEND, + .optional_modules = "func_periodic_hook", ); diff --git a/res/res_pjsip.c b/res/res_pjsip.c index 05f92f268..0c7fd5189 100644 --- a/res/res_pjsip.c +++ b/res/res_pjsip.c @@ -5238,4 +5238,5 @@ AST_MODULE_INFO(ASTERISK_GPL_KEY, AST_MODFLAG_GLOBAL_SYMBOLS | AST_MODFLAG_LOAD_ .reload = reload_module, .load_pri = AST_MODPRI_CHANNEL_DEPEND - 5, .requires = "res_pjproject", + .optional_modules = "res_statsd", ); diff --git a/res/res_pjsip/config_transport.c b/res/res_pjsip/config_transport.c index 63bf11812..713ab024b 100644 --- a/res/res_pjsip/config_transport.c +++ b/res/res_pjsip/config_transport.c @@ -30,6 +30,7 @@ #include "asterisk/acl.h" #include "asterisk/utils.h" #include "include/res_pjsip_private.h" +/* We're only using a #define from http_websocket.h, no OPTIONAL_API symbols are used. */ #include "asterisk/http_websocket.h" #define MAX_POINTER_STRING 33 diff --git a/res/res_pjsip_outbound_registration.c b/res/res_pjsip_outbound_registration.c index 79bea482b..80b885fb4 100644 --- a/res/res_pjsip_outbound_registration.c +++ b/res/res_pjsip_outbound_registration.c @@ -2290,4 +2290,5 @@ AST_MODULE_INFO(ASTERISK_GPL_KEY, AST_MODFLAG_LOAD_ORDER, "PJSIP Outbound Regist .unload = unload_module, .load_pri = AST_MODPRI_APP_DEPEND, .requires = "res_pjsip", + .optional_modules = "res_statsd", ); diff --git a/res/res_statsd.c b/res/res_statsd.c index 221b359c9..8565e41cb 100644 --- a/res/res_statsd.c +++ b/res/res_statsd.c @@ -381,13 +381,13 @@ static int reload_module(void) } } -/* The priority of this module is set to be as low as possible, since it could - * be used by any other sort of module. +/* The priority of this module is set just after realtime, since it loads + * configuration and could be used by any other sort of module. */ AST_MODULE_INFO(ASTERISK_GPL_KEY, AST_MODFLAG_GLOBAL_SYMBOLS | AST_MODFLAG_LOAD_ORDER, "Statsd client support", .support_level = AST_MODULE_SUPPORT_EXTENDED, .load = load_module, .unload = unload_module, .reload = reload_module, - .load_pri = 0, + .load_pri = AST_MODPRI_REALTIME_DRIVER + 5, ); diff --git a/rest-api-templates/res_ari_resource.c.mustache b/rest-api-templates/res_ari_resource.c.mustache index c1f680f21..c69734539 100644 --- a/rest-api-templates/res_ari_resource.c.mustache +++ b/rest-api-templates/res_ari_resource.c.mustache @@ -40,6 +40,9 @@ <depend type="module">res_ari</depend> <depend type="module">res_ari_model</depend> <depend type="module">res_stasis</depend> +{{#requires_modules}} + <depend type="module">{{.}}</depend> +{{/requires_modules}} <support_level>core</support_level> ***/ @@ -324,6 +327,6 @@ AST_MODULE_INFO(ASTERISK_GPL_KEY, AST_MODFLAG_DEFAULT, "RESTful API module - {{{ .support_level = AST_MODULE_SUPPORT_CORE, .load = load_module, .unload = unload_module, - .requires = "res_ari,res_ari_model,res_stasis", + .requires = "res_ari,res_ari_model,res_stasis{{#requires_modules}},{{.}}{{/requires_modules}}", ); {{/api_declaration}} diff --git a/rest-api-templates/swagger_model.py b/rest-api-templates/swagger_model.py index 08852415c..3f729d8b5 100644 --- a/rest-api-templates/swagger_model.py +++ b/rest-api-templates/swagger_model.py @@ -647,6 +647,7 @@ class ApiDeclaration(Stringify): self.api_version = api_decl_json.get('apiVersion') self.base_path = api_decl_json.get('basePath') self.resource_path = api_decl_json.get('resourcePath') + self.requires_modules = api_decl_json.get('requiresModules') or [] api_json = api_decl_json.get('apis') or [] self.apis = [ Api().load(j, processor, context) for j in api_json] diff --git a/rest-api/api-docs/bridges.json b/rest-api/api-docs/bridges.json index cf8ee3156..04416c123 100644 --- a/rest-api/api-docs/bridges.json +++ b/rest-api/api-docs/bridges.json @@ -6,6 +6,10 @@ "swaggerVersion": "1.1", "basePath": "http://localhost:8088/ari", "resourcePath": "/api-docs/bridges.{format}", + "requiresModules": [ + "res_stasis_recording", + "res_stasis_playback" + ], "apis": [ { "path": "/bridges", diff --git a/rest-api/api-docs/channels.json b/rest-api/api-docs/channels.json index 602606cfe..ec8bae235 100644 --- a/rest-api/api-docs/channels.json +++ b/rest-api/api-docs/channels.json @@ -6,6 +6,12 @@ "swaggerVersion": "1.1", "basePath": "http://localhost:8088/ari", "resourcePath": "/api-docs/channels.{format}", + "requiresModules": [ + "res_stasis_answer", + "res_stasis_playback", + "res_stasis_recording", + "res_stasis_snoop" + ], "apis": [ { "path": "/channels", diff --git a/rest-api/api-docs/deviceStates.json b/rest-api/api-docs/deviceStates.json index 94d5b93ce..bd389355c 100644 --- a/rest-api/api-docs/deviceStates.json +++ b/rest-api/api-docs/deviceStates.json @@ -6,6 +6,9 @@ "swaggerVersion": "1.1", "basePath": "http://localhost:8088/ari", "resourcePath": "/api-docs/deviceStates.{format}", + "requiresModules": [ + "res_stasis_device_state" + ], "apis": [ { "path": "/deviceStates", diff --git a/rest-api/api-docs/events.json b/rest-api/api-docs/events.json index e1b31bb97..d85d8d9fe 100644 --- a/rest-api/api-docs/events.json +++ b/rest-api/api-docs/events.json @@ -6,6 +6,9 @@ "swaggerVersion": "1.2", "basePath": "http://localhost:8088/ari", "resourcePath": "/api-docs/events.{format}", + "requiresModules": [ + "res_http_websocket" + ], "apis": [ { "path": "/events", diff --git a/rest-api/api-docs/mailboxes.json b/rest-api/api-docs/mailboxes.json index ed50019f5..8f5941b81 100644 --- a/rest-api/api-docs/mailboxes.json +++ b/rest-api/api-docs/mailboxes.json @@ -6,6 +6,9 @@ "swaggerVersion": "1.1", "basePath": "http://localhost:8088/ari", "resourcePath": "/api-docs/mailboxes.{format}", + "requiresModules": [ + "res_stasis_mailbox" + ], "apis": [ { "path": "/mailboxes", diff --git a/rest-api/api-docs/playbacks.json b/rest-api/api-docs/playbacks.json index 3dc4e1da5..a78dcae71 100644 --- a/rest-api/api-docs/playbacks.json +++ b/rest-api/api-docs/playbacks.json @@ -6,6 +6,9 @@ "swaggerVersion": "1.1", "basePath": "http://localhost:8088/ari", "resourcePath": "/api-docs/playbacks.{format}", + "requiresModules": [ + "res_stasis_playback" + ], "apis": [ { "path": "/playbacks/{playbackId}", diff --git a/rest-api/api-docs/recordings.json b/rest-api/api-docs/recordings.json index d0b9630da..6ffd6d8e9 100644 --- a/rest-api/api-docs/recordings.json +++ b/rest-api/api-docs/recordings.json @@ -6,6 +6,9 @@ "swaggerVersion": "1.1", "basePath": "http://localhost:8088/ari", "resourcePath": "/api-docs/recordings.{format}", + "requiresModules": [ + "res_stasis_recording" + ], "apis": [ { "path": "/recordings/stored", diff --git a/tests/test_utils.c b/tests/test_utils.c index 68a285727..bcb2df375 100644 --- a/tests/test_utils.c +++ b/tests/test_utils.c @@ -646,4 +646,9 @@ static int load_module(void) return AST_MODULE_LOAD_SUCCESS; } -AST_MODULE_INFO_STANDARD(ASTERISK_GPL_KEY, "Utils test module"); +AST_MODULE_INFO(ASTERISK_GPL_KEY, AST_MODFLAG_DEFAULT, "Utils test module", + .support_level = AST_MODULE_SUPPORT_CORE, + .load = load_module, + .unload = unload_module, + .requires = "res_agi,res_crypto", +); diff --git a/tests/test_websocket_client.c b/tests/test_websocket_client.c index 6743b5369..e33f3f930 100644 --- a/tests/test_websocket_client.c +++ b/tests/test_websocket_client.c @@ -156,4 +156,9 @@ static int unload_module(void) return 0; } -AST_MODULE_INFO_STANDARD(ASTERISK_GPL_KEY, "Websocket client test module"); +AST_MODULE_INFO(ASTERISK_GPL_KEY, AST_MODFLAG_DEFAULT, "Websocket client test module", + .support_level = AST_MODULE_SUPPORT_CORE, + .load = load_module, + .unload = unload_module, + .requires = "res_http_websocket", +); |