[Asterisk-code-review] loader: Sync load- and build-time deps. (asterisk[18])

[Corey Farrell]

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

Change subject: loader: Sync load- and build-time deps.
......................................................................


Patch Set 2:

> Patch Set 2:
> 
> > no idea why the build fails
> 
> Yepp, that is because the system on Gerrit was build with ./configure --enable-dev-mode. Now, I am able to reproduce the FRACK. I removed that .requires and therefore res_pjsip_config_wizard for now.
> 
> Still, I am curious about that sorcery stuff; astdb and config. Found no way to avoid that, yet. That would explain why they are considered dependencies at build-time. However, I am curious about sorcery_memory. Seems not to be required but why a must have dependency?!

If you don't set the sorcery provider in sorcery.conf then sorcery will use the default provider, I believe this is set by calls to ast_sorcery_apply_default.

The res_sorcery_* modules have load priority AST_MODPRI_REALTIME_DRIVER, this is what controls load order.  Unless you do `autoload=no` in your modules.conf Asterisk already does the right thing, if you disable autoload then you are responsible for manually loading the required res_sorcery_* modules.


-- 
To view, visit https://gerrit.asterisk.org/c/asterisk/+/15119
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: 18
Gerrit-Change-Id: I254dd33194ae38d2877b8021c57c2a5deb6bbcd2
Gerrit-Change-Number: 15119
Gerrit-PatchSet: 2
Gerrit-Owner: Alexander Traud <pabstraud at compuserve.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-CC: Corey Farrell <git at cfware.com>
Gerrit-Comment-Date: Mon, 02 Nov 2020 21:22:25 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20201102/3f0ffcb3/attachment.html>