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

Andrea Cervesato andrea.cervesato@suse.com
Fri Mar 28 14:02:17 CET 2025


Hi Petr,

On 3/28/25 10:57, Petr Vorel wrote:
> Although that slightly prolongs creating virtualenv, it's probably
> better to handle it as the other requirements.
>
> Suggested-by: Andrea Cervesato <andrea.cervesato@suse.com>
> Signed-off-by: Petr Vorel <pvorel@suse.cz>
> ---
> New in v4.
>
>   doc/developers/documentation.rst | 2 --
>   doc/requirements.txt             | 1 +
>   2 files changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/doc/developers/documentation.rst b/doc/developers/documentation.rst
> index 27c847e125..a303253693 100644
> --- a/doc/developers/documentation.rst
> +++ b/doc/developers/documentation.rst
> @@ -23,8 +23,6 @@ Before building, make sure you have python3 ``virtualenv`` module installed.
>       # prepare virtual environment
>       python3 -m virtualenv .venv
>       . .venv/bin/activate
> -
> -    pip install sphinx # usually packaged in distros
>       pip install -r requirements.txt
>   
>       # build documentation
> diff --git a/doc/requirements.txt b/doc/requirements.txt
> index 742fb8b4bb..1f4cefb2c6 100644
> --- a/doc/requirements.txt
> +++ b/doc/requirements.txt
> @@ -1,3 +1,4 @@
> +sphinx
In requirements.txt we usually put the right version of the package that 
we want to use. We can use the latest, what do you think?
>   sphinx-rtd-theme==2.0.0
>   linuxdoc==20231020
>   sphinxcontrib-spelling==7.7.0
Andrea


More information about the ltp mailing list