[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL 2/2] docker: Add "--enable-werror" to configure c
From: |
Fam Zheng |
Subject: |
Re: [Qemu-devel] [PULL 2/2] docker: Add "--enable-werror" to configure command line |
Date: |
Wed, 10 Aug 2016 10:15:44 +0800 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Tue, 08/09 17:21, Paolo Bonzini wrote:
>
>
> 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 <address@hidden>
> > Message-id: address@hidden
> > ---
> > 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?
That way I think build_qemu can be made sensible to the env. For now I'm not
seeing any compiling failure with centos6, even though this patch is already
merged. Do you mean this is a potential false alarm?
Fam
Re: [Qemu-devel] [PULL 0/2] Docker patches, Peter Maydell, 2016/08/05