From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail.lixom.net (lixom.net [66.141.50.11]) by ozlabs.org (Postfix) with ESMTP id 70AB6DDEEF for ; Tue, 7 Apr 2009 12:36:08 +1000 (EST) Date: Mon, 6 Apr 2009 21:37:52 -0500 From: Olof Johansson To: David Miller Subject: Re: [PATCH] powerpc: Add configurable -Werror for arch/powerpc Message-ID: <20090407023752.GB12174@lixom.net> References: <20090407022743.GA12174@lixom.net> <20090406.192830.193803179.davem@davemloft.net> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <20090406.192830.193803179.davem@davemloft.net> Cc: linuxppc-dev@ozlabs.org, kumar.gala@freescale.com List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Mon, Apr 06, 2009 at 07:28:30PM -0700, David Miller wrote: > From: Olof Johansson > Date: Mon, 6 Apr 2009 21:27:43 -0500 > > > On Tue, Apr 07, 2009 at 12:00:41PM +1000, Michael Ellerman wrote: > >> Add an option, on by default, to build all code under arch/powerpc with > >> -Werror, which causes gcc to treat warnings as errors. > >> > >> The intention is to make it harder for people to inadvertantly introduce > >> errors in the arch/powerpc code. It needs to be configurable so that > >> if a warning is introduced, people can easily work around it while it's > >> being fixed. > > > > This looks useful at least for the automated builds to catch new warnings, > > but do similar options exist on other architectures, x86 in particular? I > > think a Cc to LKML of this could be useful. > > On sparc64 we do this unconditionally. Great! I'm all for merging it in, having recently found new warnings on RFC patches that went uncaught by the poster. :) -Olof