From: Mauro Carvalho Chehab <mchehab@s-opensource.com>
To: Markus Heiser <markus.heiser@darmarit.de>
Cc: Linux Doc Mailing List <linux-doc@vger.kernel.org>,
Mauro Carvalho Chehab <mchehab@infradead.org>,
linux-kernel@vger.kernel.org, Jonathan Corbet <corbet@lwn.net>
Subject: Re: [PATCH RFC] scripts/sphinx-pre-install: add a script to check Sphinx install
Date: Fri, 14 Jul 2017 23:21:40 -0300 [thread overview]
Message-ID: <20170714232140.0608da3c@vento.lan> (raw)
In-Reply-To: <042D234A-6B3D-43D4-84BA-8332C0596AA4@darmarit.de>
Em Fri, 14 Jul 2017 19:35:59 +0200
Markus Heiser <markus.heiser@darmarit.de> escreveu:
> > Am 14.07.2017 um 18:49 schrieb Mauro Carvalho Chehab <mchehab@s-opensource.com>:
> >
> > Solving Sphinx dependencies can be painful. Add a script to
> > check if everything is ok.
>
> just my 5cent:
>
> What we need is a "requirements.txt" file to define a
> **reference environment**. E.g. to stick Sphinx 1.4.9 in
> such a reference environment::
>
> <snip: requirements.txt> ---
> Sphinx==1.4.9
> sphinx_rtd_theme
> <snap> ---------------------
>
> The rest is similarly to what you wrote in doc-guide/sphinx.rst ...
>
> The ref-environment can be build with virtualenv & pip::
>
> $ virtualenv --python=python3 docenv
> (doc-env) $ source ./docenv/bin/activate
> (doc-env) $ pip install -r requirements.txt
>
> From now we can start our build as usual. If not already done,
> first activate the environment::
>
> $ . ./docenv/bin/activate
> (doc-env) $ make htmldocs
>
> This (requirements.txt) is the way python packaging goes.
The above assumes that the user wants to use virtenv and
have python3, virtualenv3 and pip3 already installed.
I agree that a virtual environment works better than using
distro-specific packaging, as Sphinx toolchain is really
fragile. But we should give an option for the developer to
use whatever he wants.
I'm actually thinking that the final version of this script
is to have a command line parameter to would allow selecting
between virtenv or package install for Sphinx.
IMHO, virtenv should be the default.
> Ok, this won't solve TeX installation problems of Linux distros,
> for this a script like the one here in your RFC is helpful.
Your proposal doesn't solve Python, TeX, GraphViz nor
ImageMagick dependencies.
The TeX dependencies are the hardest ones, and, whatever
solution we take, this should be taken into account.
Thanks,
Mauro
next prev parent reply other threads:[~2017-07-15 2:21 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-07-14 16:49 [PATCH RFC] scripts/sphinx-pre-install: add a script to check Sphinx install Mauro Carvalho Chehab
2017-07-14 17:35 ` Markus Heiser
2017-07-14 22:51 ` Jim Davis
2017-07-15 1:27 ` Mauro Carvalho Chehab
2017-07-15 2:21 ` Mauro Carvalho Chehab [this message]
2017-07-15 9:51 ` Markus Heiser
2017-07-15 12:49 ` Mauro Carvalho Chehab
2017-07-15 12:55 ` Mauro Carvalho Chehab
2017-07-15 19:27 ` Mauro Carvalho Chehab
2017-07-16 7:29 ` Markus Heiser
2017-07-16 9:57 ` Mauro Carvalho Chehab
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20170714232140.0608da3c@vento.lan \
--to=mchehab@s-opensource.com \
--cc=corbet@lwn.net \
--cc=linux-doc@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=markus.heiser@darmarit.de \
--cc=mchehab@infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox