linux-kernel - Re: [PATCH V2] scripts/checktransupdate.py: add support for scanning directory

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: <20250811212446.GA924610@ax162>
Date: Mon, 11 Aug 2025 14:24:46 -0700
From: Nathan Chancellor <nathan@...nel.org>
To: Haoyang LIU <tttturtleruss@...il.com>, Dongliang Mu <dzm91@...t.edu.cn>,
	Yanteng Si <si.yanteng@...ux.dev>, Alex Shi <alexs@...nel.org>,
	Jonathan Corbet <corbet@....net>
Cc: Nick Desaulniers <nick.desaulniers+lkml@...il.com>,
	Bill Wendling <morbo@...gle.com>,
	Justin Stitt <justinstitt@...gle.com>,
	hust-os-kernel-patches@...glegroups.com,
	linux-kernel@...r.kernel.org, llvm@...ts.linux.dev,
	linux-doc@...r.kernel.org
Subject: Re: [PATCH V2] scripts/checktransupdate.py: add support for scanning
 directory

Hi Haoyang,

On Tue, Aug 12, 2025 at 01:00:50AM +0800, Haoyang LIU wrote:
> Origin script can only accept a file as parameter, this commit enables
> it to scan a directory.
> 
> Usage example:
> ./scripts/checktransupdate.py Documentation/translations/zh_CN/dev-tools
> 
> Signed-off-by: Haoyang LIU <tttturtleruss@...il.com>
> ---
> 
> V1 -> V2: remove the fix of missing "f" in f-string and make it a new patch
> 
>  scripts/checktransupdate.py | 24 ++++++++++++++++++++++++
>  1 file changed, 24 insertions(+)

Thanks for the update. While this seems reasonable to me from a purely
surface level glance over the actual Python, I have added the
Documentation folks that Dongliang added from the previous thread, who
really own and maintain this file (the original patch is at [1]). Please
include them in future revisions should they be necessary. It would
probably be good for something like this to be applied?

diff --git a/MAINTAINERS b/MAINTAINERS
index fe168477caa4..b7e3a8c8832e 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -7302,6 +7302,7 @@ P:	Documentation/doc-guide/maintainer-profile.rst
 T:	git git://git.lwn.net/linux.git docs-next
 F:	Documentation/
 F:	scripts/check-variable-fonts.sh
+F:	scripts/checktransupdate.py
 F:	scripts/documentation-file-ref-check
 F:	scripts/get_abi.py
 F:	scripts/kernel-doc*

[1]: https://lore.kernel.org/20250811170050.94997-1-tttturtleruss@gmail.com/

Cheers,
Nathan