linux-kernel - Re: [PATCH] docs/zh_CN: Update dev-tools/index.rst

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <871q51q2zq.fsf@trenco.lwn.net>
Date: Wed, 12 Jun 2024 15:12:57 -0600
From: Jonathan Corbet <corbet@....net>
To: Dongliang Mu <dzm91@...t.edu.cn>, Alex Shi <alexs@...nel.org>, Yanteng
 Si <siyanteng@...ngson.cn>, Haoyang Liu <tttturtleruss@...t.edu.cn>,
 Dongliang Mu <dzm91@...t.edu.cn>, Vegard Nossum <vegard.nossum@...cle.com>
Cc: linux-doc@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] docs/zh_CN: Update dev-tools/index.rst

Dongliang Mu <dzm91@...t.edu.cn> writes:

> Update to commit 8c88bc5b489e ("docs: dev-tools: Add UAPI checker
> documentation")
>
> Signed-off-by: Dongliang Mu <dzm91@...t.edu.cn>
> ---
>  Documentation/translations/zh_CN/dev-tools/index.rst | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/Documentation/translations/zh_CN/dev-tools/index.rst b/Documentation/translations/zh_CN/dev-tools/index.rst
> index fa900f5beb68..c540e4a7d5db 100644
> --- a/Documentation/translations/zh_CN/dev-tools/index.rst
> +++ b/Documentation/translations/zh_CN/dev-tools/index.rst
> @@ -20,18 +20,22 @@ Documentation/translations/zh_CN/dev-tools/testing-overview.rst
>  
>     testing-overview
>     sparse
> +   kcov
>     gcov
>     kasan
> -   kcov
>     ubsan
>     kmemleak
>     gdb-kernel-debugging
>  
>  Todolist:
>  
> + - checkpatch
>   - coccinelle
> + - kmsan
>   - kcsan
>   - kfence
>   - kgdb
>   - kselftest
>   - kunit/index
> + - ktap
> + - checkuapi

So I have to say that the changelog here is not particularly helpful.
You have *not* updated the translation to that commit, so it doesn't
seem like you should say that you did.  "Add several newish documents to
the todo list" is appropriate for something like this.

I've applied the patch, but with the changed commit message.

Thanks,

jon