lkml.org 
[lkml]   [2017]   [Jun]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 2/5] Docs: Remove redundant geometry package inclusion
Em Fri, 23 Jun 2017 13:39:51 -0600
Jonathan Corbet <corbet@lwn.net> escreveu:

> On Mon, 19 Jun 2017 07:49:06 -0300
> Mauro Carvalho Chehab <mchehab@s-opensource.com> 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@s-opensource.com>
>
> 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

\
 
 \ /
  Last update: 2017-06-23 23:26    [W:0.080 / U:0.032 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site