From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:58903) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1SwIxO-0000xJ-1C for qemu-devel@nongnu.org; Tue, 31 Jul 2012 16:22:19 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1SwIxM-0004OP-VH for qemu-devel@nongnu.org; Tue, 31 Jul 2012 16:22:17 -0400 Received: from mail-yx0-f173.google.com ([209.85.213.173]:59805) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1SwIxM-0004OK-Ra for qemu-devel@nongnu.org; Tue, 31 Jul 2012 16:22:16 -0400 Received: by yenl1 with SMTP id l1so6353366yen.4 for ; Tue, 31 Jul 2012 13:22:16 -0700 (PDT) MIME-Version: 1.0 In-Reply-To: <1342620628-12751-1-git-send-email-peter.maydell@linaro.org> References: <1342620628-12751-1-git-send-email-peter.maydell@linaro.org> From: Blue Swirl Date: Tue, 31 Jul 2012 20:21:56 +0000 Message-ID: Content-Type: text/plain; charset=UTF-8 Subject: Re: [Qemu-devel] [PATCH 00/11] configure: Fix -Werror issues List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Peter Maydell Cc: Stefan Weil , qemu-devel@nongnu.org, patches@linaro.org Thanks, applied patches 1 to 10. On Wed, Jul 18, 2012 at 2:10 PM, Peter Maydell 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 > >