Re: [PATCH v2 4/4] docs-rst: Allow Sphinx version 1.6

[Mauro Carvalho Chehab <mchehab@s-opensource.com>]

Em Thu, 24 Aug 2017 13:29:14 -0600
Jonathan Corbet <corbet@lwn.net> escreveu:

> On Thu, 24 Aug 2017 13:26:28 -0600
> Jonathan Corbet <corbet@lwn.net> wrote:
> 
> > > -	% To allow adjusting table sizes
> > > -	\\usepackage{adjustbox}
> > > -
> > >       '''
> > >  }    
> > 
> > So this change doesn't quite match the changelog...what's the story there?  
> 
> Indeed, with that patch applied, I get this:
> 
> ! LaTeX Error: Environment adjustbox undefined.
> 
> See the LaTeX manual or LaTeX Companion for explanation.
> Type  H <return>  for immediate help.
>  ...                                              
>                                                   
> l.4108 \begin{adjustbox}
>                         {width=\columnwidth}
> 
> ...so methinks something isn't quite right...

Sorry, yeah, I messed with this one. Just sent a new series of patches,
splitting it from the documentation changes.

This change (and the equivalent one at sphinx-pre-build script) is
actually a cleanup change. It can *only* be applied after the
patch that goes through the media tree.

What happens is that this patch:
	media: fix pdf build with Spinx 1.6

Stops using adjustbox, with is what causes build problem on Sphinx 1.6.

I suspect that now the PDF build will be a bit more reliable, as the
"..raw: latex" blocs now only adjust font sizes and table spacing,
when needed to output large tables, but Sphinx maintainers have been
creative on finding new ways to break backward compatibility on every
new version ;-) Still, hope is the last thing to die.

Thanks,
Mauro