On Wed, May 21, 2025 at 11:02:12AM +0200, Mauro Carvalho Chehab wrote: > Hi Jon, > > That's the third version of the kernel-doc fixup patch series. > > It address the root cause why Sphinx logger was not working: there > was a call there for logger.verbose(). According with: > > https://www.sphinx-doc.org/en/master/extdev/logging.html > > This is a valid call, but it doesn't verbose messages. Instead, it is > crashing with modern Sphinx versions, causing the log to not work. > > I got rid of it, replacing by logger.info(). I took the time to also > address an issue pointed by Andy: not having the same log message > placed everywhere. With such change, we can keep using Sphinx > logger (which produces colored messages) inside kernel-doc > classes. > > With that, we have: > > Patch 1: makes Lore and kernel-doc ML receive patches related > to kernel-doc.py and get_abi.py. > Patch 2: cleanup try/except logic and get rid of logger.verbose(); > Patch 3: fix a KeyError when trying to acess data from non-existing files; > > If you test just patch 1 on the top of next-20250516, you'll see the > keyerror message (in red): > > ERROR: Cannot find file ./drivers/gpio/gpiolib-acpi.c > ERROR: Cannot find file ./drivers/gpio/gpiolib-acpi.c > WARNING: kernel-doc './scripts/kernel-doc.py -rst -enable-lineno -export ./drivers/gpio/gpiolib-acpi.c' processing failed with: KeyError('./drivers/gpio/gpiolib-acpi.c') > > And the script doesn't crash anymore. After patch 2, the try/except > warning gets replaced by a proper message: > > ERROR: Cannot find file ./drivers/gpio/gpiolib-acpi.c > ERROR: Cannot find file ./drivers/gpio/gpiolib-acpi.c > WARNING: No kernel-doc for file ./drivers/gpio/gpiolib-acpi.c Sounds reasonable to me. Acked-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> -- With Best Regards, Andy Shevchenko
