Re: [Qemu-devel] [PULL 00/56] add check-tcg and associated machinery

[Daniel P . Berrangé]

On Wed, Jun 20, 2018 at 11:43:57AM +0100, Peter Maydell wrote:
> On 19 June 2018 at 16:43, Alex Bennée <address@hidden> wrote:
> > The following changes since commit 2ef2f16781af9dee6ba6517755e9073ba5799fa2:
> >
> >   Merge remote-tracking branch 
> > 'remotes/dgilbert/tags/pull-migration-20180615a' into staging (2018-06-15 
> > 18:13:35 +0100)
> >
> > are available in the Git repository at:
> >
> >   https://github.com/stsquad/qemu.git 
> > tags/pull-tcg-testing-revivial-190618-1
> >
> > for you to fetch changes up to 03c79cb5825f2e930a39839e8375dc0feef6d57d:
> >
> >   .travis.yml: add check-tcg test (2018-06-19 16:38:54 +0100)
> >
> > ----------------------------------------------------------------
> > Add check-tcg machinary
> >
> > This restores the ability to run TCG smoke tests by using our docker
> > infrastructure to support cross building simple tests. It represents
> > the first step to making better cross-architecture testing available
> > straight from the source tree ;-)
> >
> > ----------------------------------------------------------------
> 
> On x86-64/Linux I get:
> 

[snip]

> ERROR: configure test passed without -Werror but failed with -Werror.
>        This is probably a bug in the configure script. The failing command
>        will be at the bottom of config.log.
>        You can run configure with --disable-werror to bypass this check.

[snip]

> ccache gcc -o config-temp/qemu-conf.exe config-temp/qemu-conf.c -static
> ccache -Werror gcc -o config-temp/qemu-conf.exe config-temp/qemu-conf.c 
> -static

This looks dubious - the -Werror arg is in the wrong place on the
command line, hence ccache complaining:

> ccache: invalid option -- 'W'
> Usage:
>     ccache [options]
>     ccache compiler [compiler options]
>     compiler [compiler options]          (via symbolic link)
> 
> Options:
>     -c, --cleanup         delete old files and recalculate size counters
>                           (normally not needed as this is done automatically)
>     -C, --clear           clear the cache completely (except configuration)
>     -F, --max-files=N     set maximum number of files in cache to N (use 0 for
>                           no limit)
>     -M, --max-size=SIZE   set maximum size of cache to SIZE (use 0 for no
>                           limit); available suffixes: k, M, G, T (decimal) and
>                           Ki, Mi, Gi, Ti (binary); default suffix: G
>     -o, --set-config=K=V  set configuration key K to value V
>     -p, --print-config    print current configuration options
>     -s, --show-stats      show statistics summary
>     -z, --zero-stats      zero statistics counters
> 
>     -h, --help            print this help text
>     -V, --version         print version and copyright information
> 
> See also <http://ccache.samba.org>.

Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|