[rfc] removing/conditionalising WERROR= in Makefiles

Philip Paeps philip at freebsd.org
Tue Dec 27 10:42:03 UTC 2011


On 2011-12-26 10:10:40 (+0000), Alexander Best <arundel at 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".

Possibly wrapped in an architecture-test where appropriate.

 - Philip

-- 
Philip Paeps
Senior Reality Engineer
Ministry of Information


More information about the freebsd-current mailing list