Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1705834 > unrolled thread

Re: [PATCH] scripts/sphinx-pre-install: add minimum support for RHEL

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  Re: [PATCH] scripts/sphinx-pre-install: add minimum support for  RHEL Jonathan Corbet <corbet@lwn.net> - 2017-08-07 22:30 +0200
    Re: [PATCH] scripts/sphinx-pre-install: add minimum support for  RHEL Mauro Carvalho Chehab <mchehab@s-opensource.com> - 2017-08-08 11:00 +0200

#1705834 — Re: [PATCH] scripts/sphinx-pre-install: add minimum support for RHEL

On Mon, 24 Jul 2017 09:09:24 -0300
Mauro Carvalho Chehab <mchehab@s-opensource.com> wrote:

> RHEL 7.x and clone distros are shipped with Sphinx 1.1.x,
> with is incompatible with Kernel ReST markups.
> 
> So, on those systems, the only alternative is to install
> it via a Python virtual environment.
> 
> While seeking for "pip" on CentOS 7.3, I noticed that it
> is not really needed, as python-virtualenv has its version
> packaged there already. So, remove this from the list of
> requirements for all distributions.
> 
> With regards to PDF, twe need at least texlive-tabulary
> extension, but that is not shipped there (at least on
> CentOS). So, disable PDF packages as a hole.
> 
> Please notice, however, that texlive + amsmath is needed for
> ReST to properly handle ReST ".. math::" tags. Yet, Sphinx
> fall back to display the LaTeX math expressions as-is, if
> such extension is not available.
> 
> So, let's just disable all texlive packages as a hole.

I've applied this, finally.  Couldn't resist the temptation to fix "as a
whole" while I was at it :)

Thanks,

jon

[toc] | [next] | [standalone]

#1706210

Em Mon, 7 Aug 2017 14:27:33 -0600
Jonathan Corbet <corbet@lwn.net> escreveu:

> On Mon, 24 Jul 2017 09:09:24 -0300
> Mauro Carvalho Chehab <mchehab@s-opensource.com> wrote:
> 
> > RHEL 7.x and clone distros are shipped with Sphinx 1.1.x,
> > with is incompatible with Kernel ReST markups.
> > 
> > So, on those systems, the only alternative is to install
> > it via a Python virtual environment.
> > 
> > While seeking for "pip" on CentOS 7.3, I noticed that it
> > is not really needed, as python-virtualenv has its version
> > packaged there already. So, remove this from the list of
> > requirements for all distributions.
> > 
> > With regards to PDF, twe need at least texlive-tabulary
> > extension, but that is not shipped there (at least on
> > CentOS). So, disable PDF packages as a hole.
> > 
> > Please notice, however, that texlive + amsmath is needed for
> > ReST to properly handle ReST ".. math::" tags. Yet, Sphinx
> > fall back to display the LaTeX math expressions as-is, if
> > such extension is not available.
> > 
> > So, let's just disable all texlive packages as a hole.  
> 
> I've applied this, finally.  Couldn't resist the temptation to fix "as a
> whole" while I was at it :)

Thanks for applying - and for fixing the "whole" typo ;)

Yeah, RHEL/CentOS was likely the big missing gap at the script. With time,
people may end adding other distros there too. As most distros either
use .deb or .rpm packaging[1], I suspect it shouldn't be hard to add support
for them when needed, as they'll end by either use the existing logic,
or require a close enough logic.

[1] Well, last time I checked, Slackware still use tarballs. Slackware
was the first distro I used, back on Kernel 1.0 days. If I ever become 
nostalgic enough, I might eventually write a patch for it some day.

Thanks,
Mauro

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web