laforge has submitted this change. ( https://gerrit.osmocom.org/c/pysim/+/38386?usp=email )
Change subject: docs/Makefile: make SPHINXBUILD work in venv ...................................................................... docs/Makefile: make SPHINXBUILD work in venv sphinx-build doesn't use the PYTHONPATH from the venv, unless it runs as python3 -m sphinx.cmd.build. We need it to use the imports from PYTHONPATH, so we can update the pyosmocom version in requirements.txt in a patch, and this new version will be used in the jenkins job that runs during gerrit review. Otherwise the previously installed version (from the docker image) will be used. Related: https://github.com/sphinx-doc/sphinx/issues/8910 Change-Id: I487e1af6a3493df5b806cc2d3d2b70bc5233b89f --- M docs/Makefile 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Jenkins Builder: Verified laforge: Looks good to me, approved diff --git a/docs/Makefile b/docs/Makefile index 3b99b25..4523bf6 100644 --- a/docs/Makefile +++ b/docs/Makefile @@ -4,7 +4,7 @@ # You can set these variables from the command line, and also # from the environment for the first two. SPHINXOPTS ?= -SPHINXBUILD ?= sphinx-build +SPHINXBUILD ?= python3 -m sphinx.cmd.build SOURCEDIR = . BUILDDIR = _build -- To view, visit https://gerrit.osmocom.org/c/pysim/+/38386?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: merged Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: I487e1af6a3493df5b806cc2d3d2b70bc5233b89f Gerrit-Change-Number: 38386 Gerrit-PatchSet: 1 Gerrit-Owner: osmith <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: dexter <[email protected]> Gerrit-Reviewer: laforge <[email protected]>
