Re: [LTP] [PATCH v4 2/5] doc: Add sphinx to requirements.txt

Andrea Cervesato via ltp
2025-03-31T17:05:17Z

On 3/31/25 18:37, Petr Vorel wrote:
> Wait, one thing: we use doc/requirements.txt in .readthedocs.yml. Is it really a
> good idea to add there sphinx, when it uses containers? I can test it in a
> separate branch, but we discussed some time ago that we should avoid adding
> sphinx for readthedocs.
>
> Wouldn't be in the end better to keep 'pip install sphinx' in Makefile instead
> (the previous proposal).
>
> Kind regards,
> Petr

I forgot about it. Thanks for remembering. At the moment readthedocs
installs sphinx via pip, so if we add just sphinx ro requirements.txt
without version, pip should reinstall the same version which is already
available before building the website.

We should give a try and eventually get back to the old way.

Andrea