Subject: Re: [PATCH 1/1] scripts: sphinx-pre-install: add required ctex
 dependency
To:     Mauro Carvalho Chehab <mchehab@kernel.org>
Cc:     hch@lst.de, linux-doc@vger.kernel.org,
        linux-kernel@vger.kernel.org, corbet@lwn.net
References: <87mtkcxws9.fsf@meer.lwn.net>
 <eeb773ef-0354-be1f-8438-07e3324fa9f6@gmail.com>
 <07f8cf68-525b-4fca-a1f9-fd40150fc6e1@gmail.com>
 <20220104090548.76112534@coco.lan>
 <73e62b16-7685-ace2-18d4-e1b79f543caf@gmail.com>
 <20220105084809.157e3f34@coco.lan>
 <241d31e4-5447-a075-69b6-11c14427582d@gmail.com>
 <20220105224230.0700351f@coco.lan>
From:   Akira Yokosawa <akiyks@gmail.com>
Message-ID: <84f90b41-7a18-d83c-8739-1d394175f74a@gmail.com>
Date:   Thu, 6 Jan 2022 07:57:22 +0900
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101
 Thunderbird/78.14.0
MIME-Version: 1.0
In-Reply-To: <20220105224230.0700351f@coco.lan>
Content-Type: text/plain; charset=utf-8
Content-Language: en-US
Content-Transfer-Encoding: 7bit
Precedence: bulk

On Wed, 5 Jan 2022 22:42:30 +0100, Mauro Carvalho Chehab wrote:
> Em Wed, 5 Jan 2022 21:09:59 +0900
> Akira Yokosawa <akiyks@gmail.com> escreveu:
> 
[...]
>>
>> I'll test it under various distros/setups.
>> It may take a while.
> 
> Did a quick test here with this lxc container:
> 
> 	$ OS=debian REL=bullseye ;lxc-create -n ${OS} -t /usr/share/lxc/templates/lxc-download -- --dist ${OS} --release ${REL} --arch amd64
> 
> If we add check_missing_tex(), it will then complain about lots
> of packages that don't exist in Debian (se logs at the end).

Now I think I understand what kind of tests we need.

My plan was to intentionally uninstall texlive-ctex (or whichever
package contains ctexhook.sty) and see if the lack of ctexhook.sty
is detected or not.

Apparently, such a test would mean little for sphinx_pre_install.

> 
> That's one of the biggest issue with texlive: each distro splits it on
> different ways. So, basically, the %texlive hash at the script is meant
> to the way Fedora and openSuse distros usually map packages. I would
> expect that Mageia would also be similar if not identical, but other
> distros map packages on different ways.
> 
> I'll prepare another patch for fixing the debian issue.

Then I'll wait for it.

        Thanks, Akira

> 
> Regards,
> Mauro
> 
> ---
> 
> mchehab@debian:~/docs$ make pdfdocs
> Documentation/Makefile:41: The 'sphinx-build' command was not found. Make sure you have Sphinx installed and in PATH, or set the SPHINXBUILD make variable to point to the full path of the 'sphinx-build' executable.
> 
> Detected OS: Debian GNU/Linux 11.
> Warning: better to also install "convert".
> Warning: better to also install "dot".
> Warning: better to also install "dvipng".
> Warning: better to also install "fonts-dejavu".
> Warning: better to also install "fonts-noto-cjk".
> ERROR: please install "gcc", otherwise, build won't work.
> Warning: better to also install "latexmk".
> ERROR: please install "python", otherwise, build won't work.
> Warning: better to also install "rsvg-convert".
> Warning: better to also install "texlive-amscls".
> Warning: better to also install "texlive-amsfonts".
> Warning: better to also install "texlive-amsmath".
> Warning: better to also install "texlive-anyfontsize".
> Warning: better to also install "texlive-capt-of".
> Warning: better to also install "texlive-cmap".
> Warning: better to also install "texlive-ctex".
[...]