20/11/2024 16:05, Thomas Monjalon:
> 20/11/2024 15:41, Luca Vizzarro:
> > Sphinx-autodoc issues warnings about some symbols due to bad autodoc
> > "automock"ing when optional dependencies are missing.
> >
> > The current meson setup allows to treat warnings as errors, halting the
> > process and exiting early. Given these warnings should be ignored, this
> > patch removes this behavior.
>
> So the patch is not fixing warnings, it allows them.
> I will change the title.
>
> > Until optional dependencies are enforced,
> > warnings for DTS API docs will be checked manually as part of the DTS
> > approval process.
>
> No, optional dependencies should never be enforced.
>
> [...]
> > -if get_option('werror')
> > - extra_sphinx_args += '-W'
> > -endif
It makes the CI issue disappear.
Applied with this modified message, thanks for fixing quickly:
doc: allow warnings in Sphinx for DTS
Sphinx-autodoc issues warnings about some symbols due to bad autodoc
"automock"ing when optional dependencies are missing:
Warning, treated as error:
Failed to get a method signature for
framework.config.TestSuiteConfig.convert_from_string:
<classmethod(<function TestSuiteConfig.convert_from_string
at 0x7f1a4ec10790>)> is not a callable object
The current meson setup allows to treat warnings as errors, halting the
process and exiting early. Given these warnings should be ignored, this
patch removes this behavior. Until optional dependencies are detected,
warnings for DTS API docs will be checked manually as part of the DTS
approval process.
Fixes: 6597fa4a30ad ("dts: add autodoc Pydantic")
Signed-off-by: Luca Vizzarro <luca.vizza...@arm.com>
