Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <278e0041-a2df-d067-59eb-87b8ac178567@gmail.com>
Date:   Fri, 12 Aug 2022 00:33:34 +0900
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101
 Thunderbird/91.11.0
Subject: Re: [PATCH v2 3/3] docs: kerneldoc-preamble: Test xeCJK.sty before
 loading
Content-Language: en-US
To:     Jonathan Corbet <corbet@lwn.net>
Cc:     linux-doc@vger.kernel.org,
        Mauro Carvalho Chehab <mchehab+huawei@kernel.org>,
        linux-kernel@vger.kernel.org, Akira Yokosawa <akiyks@gmail.com>,
        Mauro Carvalho Chehab <mchehab@kernel.org>
References: <c41cab17-afd6-bc99-56a1-e4e73b8c1ef6@gmail.com>
 <c24c2a87-70b2-5342-bcc9-de467940466e@gmail.com>
From:   Akira Yokosawa <akiyks@gmail.com>
In-Reply-To: <c24c2a87-70b2-5342-bcc9-de467940466e@gmail.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: bulk

Hi Jon,

On Mon, 8 Aug 2022 17:53:57 +0900, Akira Yokosawa wrote:
> On distros whose texlive packaging is fine-grained, texlive-xecjk
> can be installed/removed independently of other texlive packages.
> Conditionally loading xeCJK depending only on the existence of the
> "Noto Sans CJK SC" font might end up in xelatex error of
> "xeCJK.sty not found!".
> 
> Improve the situation by testing existence of xeCJK.sty before
> loading it.
> 
> This is useful on RHEL 9 and its clone distros where texlive-xecjk
> doesn't work at the moment due to a missing dependency [1].
> "make pdfdocs" for non-CJK contents should work after removing
> texlive-xecjk.
In hindsight, I think this qualifies as a stable material.
Let me append a couple of tags.

> 
> Link: [1] https://bugzilla.redhat.com/show_bug.cgi?id=2086254
> Signed-off-by: Akira Yokosawa <akiyks@gmail.com>
> Acked-by: Mauro Carvalho Chehab <mchehab@kernel.org>
Fixes: 398f7abdcb7e ("docs: pdfdocs: Pull LaTeX preamble part out of conf.py")
Cc: stable@vger.kernel.org # v5.18+

Jon, it's up to you whether this one is destined for v6.0-rcX or
for the next merge window.  It has no dependency on patches 1/3
and 2/3.

        Thanks, Akira