On Tue, Jul 17, 2012 at 7:28 PM, Peter Maydell <peter.mayd...@linaro.org> wrote: > On 17 July 2012 20:24, Stefan Weil <s...@weilnetz.de> wrote: >> The arguments why -Werror is a bad idea for some configure tests >> are reasonable. >> >> Nevertheless the QEMU community was able to produce thousands of >> lines of code which compile without a warning, so we should be able >> to create warning and error free code for a handful of configure >> tests. > > The trouble is that the warnings and errors here don't cause the > build to fail noisily; that's a big distinction IMHO. > I suppose we could make compile_prog do something like: > * run the compile test
Unfortunately that would break cross compiling. > * if it fails => test failure as now > * if it succeeds (and we're doing a Werror build at all), > rerun the same test with -Werror > * if that fails, abort configure with an error message > Then we would have the same "make the problem obvious" effect > that plain -Werror provides for our main compilation. > >> The 4 patches above are valid and can be applied with or without >> -Werror, therefore qemu-trivial or whoever does not have to wait for >> Peter's patch. > > Yes, I agree we might as well fix these errors since we've now > noticed them, regardless of whether or not we apply my patch > (which I've just sent). > > -- PMM