docs: conf.py: drop xindy rule

[PATCH] docs: conf.py: drop xindy rule

Posted by Mauro Carvalho Chehab 5 months, 2 weeks ago

The rule as-is is wrong, as it was inverted. Besides that, after
retest building all repos with suggested LaTeX packages given
by sphinx-pre-install, I was unable to reproduce the issues
I saw with xindy in the past.

So, let's just drop. If anyone reports issues with xindy, we
may need to readd, but at the right way, e.g. {options}{pkgname}.

Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
---
 Documentation/conf.py | 2 --
 1 file changed, 2 deletions(-)

diff --git a/Documentation/conf.py b/Documentation/conf.py
index 8fcecdb927b1..574896cca198 100644
--- a/Documentation/conf.py
+++ b/Documentation/conf.py
@@ -458,8 +458,6 @@ latex_elements = {
     "papersize": "a4paper",
     "passoptionstopackages": dedent(r"""
         \PassOptionsToPackage{svgnames}{xcolor}
-        % Avoid encoding troubles when creating indexes
-        \PassOptionsToPackage{xindy}{language=english,codepage=utf8,noautomatic}
     """),
     # The font size ('10pt', '11pt' or '12pt').
     "pointsize": "11pt",
-- 
2.51.0

Re: [PATCH] docs: conf.py: drop xindy rule

Posted by Jonathan Corbet 5 months, 1 week ago

Mauro Carvalho Chehab <mchehab+huawei@kernel.org> writes:

> The rule as-is is wrong, as it was inverted. Besides that, after
> retest building all repos with suggested LaTeX packages given
> by sphinx-pre-install, I was unable to reproduce the issues
> I saw with xindy in the past.
>
> So, let's just drop. If anyone reports issues with xindy, we
> may need to readd, but at the right way, e.g. {options}{pkgname}.
>
> Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
> ---
>  Documentation/conf.py | 2 --
>  1 file changed, 2 deletions(-)
>
> diff --git a/Documentation/conf.py b/Documentation/conf.py
> index 8fcecdb927b1..574896cca198 100644
> --- a/Documentation/conf.py
> +++ b/Documentation/conf.py
> @@ -458,8 +458,6 @@ latex_elements = {
>      "papersize": "a4paper",
>      "passoptionstopackages": dedent(r"""
>          \PassOptionsToPackage{svgnames}{xcolor}
> -        % Avoid encoding troubles when creating indexes
> -        \PassOptionsToPackage{xindy}{language=english,codepage=utf8,noautomatic}
>      """),
>      # The font size ('10pt', '11pt' or '12pt').
>      "pointsize": "11pt",

Applied, thanks.

jon