[Asterisk-code-review] loader: Add support for built-in modules. (asterisk[master])

[Corey Farrell]

Corey Farrell has posted comments on this change. ( https://gerrit.asterisk.org/7988 )

Change subject: loader: Add support for built-in modules.
......................................................................


Patch Set 2:

> (4 comments)
 > 
 > Why is this built-in module stuff even needed?  Core built-in
 > modules are always available.

The intent is to render 'preload=res_config_whatever' unneeded (still supported, but redundant 99.9% of the time).  Take AMI as an example, we would make it a built-in module with load_pri to start just after realtime drivers.  We would rewire the init/reload/shutdown functions in main/manager.c to the standard AST_MODULE_INFO macro, remove the call to initialize it from main/asterisk.c.  Then AMI would be able to use realtime without having to preload anything.

Obviously this commit doesn't accomplish anything for end users - I'm planning on those changes being done separately after the current series is merged.


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

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I920852bc17513a9c2627061a4ad40511e3a20499
Gerrit-Change-Number: 7988
Gerrit-PatchSet: 2
Gerrit-Owner: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: Jenkins2
Gerrit-Reviewer: Richard Mudgett <rmudgett at digium.com>
Gerrit-Comment-Date: Thu, 18 Jan 2018 00:16:45 +0000
Gerrit-HasComments: No
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20180117/6c983207/attachment.html>