Re: [PATCH 2/5] Docs: Remove redundant geometry package inclusion

[Mauro Carvalho Chehab]

Em Fri, 23 Jun 2017 13:39:51 -0600
Jonathan Corbet <corbet@xxxxxxx> escreveu:

> On Mon, 19 Jun 2017 07:49:06 -0300
> Mauro Carvalho Chehab <mchehab@xxxxxxxxxxxxxxxx> wrote:
> 
> > There's a catch, though: as sphinxsetup hmargin/vmargin was added
> > only on Sphinx version 1.5.3, the enclosed patch won't adjust the
> > margins for versions 1.5.0 to 1.5.2. While I didn't test, I bet it
> > will just ignore the keys, showing some warning.  
> 
> And people routinely accuse *me* of being overly optimistic...

:-)

> > ! Package keyval Error: hmargin undefined.
> > 
> > See the keyval package documentation for explanation.
> > Type  H <return>  for immediate help.
> >  ...                                              
> >                                                   
> > l.19 \sphinxsetup{hmargin=0.5in, vmargin=0.5in}
> >                                                
> > ?   
> 
> So it kills the build on 1.5.2.

Murphy's law applied to coding: Anything untested that can go wrong 
will go wrong :-)

Sorry for that.

> 
> I've made a tweak to special-case early 1.5.x.  
> 
> > Fixes: 85c21e5c3ee74fb75d690c57f7066bae7e2dca55
> > Sighed-off-by: Mauro Carvalho Chehab <mchehab@xxxxxxxxxxxxxxxx>  
> 
> Sighed-off-by indeed, I agree.  It's a good thing there's a lot of beer in
> the fridge...

Heh. Yeah, your changes look OK to me.

Btw, I tried for some time to fix pdf build with Sphinx 1.6 without
much luck. The new table macros version 1.6 adds makes almost
impossible to use adjustbox before a table, because it calls
vskip on them. The solution seems to either redefine those macros
or to override tabulary for it to always call adjustbox.

Unfortunately, I'm not fluent on LaTeX macros, so it takes me a
lot of time to work on it.

I'll try to look on it later, after the merge window.

Regards,
Mauro