[PATCH 1/5] scripts/sphinx-pre-install: add a script to check Sphinx install

2017-07-16 Thread Mauro Carvalho Chehab
Solving Sphinx dependencies can be painful. Add a script to check if everything is ok. Tested on: - Fedora 25 and 26; - Ubuntu 17.04; - OpenSuse Tumbleweed; - Arch Linux; - Gentoo. Signed-off-by: Mauro Carvalho Chehab ---

[PATCH 5/5] doc: Makefile: if sphinx is not found, run a check script

2017-07-16 Thread Mauro Carvalho Chehab
Right now, if the building system doesn't find Sphinx, it bails out, without providing any instructions about what should be done. Instead, run a script, providing some guidance about the steps needed for Sphinx build to work. Signed-off-by: Mauro Carvalho Chehab ---

[PATCH 4/5] sphinx.rst: document scripts/sphinx-pre-install script

2017-07-16 Thread Mauro Carvalho Chehab
Now that we have a script to check for Sphinx dependencies, document it. Signed-off-by: Mauro Carvalho Chehab --- Documentation/doc-guide/sphinx.rst | 48 ++ 1 file changed, 44 insertions(+), 4 deletions(-) diff --git

[PATCH 2/5] sphinx-pre-install: detect an existing virtualenv

2017-07-16 Thread Mauro Carvalho Chehab
Detect if the script runs after creating the virtualenv, printing the command line commands to enable the virtualenv. Signed-off-by: Mauro Carvalho Chehab --- scripts/sphinx-pre-install | 26 +- 1 file changed, 17 insertions(+), 9 deletions(-)

[PATCH 0/5] Add a script to check for Sphinx install requirements

2017-07-16 Thread Mauro Carvalho Chehab
Sphinx installation is not trivial, as not all versions are supported, and it requires a lot of stuff for math, images and PDF/LaTeX output to work. So, add a script that checks if everything is fine, providing distro-specific hints about what's needed for it to work. I posted already RFC

[PATCH 3/5] sphinx-pre-install: use a requirements file

2017-07-16 Thread Mauro Carvalho Chehab
Instead of using 3 commands to install a virtualenv, use a single one, reading the requirements from this file: Documentation/sphinx/requirements.txt Signed-off-by: Mauro Carvalho Chehab --- scripts/sphinx-pre-install | 5 ++--- 1 file changed, 2

[PATCH RFC v8] scripts/sphinx-pre-install: add a script to check Sphinx install

2017-07-16 Thread Mauro Carvalho Chehab
Solving Sphinx dependencies can be painful. Add a script to check if everything is ok. Tested on: - Fedora 25 and 26; - Ubuntu 17.04; - OpenSuse Tumbleweed; - Arch Linux; - Gentoo. Signed-off-by: Mauro Carvalho Chehab --- v8:

Re: [PATCH RFC] scripts/sphinx-pre-install: add a script to check Sphinx install

2017-07-16 Thread Mauro Carvalho Chehab
Em Sun, 16 Jul 2017 09:29:35 +0200 Markus Heiser escreveu: > Hy Mauro, > > thanks a lot for your RFC, your patch consolidate a lot of > knowledge around Sphinx build requirements and brings a huge > value I will no longer miss. > > I tested v6 of this patch on ubuntu

Re: [PATCH RFC] scripts/sphinx-pre-install: add a script to check Sphinx install

2017-07-16 Thread Markus Heiser
Hy Mauro, thanks a lot for your RFC, your patch consolidate a lot of knowledge around Sphinx build requirements and brings a huge value I will no longer miss. I tested v6 of this patch on ubuntu and there is only some conceptual bikeshedding I can do. > Am 15.07.2017 um 14:49 schrieb Mauro