Re: [PATCH v1 1/1] Makefile.extrawarn: Turn off -Werror when extra warnings are enabled

From: Andy Shevchenko
Date: Fri Mar 25 2022 - 16:00:29 EST


On Fri, Mar 25, 2022 at 10:18:06AM -0700, Linus Torvalds wrote:
> On Fri, Mar 25, 2022 at 6:13 AM Andy Shevchenko
> <andriy.shevchenko@xxxxxxxxxxxxxxx> wrote:
> >
> > +#
> > +# Turn off -Werror when extra warnings are enabled
> > +#
> > +ifneq ($(KBUILD_EXTRA_WARN),)
> > + KBUILD_CFLAGS += -Wno-error
> > +endif
>
> NAK.
>
> If you enabled CONFIG_WERROR, then you get CONFIG_WERROR.
>
> If you enabled W=1, then you get extra warnings.
>
> If you enabled both, then you get extra warnings and they are errors.
>
> This patch is just stupid.

Fair enough.

--
With Best Regards,
Andy Shevchenko