[Sugar-devel] [PATCH] sugar-artwork: stricter compiler flags

Benjamin Berg benzea at sugarlabs.org
Mon May 17 09:53:48 EDT 2010


On Tue, 2010-05-11 at 18:56 -0400, Bernie Innocenti wrote:
> WARN_CFLAGS was already used in gtk/engine/Makefile.am, but not
> initialized by configure.
> 
> For backwards compatibility, GCC's by default is very permissive.
> In my case, it was silently miscompiling code which was calling
> a non-existent GTK function. For this particular case, the
> proposed WARN_CFLAGS should make the build fail early in this case.
> 
> I'm not proposing -Werror because it frequetly results in broken
> builds for trivial reasons when distros update the toolchain.
> 
> Signed-off-by: Bernie Innocenti <bernie at codewiz.org>

That looks like a very good change. Thanks for the patch.

Reviewed-by: Benjamin Berg <benjamin at sipsolutions.net>



More information about the Sugar-devel mailing list