All of lore.kernel.org
 help / color / mirror / Atom feed
From: Weijie Yuan <wy@wyuan.org>
To: Dongliang Mu <dzm91@hust.edu.cn>
Cc: Alex Shi <alexs@kernel.org>, Yanteng Si <si.yanteng@linux.dev>,
	Jonathan Corbet <corbet@lwn.net>,
	Shuah Khan <skhan@linuxfoundation.org>,
	Randy Dunlap <rdunlap@infradead.org>,
	linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 1/2] docs/zh_CN: remove translator-related headers
Date: Sat, 17 Oct 2026 17:13:45 +0800	[thread overview]
Message-ID: <atM8STwzREGJ_-Vd@wyuan.org> (raw)
In-Reply-To: <atMxWvnyYmD5r6O3@wyuan.org>

On 2026-10-17 16:27:12 +0800, Weijie Yuan <wy@wyuan.org> wrote:

> On 2026-10-17 10:06:39 +0800, Dongliang Mu <dzm91@hust.edu.cn> wrote:
> 
> > On 10/17/26 12:52 AM, Weijie Yuan wrote:
> > > The headers related to translators and proofreaders appear multiple
> > > times in our directory. Their information is duplicated with that in the
> > > commit logs, making it difficult to maintain over an extended period
> > > of time, as there have been multiple instances of requests to update the
> > > email address.
> > > 
> > > Remove all the related fields to make our source clean.
> > > 
> > > Suggested-by: Jonathan Corbet <corbet@lwn.net>
> > > Signed-off-by: Weijie Yuan <wy@wyuan.org>
> > > ---
> > > The diffs are inconvenient to review, sorry. Because I cannot find a
> > > better way to split them up besides by directories.
> > 
> > 1. I recommend to preserve the RST anchors because other files would use
> > this.
> 
> Yes, I also removed some top-level labels while I was at it, though I
> would better have done that in a separate patch. (My Git index was messy
> with these large diff. So later, the topic was expanded to a cleanup.
> But yes, I should split it)
> 
> But I checked that none of these removed labels are referenced
> elsewhere, if I was doing it correctly. (No invalid citations were
> reported by 'make htmldocs'.)

Oops, I mean the removed labels are not used by other places according
to 'make htmldocs'.

Thanks.

> 
> Anyway, I'll not touch these labels in next re-roll.
> 
> > 2. There are three checkpatch SPDX warnings during the review:
> > 
> > dev-tools/sparse.rst | Missing or malformed SPDX-License-Identifier tag in
> > line 1
> > process/howto.rst | Missing or malformed SPDX-License-Identifier tag in line
> > 1
> > process/kernel-enforcement-statement.rst | Missing or malformed
> > SPDX-License-Identifier tag in line 1
> > 
> > Each warning is triggered because the patch changes that file´s first line.
> > All three files already lacked an SPDX tag before the patch.
> 
> Oh, I forgot to mention this when writing cover lettter, sorry I did
> send these emails a bit too hastily.
> 
> I did run checkpatch and got these reminders.
> 
> The original Engish versions of these three are missing
> SPDX-License-Identifier(s), so as far as I know, our translation is
> better not to add this first.?
> 
> Please see: https://lore.kernel.org/linux-doc/87y0dv1wp3.fsf@trenco.lwn.net/
> 
> But I guess we could add the identifier with Alex's and Yanteng's ack,
> as they are the original translator if we want to.
> 
> Thanks!
> 
> 
> p.s. Many of the recipients' addresses are no longer in use. To avoid
> unnecessary clutter, I will simplify the list of recipients in the next
> version.



  reply	other threads:[~2026-10-17  9:13 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-10-16 16:52 [PATCH 0/2] docs/zh_CN: cleanup headers and fix minor issues Weijie Yuan
2026-10-16 16:52 ` [PATCH 1/2] docs/zh_CN: remove translator-related headers Weijie Yuan
2026-10-17  2:06   ` Dongliang Mu
2026-10-17  8:27     ` Weijie Yuan
2026-10-17  9:13       ` Weijie Yuan [this message]
2026-10-16 16:52 ` [PATCH 2/2] docs/zh_CN: remove Translator field in header template Weijie Yuan

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=atM8STwzREGJ_-Vd@wyuan.org \
    --to=wy@wyuan.org \
    --cc=alexs@kernel.org \
    --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=si.yanteng@linux.dev \
    --cc=skhan@linuxfoundation.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.