From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1MEjV2-0007SR-Bt for qemu-devel@nongnu.org; Thu, 11 Jun 2009 08:35:20 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1MEjUx-0007QS-N1 for qemu-devel@nongnu.org; Thu, 11 Jun 2009 08:35:20 -0400 Received: from [199.232.76.173] (port=50453 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1MEjUx-0007QJ-H0 for qemu-devel@nongnu.org; Thu, 11 Jun 2009 08:35:15 -0400 Received: from mx20.gnu.org ([199.232.41.8]:37195) by monty-python.gnu.org with esmtps (TLS-1.0:RSA_AES_256_CBC_SHA1:32) (Exim 4.60) (envelope-from ) id 1MEjUx-0006lm-8a for qemu-devel@nongnu.org; Thu, 11 Jun 2009 08:35:15 -0400 Received: from mail.codesourcery.com ([65.74.133.4]) by mx20.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1MEjUw-0005rK-JB for qemu-devel@nongnu.org; Thu, 11 Jun 2009 08:35:14 -0400 From: Paul Brook Date: Thu, 11 Jun 2009 13:35:09 +0100 MIME-Version: 1.0 Content-Type: Multipart/Mixed; boundary="Boundary-00=_+nPMKJUoNSZx7CY" Message-Id: <200906111335.10116.paul@codesourcery.com> Subject: [Qemu-devel] [RFC] Enable Werrror by default List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org --Boundary-00=_+nPMKJUoNSZx7CY Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Content-Disposition: inline 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 --Boundary-00=_+nPMKJUoNSZx7CY Content-Type: text/x-patch; charset="us-ascii"; name="0001-Enable-Werror-by-default.patch" Content-Transfer-Encoding: quoted-printable Content-Disposition: inline; filename="0001-Enable-Werror-by-default.patch" =46rom 5ccaebfce38fc161a776725c6c945b2173459397 Mon Sep 17 00:00:00 2001 =46rom: Paul Brook Date: Thu, 11 Jun 2009 13:31:46 +0100 Subject: [PATCH] Enable Werror by default Signed-off-by: Paul Brook =2D-- configure | 6 +----- 1 files changed, 1 insertions(+), 5 deletions(-) diff --git a/configure b/configure index 89e7f53..9e03ee3 100755 =2D-- a/configure +++ b/configure @@ -356,11 +356,7 @@ else fi [ -f "$workdir/vl.c" ] || source_path_used=3D"yes" =20 =2Dwerror=3D"no" =2D# generate compile errors on warnings for development builds =2D#if grep cvs $source_path/VERSION > /dev/null 2>&1 ; then =2D#werror=3D"yes"; =2D#fi +werror=3D"yes" =20 for opt do optarg=3D`expr "x$opt" : 'x[^=3D]*=3D\(.*\)'` =2D-=20 1.6.3.1 --Boundary-00=_+nPMKJUoNSZx7CY--