[Asterisk-code-review] xmldoc: Allow XML docs to be reloaded. (asterisk[master])

N A asteriskteam at digium.com
Mon Nov 7 10:23:20 CST 2022


Attention is currently required from: Joshua Colp, George Joseph.

N A has posted comments on this change. ( https://gerrit.asterisk.org/c/asterisk/+/19474 )

Change subject: xmldoc: Allow XML docs to be reloaded.
......................................................................


Patch Set 3:

(1 comment)

Patchset:

PS3: 
> Can you give me an example of how to reproduce this? […]
For modules in general, they will load but you will get a warning that XML documentation could not be found. This is if the XML documentation for the module wasn't there when the docs get built.

It may/probably requires developer mode.

Running "xmldoc reload" and then "module refresh <module>" will load the module and there will be no warnings.

Modules that use ACO/sorcery for config will fail to load entirely. The example that was pointed out was codec_opus, which will not load while Asterisk is running if the XML docs aren't there (so xmldoc reload would allow it to load).



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

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Change-Id: I4f265b0e5517e757c5453a0f241201a5788d3a07
Gerrit-Change-Number: 19474
Gerrit-PatchSet: 3
Gerrit-Owner: N A <asterisk at phreaknet.org>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Joshua Colp <jcolp at sangoma.com>
Gerrit-Attention: Joshua Colp <jcolp at sangoma.com>
Gerrit-Attention: George Joseph <gjoseph at digium.com>
Gerrit-Comment-Date: Mon, 07 Nov 2022 16:23:20 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: George Joseph <gjoseph at digium.com>
Gerrit-MessageType: comment
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20221107/340615ab/attachment.html>


More information about the asterisk-code-review mailing list