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;
From:   Jonathan Corbet <corbet@lwn.net>
To:     Akira Yokosawa <akiyks@gmail.com>
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>
Subject: Re: [PATCH v2 3/3] docs: kerneldoc-preamble: Test xeCJK.sty before
 loading
In-Reply-To: <278e0041-a2df-d067-59eb-87b8ac178567@gmail.com>
References: <c41cab17-afd6-bc99-56a1-e4e73b8c1ef6@gmail.com>
 <c24c2a87-70b2-5342-bcc9-de467940466e@gmail.com>
 <278e0041-a2df-d067-59eb-87b8ac178567@gmail.com>
Date:   Thu, 18 Aug 2022 11:28:34 -0600
Message-ID: <87o7whbgvx.fsf@meer.lwn.net>
MIME-Version: 1.0
Content-Type: text/plain
Precedence: bulk

Akira Yokosawa <akiyks@gmail.com> writes:

> 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.

OK, I've gone ahead and applied it with those tags.

Thanks,

jon