On 05/03/2013 03:46 PM, Jeroen Roovers wrote: > On Fri, 3 May 2013 16:06:01 +0800 > Ben de Groot <yng...@gentoo.org> wrote: > >> Personally I've always thought -Werror is a mistake in release code, >> but was accepted practice. I've almost never actively removed it from >> packages I maintain. That will change now, upon learning of this >> policy. > > Debian here and there actively fixes compiler warnings by enabling > -Werror and then checking what fails to build. I even see revision > bumps because of some of the milder compiler warnings. Maybe other > distros do this as well. It's kind of a blunt instrument to perform QA > with, but I guess it does get code cleaned up slightly better than with > no checks at all or by merely depending on runtime failures to be > reported. > > > jer >
We don't need that. We already get QA warnings for severe compiler warnings with a note that it should be reported upstream. Turning them into errors does not improve anything.