linux-kernel - Re: [PATCH 1/1] docs: kerneldoc.py: don't use Sphinx logger

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 for Android: free password hash cracker in your pocket

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

Message-ID: <20250520213906.6ec263d8@canb.auug.org.au>
Date: Tue, 20 May 2025 21:39:06 +1000
From: Stephen Rothwell <sfr@...b.auug.org.au>
To: Mauro Carvalho Chehab <mchehab+huawei@...nel.org>
Cc: Akira Yokosawa <akiyks@...il.com>, Randy Dunlap <rdunlap@...radead.org>,
 Jonathan Corbet <corbet@....net>, Andy Shevchenko
 <andriy.shevchenko@...ux.intel.com>, Nicolas Schier
 <nicolas.schier@...ux.dev>, linux-doc@...r.kernel.org,
 linux-kbuild@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 1/1] docs: kerneldoc.py: don't use Sphinx logger

Hi Mauro,

I do appreciate your work on the document system.  Particularly you
have made the process much faster.  However, see below.

On Tue, 20 May 2025 11:19:56 +0200 Mauro Carvalho Chehab <mchehab+huawei@...nel.org> wrote:
>
> If by "innocent typo" you mean removing/renaming/moving a file from
> the Kernel tree without updating Makefiles, or adding them with wrong
> names, then yes: I do think this should be fatal for the affected "make" 
> targets. The quickest such build issues are caught, the best for everyone.

They are found if you leave them as warnings and just continue on.  I
have been finding them by when the current python version just crashes
(without any useful message before your current change) I rerun the
htmldocs using the perl version to get the name of the missing file and
then report it so it can be fixed.  But the perl version continues on
and produces all the other warnings so I can compare with the previous
and only report new errors/warnigs.  Sometime these reports are not
foxed for days or weeks (or months).

So, please, now that you are producing more warnings, make a missing
file just a warning (or at least continue on).  I have enough to do
every day without having to run "make htmldocs" more than once.

-- 
Cheers,
Stephen Rothwell

Content of type "application/pgp-signature" skipped