Messages in this thread | | | Date | Mon, 27 Jul 2020 16:28:05 -0600 | From | Jonathan Corbet <> | Subject | Re: [PATCH] scripts/kernel-doc: optionally treat warnings as errors |
| |
On Fri, 24 Jul 2020 18:01:38 -0500 Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com> wrote:
> The kbuild bot recently added the W=1 option, which triggered > documentation cleanups to squelch hundreds of kernel-doc warnings. > > To make sure new kernel contributions don't add regressions to > kernel-doc descriptors, this patch suggests an option to treat > warnings as errors in CI/automated tests. A command-line option is > provided to the kernel-doc script, as well as a check on environment > variables to turn this optional behavior on. > > Examples for the two subsystems I contribute to: > > KCFLAGS="-Wall -Werror" make W=1 sound/ > KCFLAGS="-Wall -Werror" make W=1 drivers/soundwire/ > > Randy Dunlap also suggested adding a log for when generating > documentation. The documentation build is however not stopped for now. > > KDOC_WERROR=1 make htmldocs
So I'm not opposed to this, but I'm missing a couple of things in the changelog:
- A statement that you are adding a -Werror option that invokes this behavior.
- Mention of the fact that you also cause it to look at a couple of environment variables and change its behavior based on that.
Could I get a version with that clarified a bit?
Thanks,
jon
| |