linux-doc.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Akira Yokosawa <akiyks@gmail.com>
To: mchehab+huawei@kernel.org
Cc: corbet@lwn.net, linux-doc@vger.kernel.org,
	linux-kernel@vger.kernel.org, Akira Yokosawa <akiyks@gmail.com>
Subject: Re: [PATCH v2 07/14] docs: conf.py: extra cleanups and fixes
Date: Fri, 22 Aug 2025 21:01:03 +0900	[thread overview]
Message-ID: <f5d4804c-9a51-443a-a73e-d9c043673cbc@gmail.com> (raw)
In-Reply-To: <08d16c2ad817910eb5606842f776d3f77d83282f.1755763127.git.mchehab+huawei@kernel.org>

Hi Mauro,

I see this has already been merged, but let me comment after the fact.

On Thu, 21 Aug 2025 10:16:43 +0200, Mauro Carvalho Chehab wrote:
> Makes it more adehent with modern Sphinx LaTeX build setup as
> defined at:
> 
> 	https://www.sphinx-doc.org/en/master/latex.html
> 
> There, it suggests:
> 
> 1) to add: 'printindex'
> 
>     “printindex” call, the last thing in the file. Override if you want to generate
>     the index differently, append some content after the index, or change the font.
>     As LaTeX uses two-column mode for the index it is often advisable to set this
>     key to r'\footnotesize\raggedright\printindex'.
> 
> This indeed solved a corner case on a distro where the index was not properly
> generated.

This looks good.

> 
> 2) to add at the main example:
>          \PassOptionsToPackage{svgnames}{xcolor}

OK.  So I guess Sphinx's latex builder is going to start using SVG color
naming in the near future.

> 
> 3) I got a corner case on one of the distros was using xindy to produce
>    indexes. This ended causing the build logic to incorretly try to use
>    T1 fontenc, which is not UTF-8 compatible.
> 
>    This patch adds:
> 
>         \PassOptionsToPackage{xindy}{language=english,codepage=utf8,noautomatic}
> 
>    to cover such case. It should be noticed that, as the config doesn't
>    have \usepackage{xindy}, this will be used only if latexmk ends
>    using xindy.

But I can't see how this macro (executed by XeTeX engine) would have any
effect on xindy (command) invoked from latexmk.

Can you elaborate on your theory of how it works?
And which distro runs xindy?

Thanks,
Akira


  reply	other threads:[~2025-08-22 12:01 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-08-21  8:16 [PATCH v2 00/14] Fix PDF doc builds on major distros Mauro Carvalho Chehab
2025-08-21  8:16 ` [PATCH v2 01/14] docs: Makefile: Fix LaTeX paper size settings Mauro Carvalho Chehab
2025-08-21  8:16 ` [PATCH v2 02/14] docs: conf.py: better handle latex documents Mauro Carvalho Chehab
2025-08-21  8:16 ` [PATCH v2 03/14] docs: conf.py: fix doc name with SPHINXDIRS Mauro Carvalho Chehab
2025-08-21  8:16 ` [PATCH v2 04/14] docs: conf.py: rename some vars at latex_documents logic Mauro Carvalho Chehab
2025-08-21  8:16 ` [PATCH v2 05/14] docs: conf.py: use dedent and r-strings for LaTeX macros Mauro Carvalho Chehab
2025-08-21  8:16 ` [PATCH v2 06/14] docs: conf.py: fix some troubles for LaTeX output Mauro Carvalho Chehab
2025-08-21  8:16 ` [PATCH v2 07/14] docs: conf.py: extra cleanups and fixes Mauro Carvalho Chehab
2025-08-22 12:01   ` Akira Yokosawa [this message]
2025-08-22 13:27     ` Mauro Carvalho Chehab
2025-08-22 22:34       ` Jonathan Corbet
2025-08-25 12:07         ` Mauro Carvalho Chehab
2025-08-25 12:15           ` Jonathan Corbet
2025-08-25 17:11             ` Mauro Carvalho Chehab
2025-08-21  8:16 ` [PATCH v2 08/14] scripts: sphinx-pre-install: fix PDF build issues on Ubuntu Mauro Carvalho Chehab
2025-08-21  8:16 ` [PATCH v2 09/14] scripts: sphinx-pre-install: add missing gentoo pdf dependencies Mauro Carvalho Chehab
2025-08-21  8:16 ` [PATCH v2 10/14] scripts: sphinx-pre-install: fix PDF dependencies for openSuse Mauro Carvalho Chehab
2025-08-21  8:16 ` [PATCH v2 11/14] scripts: sphinx-pre-install: fix dependencies for OpenMandriva Mauro Carvalho Chehab
2025-08-21  8:16 ` [PATCH v2 12/14] scripts: sphinx-pre-install: fix pdf dependencies for Mageia 9 Mauro Carvalho Chehab
2025-08-21  8:16 ` [PATCH v2 13/14] scripts: sphinx-pre-install: fix PDF dependencies for gentoo Mauro Carvalho Chehab
2025-08-21  8:16 ` [PATCH v2 14/14] scripts/sphinx-pre-install: fix Archlinux PDF dependencies Mauro Carvalho Chehab
2025-08-21 19:22 ` [PATCH v2 00/14] Fix PDF doc builds on major distros Jonathan Corbet
2025-08-21 19:43   ` Mauro Carvalho Chehab

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=f5d4804c-9a51-443a-a73e-d9c043673cbc@gmail.com \
    --to=akiyks@gmail.com \
    --cc=corbet@lwn.net \
    --cc=linux-doc@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mchehab+huawei@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).