[Qemu-devel] [RFC] Enable Werrror by default

[Paul Brook <paul@codesourcery.com>]

[-- Attachment #1: Type: text/plain, Size: 320 bytes --]

I'd like to enable Werror by default. I've been using --enable-werror locally 
for a while now, and it's been extremely useful in picking up dumb errors 
(like the recent stellaris_enet.c breakage).

Any objections?

You can of course configure with --disable-werror if you really want the force 
things to build.

Paul

[-- Attachment #2: 0001-Enable-Werror-by-default.patch --]
[-- Type: text/x-patch, Size: 771 bytes --]

From 5ccaebfce38fc161a776725c6c945b2173459397 Mon Sep 17 00:00:00 2001
From: Paul Brook <paul@codesourcery.com>
Date: Thu, 11 Jun 2009 13:31:46 +0100
Subject: [PATCH] Enable Werror by default

Signed-off-by: Paul Brook <paul@codesourcery.com>
---
 configure |    6 +-----
 1 files changed, 1 insertions(+), 5 deletions(-)

diff --git a/configure b/configure
index 89e7f53..9e03ee3 100755
--- a/configure
+++ b/configure
@@ -356,11 +356,7 @@ else
 fi
 [ -f "$workdir/vl.c" ] || source_path_used="yes"
 
-werror="no"
-# generate compile errors on warnings for development builds
-#if grep cvs $source_path/VERSION > /dev/null 2>&1 ; then
-#werror="yes";
-#fi
+werror="yes"
 
 for opt do
   optarg=`expr "x$opt" : 'x[^=]*=\(.*\)'`
-- 
1.6.3.1