Re: [Qemu-devel] [PATCH 00/11] configure: Fix -Werror issues

[Blue Swirl]

Thanks, applied patches 1 to 10.

On Wed, Jul 18, 2012 at 2:10 PM, Peter Maydell <address@hidden> wrote:
> This patch series:
>  1. turns off -Werror for configure tests
>  2. fixes a large pile of warnings in various configure tests
>  3. turns on -Werror for configure tests again, but in a way that means
>     that errors mean configure stops so the warnings are as obvious
>     to developers as they would be for a -Werror compile of the main code
>
> The "turn off -Werror" patch has already been sent to the list;
> I've included it here as it is a dependency, but it should probably
> be committed ASAP even if the rest of this series needs review and
> updating. I've also put all of Stefan's recent patches in since without
> them configure won't run once -Werror is reenabled.
>
> NB: this works for me, but it's possible there are some still lurking
> warnings in a few compilation tests that are themselves protected by
> tests for some library/host I don't have. They should be easy to fix
> as they turn up, though, and the configure failure message is clear
> about how to work around in the meantime.
>
> Peter Maydell (7):
>   configure: Don't run configure tests with -Werror enabled
>   configure: -march=i486 belongs in QEMU_CFLAGS, not CFLAGS
>   configure: Fix compile warning in PNG test
>   configure: Fix warnings in VDE library probe
>   configure: Fix compile warning in utimensat/futimens test
>   configure: -I\$(SRC_PATH) goes in QEMU_INCLUDES not QEMU_CFLAGS
>   configure: Check for -Werror causing failures when compiling tests
>
> Stefan Weil (4):
>   configure: Fix build with ALSA audio driver
>   configure: Fix build with capabilities
>   configure: Replace bash code by standard shell code
>   configure: Fix errors in test for__sync_fetch_and_and
>
>  configure |   73 +++++++++++++++++++++++++++++++++++++++++++++++-------------
>  1 files changed, 57 insertions(+), 16 deletions(-)
>
> --
> 1.7.5.4
>
>