Re: [rfc] removing/conditionalising WERROR= in Makefiles

On 2011-12-27 02:04, Philip Paeps wrote:
> On 2011-12-26 10:10:40 (+0000), Alexander Best<arundel@freebsd.org>  wrote:
>> i grep'ed through src/sys and found several places where WERROR= was set in
>> order to get rid of the default -Werror setting. i tried to remove those
>> WERROR= overrides from any Makefile, where doing so did not break tinderbox.
>>
>> in those cases, where it couldn't be completely removed, i added conditions to
>> only set WERROR= for the particular achitecture or compiler, where tinderbox
>> did not suceed without the WERROR=.
>
> Wouldn't it be better to set WARNS=x rather than WERROR=?  WERROR= says "this
> code has bugs, it breaks tinderbox" whereas WARNS=x says "this code has the
> following kind of bugs which break tinderbox".

In my opinion, WERROR= says: there are warnings in this code which
cannot be fixed right now, due to varying reasons, but we don't want to
muffle them entirely, so somebody will eventually fix them in the future
(or just delete the code, if it is unmaintained, or unmaintainable, like
nve).

If you set WARNS to a low level, you can be sure nobody ever sees the
warnings, and they will never be fixed.  That may be appropriate in some
cases, but not the ones I just added WERROR= to.  Those are just crufty
drivers, that nobody wants to burn their fingers on. :)
_______________________________________________
freebsd-current@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-current
To unsubscribe, send any mail to "freebsd-current-unsubscribe@freebsd.org"