Re: [PATCH v2 0/3] Add Chinese translations for KSM documents

From: Jonathan Corbet <corbet@lwn.net>
To: cgel.zte@gmail.com, sterlingteng@gmail.com, seakeel@gmail.com
Cc: siyanteng01@gmail.com, linux-doc@vger.kernel.org,
	xu.xin16@zte.com.cn, cgel.zte@gmail.com
Subject: Re: [PATCH v2 0/3] Add Chinese translations for KSM documents
Date: Thu, 27 Jan 2022 11:45:33 -0700	[thread overview]
Message-ID: <87pmodqape.fsf@meer.lwn.net> (raw)
In-Reply-To: <20220117043455.861550-1-xu.xin16@zte.com.cn>

cgel.zte@gmail.com writes:

> From: xu xin <xu.xin16@zte.com.cn>
>
> According to the suggestions from Yanteng Si, Alex Shi and
> Jonathan Corbet, I have modified my patches. Here are my specific
> changelog:
>
> 	1. Remove Redundant Labels in added Documents like ``.. _ksm_sysfs:``
>
> 	   Yanteng Si said: Too many tags will cause a compilation
> 	   warning, because an identical one already exists for the Origin
> 	   document. Jonathan Corbet think so.
>
> 	2. Align with standard Chinese format on the 'original', 'translator',
> 	   etc.
>
> 	3. fix some translation error like “pages_unshared”, I remove  '独享'.
>
> 	   Alex Shi suggest to remove '独享'.
>
> 	4. Refactor translations/zh_CN/admin-guide/mm/index.rst.
>
> 	   Yanteng Si: Compile pass is only one of the basis for checking
> 	   through, we also have to check the html for any problems, as
> 	   far as I know, the above treatment is very ugly.
>
> I have rebased this patch series on top of Yanteng Si's series on his
> commit:
> https://lore.kernel.org/linux-doc/60c579b34792c4c76194c4843a695263a982b37d.1641866889.git.siyanteng@loongson.cn/
>
> xu xin (3):
>   Add Chinese translation for vm/ksm.rst
>   Add Chinese translations for admin-guide/mm/ksm.rst
>   Add admin-guide/mm/index.rst
>
>  .../translations/zh_CN/admin-guide/index.rst       |   2 +-
>  .../translations/zh_CN/admin-guide/mm/index.rst    |  49 +++++++
>  .../translations/zh_CN/admin-guide/mm/ksm.rst      | 148
> +++++++++++++++++++++
>  Documentation/translations/zh_CN/vm/index.rst      |   1 +
>  Documentation/translations/zh_CN/vm/ksm.rst        |  70 ++++++++++
>  5 files changed, 269 insertions(+), 1 deletion(-)
>  create mode 100644
> Documentation/translations/zh_CN/admin-guide/mm/index.rst
>  create mode 100644
> Documentation/translations/zh_CN/admin-guide/mm/ksm.rst
>  create mode 100644 Documentation/translations/zh_CN/vm/ksm.rst

Unfortunately, these patches don't apply.  Among other things, they have
the famous charset=y problem.  Please regenerate, email to yourself, and
verify that you can apply the result before sending again.

Thanks,

jon