[Buildroot] svn commit: trunk/buildroot/package/libconfuse

Sven Neumann s.neumann at phase-zero.de
Thu Jan 22 08:47:01 UTC 2009


Hi,

On Thu, 2009-01-22 at 07:14 +0100, Hans-Christian Egtvedt wrote:
> On Wed, 21 Jan 2009 20:45:12 +0000 (UTC)
> jacmet at uclibc.org wrote:
> 
> > Author: jacmet
> > Date: 2009-01-21 20:45:10 +0000 (Wed, 21 Jan 2009)
> > New Revision: 24944
> > 
> > Log:
> > libconfuse: fix build by removing -Werror flag
> > 
> 
> You know that is not fixing it ;) Source code should not generate
> warnings, they are usually a sign of something bad is happening.

Source code that compiles without warnings on one compiler may generate
warnings on a different compiler (or just a different version or a
slightly differently configured instance of the very same compiler).
Having -Werror in public source code is always wrong as you can't
possibly know that it will compile cleanly for everyone.

And there are tons of warnings created by compilers that don't
necessarily indicate a problem in the source code. But of course you are
right that one should always check every warning carefully.


Sven




More information about the buildroot mailing list