From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:38639) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bX8qo-0000Jg-8b for qemu-devel@nongnu.org; Tue, 09 Aug 2016 11:21:55 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bX8qk-0002uT-1V for qemu-devel@nongnu.org; Tue, 09 Aug 2016 11:21:53 -0400 Received: from mail-wm0-x242.google.com ([2a00:1450:400c:c09::242]:33111) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bX8qj-0002uG-QM for qemu-devel@nongnu.org; Tue, 09 Aug 2016 11:21:49 -0400 Received: by mail-wm0-x242.google.com with SMTP id o80so3836520wme.0 for ; Tue, 09 Aug 2016 08:21:49 -0700 (PDT) Sender: Paolo Bonzini References: <1470387870-30438-1-git-send-email-famz@redhat.com> <1470387870-30438-3-git-send-email-famz@redhat.com> From: Paolo Bonzini Message-ID: <66317136-7ead-e3ba-ad3b-e295ced14164@redhat.com> Date: Tue, 9 Aug 2016 17:21:46 +0200 MIME-Version: 1.0 In-Reply-To: <1470387870-30438-3-git-send-email-famz@redhat.com> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PULL 2/2] docker: Add "--enable-werror" to configure command line List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Fam Zheng , qemu-devel@nongnu.org Cc: Peter Maydell On 05/08/2016 11:04, Fam Zheng wrote: > We don't have .git in the docker checkout, add this to enable -Werror > explicitly. > > Signed-off-by: Fam Zheng > Message-id: 1469453510-658-1-git-send-email-famz@redhat.com > --- > tests/docker/common.rc | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/tests/docker/common.rc b/tests/docker/common.rc > index 77069e1..0c6d8d5 100755 > --- a/tests/docker/common.rc > +++ b/tests/docker/common.rc > @@ -24,6 +24,7 @@ requires() > build_qemu() > { > $QEMU_SRC/configure \ > + --enable-werror \ > ${TARGET_LIST:+"--target-list=${TARGET_LIST}"} \ > --prefix="$PWD/install" \ > $EXTRA_CONFIGURE_OPTS \ > centos6 has -Wuninitialized errors. Should this be applied only to Fedora and Ubuntu? Paolo