On Tue, Apr 07, 2009 at 03:01:33AM +0000, Michael Ellerman wrote: > On Mon, 2009-04-06 at 21:27 -0500, Olof Johansson wrote: > > 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. > > > > This is really only beneficial if various people build for powerpc often > > enough. If major subsystem maintainers aren't going to hit the errors > > it's more of a hinderance for PPC than a global benefit, right? > > I don't think so. We're only enabling it for code under arch/powerpc - > and most modifications to that code should come through linuxppc.
Good point. > It's still possible that random stuff will get merged, or that someone > will change code in a header that only causes warnings on powerpc, but > it's less likely. > > And that's why it's an option, if someone breaks the build we can work > around it until they're appropriately LARTed. Yeah, sounds good to me. -Olof _______________________________________________ Linuxppc-dev mailing list [email protected] https://ozlabs.org/mailman/listinfo/linuxppc-dev
