Re: [PATCH] changes.rst: explain the usage of virtual environment
From: Mauro Carvalho Chehab
Date: Mon Jun 19 2017 - 10:38:20 EST
Em Mon, 19 Jun 2017 16:11:56 +0200
Markus Heiser <markus.heiser@xxxxxxxxxxx> escreveu:
> > Am 19.06.2017 um 15:46 schrieb Jonathan Corbet <corbet@xxxxxxx>:
> > On Mon, 19 Jun 2017 06:08:37 -0700
> > Christoph Hellwig <hch@xxxxxxxxxxxxx> wrote:
> >> On Mon, Jun 19, 2017 at 08:24:10AM -0300, Mauro Carvalho Chehab wrote:
> >>> As the Sphinx build seems very fragile, specially for
> >>> PDF output, add a notice about how to use it on a virtual
> >>> environment.
> >> Please don't. python venv are good at one thing only, and that is
> >> making a mess of your python module path. Don't ever use them on a
> >> system that has proper package management.
> > Well, they are also good for running specific versions of a given package
> > without disrupting the setup of your system as a whole, and when even a
> > system with proper package management may not offer the version you need.
> > I guess my question with the patch is whether this text really belongs in
> > changes.rst, or whether it should be part of the doc-guide.
> Hi, here are my 5cents:
> Typically I have a PY_ENV target in my projects, building a virtualenv
> in a folder named ./local. E.g. in LinuxDoc  I use something like this:
> PY ?=3
> PYTHON ?= python$(PY)
> VIRTUALENV = virtualenv --python=$(PYTHON)
> VTENV_OPTS = "--no-site-packages"
> PY_ENV = ./local/py$(PY)
I would split the PATH name on a separate var. This way, if one would
like to have multiple Sphinx versions, all it would need would be to
change the directory.
I would prefer to call it as "./sphinx" (or something similar).
So, I would do:
PY_DIR = ./sphinx
PY_ENV = $(PY_DIR)/py$(PY)
Don't forget to add $PY_DIR directory to .gitignore on your patch.
> quiet_cmd_virtualenv = PYENV $@
> cmd_virtualenv = \
> if [ ! -d "./$(PY_ENV)" ];then \
> $(VIRTUALENV) $(VIRTUALENV_VERBOSE) $(VTENV_OPTS) $2; \
> else \
> echo "using virtualenv from $2"; \
> # to build *local* environment, python and virtualenv from the OS is needed!
> $(PY_ENV): virtualenv-exe python-exe
> $(call cmd,virtualenv,$(PY_ENV))
> @$(PY_ENV_BIN)/pip install $(PIP_VERBOSE) -r requirements.txt
Shouldn't it be using "pip$(PY)" instead?
Also, better to seek for requirements on a file under Documentation/sphinx/
> And the sphinxbuild coammand is used from there::
> SPHINXBUILD ?= $(PY_ENV_BIN)/sphinx-build
> By this I can stick versions packages. E.g. to select last version of
> RTD theme and Sphinx version 1.5 or upper, I add the following lines to
> my reqierements.txt::
> If you are interested, I can prepare a patch, to add such functionality
> (as option) to Documents/Makefile (which will be documented in the doc-guide).
Yeah, IMHO, it makes sense to have something like that at the main build,
as an optional feature, e. g. perhaps adding a new make target, like:
$ make sphinx_virtenv
That would create and populate PY_DIR. If $PY_DIR/bin/sphinx-build exists
and it is executable file, run it. Otherwise, use the system's sphinx,
>  https://github.com/return42/linuxdoc/blob/master/utils/makefile.python
> -- Markus --
> > jon
> > --
> > To unsubscribe from this list: send the line "unsubscribe linux-doc" in
> > the body of a message to majordomo@xxxxxxxxxxxxxxx
> > More majordomo info at http://vger.kernel.org/majordomo-info.html