<div dir="ltr">Thanks Luca, Thomas,<div><br></div><div>I see Thomas just applied it to main, but I'll mention I tested with meson werror and I think this fix is appropriate.</div><div><br></div><div>This is a good reminder for us to always build with --werror </div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Nov 20, 2024 at 10:54 AM Thomas Monjalon <<a href="mailto:thomas@monjalon.net">thomas@monjalon.net</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">20/11/2024 16:05, Thomas Monjalon:<br>
> 20/11/2024 15:41, Luca Vizzarro:<br>
> > Sphinx-autodoc issues warnings about some symbols due to bad autodoc<br>
> > "automock"ing when optional dependencies are missing.<br>
> > <br>
> > The current meson setup allows to treat warnings as errors, halting the<br>
> > process and exiting early. Given these warnings should be ignored, this<br>
> > patch removes this behavior.<br>
> <br>
> So the patch is not fixing warnings, it allows them.<br>
> I will change the title.<br>
> <br>
> > Until optional dependencies are enforced,<br>
> > warnings for DTS API docs will be checked manually as part of the DTS<br>
> > approval process.<br>
> <br>
> No, optional dependencies should never be enforced.<br>
> <br>
> [...]<br>
> > -if get_option('werror')<br>
> > -    extra_sphinx_args += '-W'<br>
> > -endif<br>
<br>
It makes the CI issue disappear.<br>
<br>
Applied with this modified message, thanks for fixing quickly:<br>
<br>
    doc: allow warnings in Sphinx for DTS<br>
<br>
    Sphinx-autodoc issues warnings about some symbols due to bad autodoc<br>
    "automock"ing when optional dependencies are missing:<br>
<br>
    Warning, treated as error:<br>
            Failed to get a method signature for<br>
            framework.config.TestSuiteConfig.convert_from_string:<br>
            <classmethod(<function TestSuiteConfig.convert_from_string<br>
            at 0x7f1a4ec10790>)> is not a callable object<br>
<br>
    The current meson setup allows to treat warnings as errors, halting the<br>
    process and exiting early. Given these warnings should be ignored, this<br>
    patch removes this behavior. Until optional dependencies are detected,<br>
    warnings for DTS API docs will be checked manually as part of the DTS<br>
    approval process.<br>
<br>
    Fixes: 6597fa4a30ad ("dts: add autodoc Pydantic")<br>
<br>
    Signed-off-by: Luca Vizzarro <<a href="mailto:luca.vizzarro@arm.com" target="_blank">luca.vizzarro@arm.com</a>><br>
<br>
<br>
<br>
<br>
</blockquote></div>