[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <cbd31e8c-4e73-42f6-bb16-ed44091a353c@hust.edu.cn>
Date: Tue, 12 Aug 2025 12:35:02 +0800
From: Dongliang Mu <dzm91@...t.edu.cn>
To: Dan Carpenter <dan.carpenter@...aro.org>,
Haoyang LIU <tttturtleruss@...il.com>
Cc: Nathan Chancellor <nathan@...nel.org>,
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
Subject: Re: [PATCH] scripts/checktransupdate.py: add support for scanning
directory
On 8/11/25 3:18 PM, Dan Carpenter wrote:
> On Mon, Aug 11, 2025 at 12:17:30AM +0800, Haoyang LIU wrote:
>>
>> import os
>> @@ -131,7 +144,7 @@ def check_per_file(file_path):
>> opath = get_origin_path(file_path)
>>
>> if not os.path.isfile(opath):
>> - logging.error("Cannot find the origin path for {file_path}")
>> + logging.error(f"Cannot find the origin path for {file_path}")
> Send this as a separate patch with a Fixes tag.
> Fixes: 63e96ce050e5 ("scripts: fix all issues reported by pylint")
>
> Ideally, pylint should be modified to complain about this or something...
> I have a script for kernel patches which checks these kinds of mechanical
> changes and someone could make a similar script which checks pylint
> changes. https://github.com/error27/rename_rev
Yes, this seems really wired. We have a project - Linux Kernel Patch
Statistic among Universities[1]. In one PR[2], Haoyang also found a
quote string issue.
Interestingly, this PR is trying to fix issues raised by pylint.
[1] https://github.com/hust-open-atom-club/linux-edu-rank
[2] https://github.com/hust-open-atom-club/linux-edu-rank/pull/45
>
> regards,
> dan carpenter
>
Powered by blists - more mailing lists