[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 03/10] meson: prefer 'sphinx-build' to 'sphinx-build-3'
From: |
Markus Armbruster |
Subject: |
Re: [PATCH 03/10] meson: prefer 'sphinx-build' to 'sphinx-build-3' |
Date: |
Wed, 22 Feb 2023 17:26:00 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/28.2 (gnu/linux) |
Paolo Bonzini <pbonzini@redhat.com> writes:
> From: John Snow <jsnow@redhat.com>
>
> Once upon a time, "sphinx-build" on certain RPM platforms invoked
> specifically a Python 2.x version, while "sphinx-build-3" was a distro
> shim for the Python 3.x version.
>
> These days, none of our supported platforms utilize a 2.x version, so it
> should be safe to search for 'sphinx-build' prior to 'sphinx-build-3',
> which will prefer pip/venv installed versions of sphinx if they're
> available.
>
> This adds an extremely convenient ability to test document building
> ability in QEMU across multiple versions of Sphinx for the purposes of
> compatibility testing.
>
> Signed-off-by: John Snow <jsnow@redhat.com>
> Message-Id: <20230221012456.2607692-6-jsnow@redhat.com>
Accident?
> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
> ---
> docs/meson.build | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/docs/meson.build b/docs/meson.build
> index 9136fed3b730..906034f9a87f 100644
> --- a/docs/meson.build
> +++ b/docs/meson.build
> @@ -1,5 +1,5 @@
> if get_option('sphinx_build') == ''
> - sphinx_build = find_program(['sphinx-build-3', 'sphinx-build'],
> + sphinx_build = find_program(['sphinx-build', 'sphinx-build-3'],
> required: get_option('docs'))
> else
> sphinx_build = find_program(get_option('sphinx_build'),
In review of the original "[PATCH v3 5/6] meson: prefer 'sphinx-build'
to 'sphinx-build-3'", I challenged the check for sphinx-build-3.
- [PATCH v4 0/9] improvement to Python detection, preparation for dropping 3.6, Paolo Bonzini, 2023/02/22
- [PATCH 01/10] python: support pylint 2.16, Paolo Bonzini, 2023/02/22
- [PATCH 05/10] configure: Look for auxiliary Python installations, Paolo Bonzini, 2023/02/22
- [PATCH 02/10] python: drop pipenv, Paolo Bonzini, 2023/02/22
- [PATCH 03/10] meson: prefer 'sphinx-build' to 'sphinx-build-3', Paolo Bonzini, 2023/02/22
- Re: [PATCH 03/10] meson: prefer 'sphinx-build' to 'sphinx-build-3',
Markus Armbruster <=
- [PATCH 04/10] configure: protect against escaping venv when running Meson, Paolo Bonzini, 2023/02/22
- [PATCH 08/10] ci, docker: update CentOS and OpenSUSE Python to non-EOL versions, Paolo Bonzini, 2023/02/22
- [PATCH 07/10] docs/devel: update and clarify lcitool instructions, Paolo Bonzini, 2023/02/22
- [PATCH 06/10] lcitool: update submodule, Paolo Bonzini, 2023/02/22