Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1673799 > unrolled thread
| Started by | Jonathan Corbet <corbet@lwn.net> |
|---|---|
| First post | 2017-06-23 21:50 +0200 |
| Last post | 2017-06-23 23:30 +0200 |
| Articles | 2 — 2 participants |
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.
Re: [PATCH 2/5] Docs: Remove redundant geometry package inclusion Jonathan Corbet <corbet@lwn.net> - 2017-06-23 21:50 +0200
Re: [PATCH 2/5] Docs: Remove redundant geometry package inclusion Mauro Carvalho Chehab <mchehab@s-opensource.com> - 2017-06-23 23:30 +0200
| From | Jonathan Corbet <corbet@lwn.net> |
|---|---|
| Date | 2017-06-23 21:50 +0200 |
| Subject | Re: [PATCH 2/5] Docs: Remove redundant geometry package inclusion |
| Message-ID | <tVCKC-6lJ-7@gated-at.bofh.it> |
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.
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...
jon
[toc] | [next] | [standalone]
| From | Mauro Carvalho Chehab <mchehab@s-opensource.com> |
|---|---|
| Date | 2017-06-23 23:30 +0200 |
| Message-ID | <tVEjn-7nt-3@gated-at.bofh.it> |
| In reply to | #1673799 |
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
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web