Messages in this thread | | | From | Jonathan Corbet <> | Subject | Re: [PATCH v3] docs: scripts: sphinx-pre-install: Fix building docs with pyyaml package | Date | Wed, 14 Feb 2024 15:32:48 -0700 |
| |
Thorsten Blum <thorsten.blum@toblux.com> writes:
> The Python module pyyaml is required to build the docs, but it is only > listed in Documentation/sphinx/requirements.txt and is therefore missing > when Sphinx is installed as a package and not via pip/pypi. > > Add pyyaml as an optional package for multiple distros to fix building the > docs if you prefer to install Sphinx as a package. > > Signed-off-by: Thorsten Blum <thorsten.blum@toblux.com> > Reviewed-by: Vegard Nossum <vegard.nossum@oracle.com> > Tested-by: Vegard Nossum <vegard.nossum@oracle.com> > --- > Changes in v2: > - s/pyyaml/yaml/ as suggested by Vegard Nossum > - Make the check require the Python module; was optional > > Changes in v3: > - Preserve Reviewed-by: and Tested-by: tags > - Add pyyaml to openSUSE as suggested by Vegard Nossum > --- > scripts/sphinx-pre-install | 4 ++++ > 1 file changed, 4 insertions(+)
Applied, thanks.
jon
| |