mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Weijie Yuan <kernel@wyuan.org>
To: Dongliang Mu <dzm91@hust.edu.cn>
Cc: Jonathan Corbet <corbet@lwn.net>,
	Shuah Khan <skhan@linuxfoundation.org>,
	Randy Dunlap <rdunlap@infradead.org>, Weijie Yuan <wy@wyuan.org>,
	Chen-Yu Yeh <chenyou910331@gmail.com>,
	linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] docs/zh_TW: remove top-level labels from process/
Date: Thu, 29 Oct 2026 20:13:16 +0800	[thread overview]
Message-ID: <auM4XND4sgE9kko_@wyuan.org> (raw)
In-Reply-To: <d32825a1-8da4-41f3-964b-8854618890d5@hust.edu.cn>

On Thu, Oct 29, 2026 at 07:50:43PM +0800, Dongliang Mu wrote:
> 
> On 10/29/26 5:26 PM, Weijie Yuan wrote:
> > I happened to notice that there was actually an embarrassing label on
> > this page [*] that had been rendered in HTML.
> > 
> >   * https://docs.kernel.org/translations/zh_TW/process/kernel-enforcement-statement.html
> > 
> > Therefore let me drop these labels in process dir but leave the cleanup
> > job later.
> > 
> > --- >8 ---
> > From: Weijie Yuan <wy@wyuan.org>
> > 
> > We can link to these documents by path, so the top-level labels
> > do not buy us anything.
> > 
> > Drop these redundant labels from the Traditional Chinese process
> > documents.
> 
> Hi Weijie,
> 
> Removing these labels breaks existing zh_TW cross-references. The patch
> deletes 25 labels, but 17 of them are still targets of 25 :ref: uses.
> 
> Dongliang Mu

Thanks for your quick check! Indeed, there are many of them being used.
I'm doing this stupidly, sorry.

Then I'm wondering why 'make htmldocs' did not complain about this..

[compiled again..] OK, that's because my machine stuck for a moment.
Sphinx did complain about this, ouch.

Anyway, the actual reason of this patch is here:

> > diff --git a/Documentation/translations/zh_TW/process/kernel-enforcement-statement.rst b/Documentation/translations/zh_TW/process/kernel-enforcement-statement.rst
> > index d54d20529294..45bb50fd1e6a 100644
> > --- a/Documentation/translations/zh_TW/process/kernel-enforcement-statement.rst
> > +++ b/Documentation/translations/zh_TW/process/kernel-enforcement-statement.rst
> > @@ -1,7 +1,5 @@
> >   .. SPDX-License-Identifier: GPL-2.0
> > -.. _tw_process_statement_kernel:
> > -
> >   .. include:: ../disclaimer-zh_TW.rst
> >   :Original: :ref:`Documentation/process/kernel-enforcement-statement.rst <process_statement_kernel>`

There is an invisible U+FEFF character before '.. _tw_process_statement_kernel:'
prevents it from being recognized as a label, causing it to be rendered
as plain text in the HTML output.

Will handle these issues soon.

Thanks!

      reply	other threads:[~2026-10-29 12:13 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-10-29  9:26 Weijie Yuan
2026-10-29 11:50 ` Dongliang Mu
2026-10-29 12:13   ` Weijie Yuan [this message]

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=auM4XND4sgE9kko_@wyuan.org \
    --to=kernel@wyuan.org \
    --cc=chenyou910331@gmail.com \
    --cc=corbet@lwn.net \
    --cc=dzm91@hust.edu.cn \
    --cc=linux-doc@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=rdunlap@infradead.org \
    --cc=skhan@linuxfoundation.org \
    --cc=wy@wyuan.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

all inboxes | Powered by JetHome®