From 17e7bf9453ae9a9ea5151ae5c76f1bf468bd00b6 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Filipe=20La=C3=ADns?= Date: Mon, 20 Jan 2025 18:33:11 +0000 Subject: [PATCH] doc: fix venv creating for the local Python using uv MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit GH-120711 introduced uv as an alternative venv provisioning and method, but failed to honor the `PYTHON` variable, leading uv to the system's default Python installation. Honoring `PYTHON` is important for use cases such as `doctest`, which should run with the locally built Python. Signed-off-by: Filipe LaĆ­ns --- Doc/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Doc/Makefile b/Doc/Makefile index 1a66642a4a03ed..b8896da4a91869 100644 --- a/Doc/Makefile +++ b/Doc/Makefile @@ -172,7 +172,7 @@ venv: else \ echo "Creating venv in $(VENVDIR)"; \ if $(UV) --version >/dev/null 2>&1; then \ - $(UV) venv $(VENVDIR); \ + $(UV) venv --python=$(PYTHON) $(VENVDIR); \ VIRTUAL_ENV=$(VENVDIR) $(UV) pip install -r $(REQUIREMENTS); \ else \ $(PYTHON) -m venv $(VENVDIR); \