Message ID | 20180619154435.18898-1-alex.bennee@linaro.org |
---|---|
Headers | show |
Series | add check-tcg and associated machinery | expand |
On 19 June 2018 at 16:43, Alex Bennée <alex.bennee@linaro.org> 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: Doing build and test cycle... make: Entering directory '/home/petmay01/linaro/qemu-for-merges/build/alldbg' Traceback (most recent call last): File "/home/petmay01/linaro/qemu-for-merges/tests/docker/docker.py", line 523, in <module> sys.exit(main()) File "/home/petmay01/linaro/qemu-for-merges/tests/docker/docker.py", line 520, in main return args.cmdobj.run(args, argv) File "/home/petmay01/linaro/qemu-for-merges/tests/docker/docker.py", line 475, in run dkr = Docker() File "/home/petmay01/linaro/qemu-for-merges/tests/docker/docker.py", line 137, in __init__ self._command = _guess_docker_command() File "/home/petmay01/linaro/qemu-for-merges/tests/docker/docker.py", line 62, in _guess_docker_command commands_txt) Exception: Cannot find working docker command. Tried: docker sudo -n docker make: Leaving directory '/home/petmay01/linaro/qemu-for-merges/build/alldbg' make: *** Deleting file 'config-host.mak' make: Entering directory '/home/petmay01/linaro/qemu-for-merges/build/alldbg' config-host.mak is out-of-date, running configure Traceback (most recent call last): File "/home/petmay01/linaro/qemu-for-merges/tests/docker/docker.py", line 29, in <module> from StringIO import StringIO ImportError: No module named 'StringIO' Install prefix /usr/local BIOS directory /usr/local/share/qemu firmware path /usr/local/share/qemu-firmware [other configure output trimmed] VxHS block device no capstone git docker 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. Makefile:73: recipe for target 'config-host.mak' failed make: *** [config-host.mak] Error 1 and the last part of config.log is ====== ccache gcc -Werror -pthread -I/usr/include/glib-2.0 -I/usr/lib/x86_64-linux-gnu/glib-2.0/include -fPIE -DPIE -m64 -mcx16 -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -Wstrict-prototypes -Wredundant-decls -Wall -Wundef -Wwrite-strings -Wmissing-prototypes -fno-strict-aliasing -fno-common -fwrapv -Wendif-labels -Wno-missing-include-dirs -Wempty-body -Wnested-externs -Wformat-security -Wformat-y2k -Winit-self -Wignored-qualifiers -Wold-style-declaration -Wold-style-definition -Wtype-limits -fstack-protector-strong -I/usr/include/p11-kit-1 -I/usr/include/libpng12 -I/usr/include/spice-server -I/usr/include/spice-1 -I$(SRC_PATH)/capstone/include -c -o config-temp/qemu-conf.o config-temp/qemu-conf.c c++ -D__STDC_LIMIT_MACROS -Werror -pthread -I/usr/include/glib-2.0 -I/usr/lib/x86_64-linux-gnu/glib-2.0/include -fPIE -DPIE -m64 -mcx16 -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -Wall -Wundef -Wwrite-strings -fno-strict-aliasing -fno-common -fwrapv -Wendif-labels -Wno-missing-include-dirs -Wempty-body -Wformat-security -Wformat-y2k -Winit-self -Wignored-qualifiers -Wtype-limits -fstack-protector-strong -I/usr/include/p11-kit-1 -I/usr/include/libpng12 -I/usr/include/spice-server -I/usr/include/spice-1 -I$(SRC_PATH)/capstone/include -o config-temp/qemu-conf.exe config-temp/qemu-conf.cxx config-temp/qemu-conf.o -Wl,--warn-common -Wl,-z,relro -Wl,-z,now -pie -m64 -g aarch64-linux-gnu-gcc -o config-temp/qemu-conf.exe config-temp/qemu-conf.c -static aarch64-linux-gnu-gcc -Werror -o config-temp/qemu-conf.exe config-temp/qemu-conf.c -static arm-linux-gnueabihf-gcc -o config-temp/qemu-conf.exe config-temp/qemu-conf.c -static arm-linux-gnueabihf-gcc -Werror -o config-temp/qemu-conf.exe config-temp/qemu-conf.c -static powerpc-linux-gnu-gcc -o config-temp/qemu-conf.exe config-temp/qemu-conf.c -static powerpc-linux-gnu-gcc -Werror -o config-temp/qemu-conf.exe config-temp/qemu-conf.c -static 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 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>. ====== The configure options for this build are '--cc=ccache gcc' '--enable-debug' '--python=python3' Looks like something is mishandling CC. On FreeBSD I get an odd linker error: LINK qemu-ga /usr/bin/ld: cannot open linker script file ../config-host.ld: No such file or directory c++: error: linker command failed with exit code 1 (use -v to see invocation) gmake: *** [Makefile:688: qemu-ga] Error 1 thanks -- PMM
Peter Maydell <peter.maydell@linaro.org> writes: > On 19 June 2018 at 16:43, Alex Bennée <alex.bennee@linaro.org> 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: > > Doing build and test cycle... > make: Entering directory '/home/petmay01/linaro/qemu-for-merges/build/alldbg' > Traceback (most recent call last): > File "/home/petmay01/linaro/qemu-for-merges/tests/docker/docker.py", > line 523, in <module> > sys.exit(main()) > File "/home/petmay01/linaro/qemu-for-merges/tests/docker/docker.py", > line 520, in main > return args.cmdobj.run(args, argv) > File "/home/petmay01/linaro/qemu-for-merges/tests/docker/docker.py", > line 475, in run > dkr = Docker() > File "/home/petmay01/linaro/qemu-for-merges/tests/docker/docker.py", > line 137, in __init__ > self._command = _guess_docker_command() > File "/home/petmay01/linaro/qemu-for-merges/tests/docker/docker.py", > line 62, in _guess_docker_command > commands_txt) > Exception: Cannot find working docker command. Tried: > docker > sudo -n docker > make: Leaving directory '/home/petmay01/linaro/qemu-for-merges/build/alldbg' > make: *** Deleting file 'config-host.mak' > make: Entering directory '/home/petmay01/linaro/qemu-for-merges/build/alldbg' > config-host.mak is out-of-date, running configure > Traceback (most recent call last): > File "/home/petmay01/linaro/qemu-for-merges/tests/docker/docker.py", > line 29, in <module> > from StringIO import StringIO > ImportError: No module named 'StringIO' > Install prefix /usr/local > BIOS directory /usr/local/share/qemu > firmware path /usr/local/share/qemu-firmware > [other configure output trimmed] > VxHS block device no > capstone git > docker Hmm I'm sure I tested this. I'll go and find a box without docker to double check. > > 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. > > Makefile:73: recipe for target 'config-host.mak' failed > make: *** [config-host.mak] Error 1 > > and the last part of config.log is > > ====== > ccache gcc -Werror -pthread -I/usr/include/glib-2.0 > -I/usr/lib/x86_64-linux-gnu/glib-2.0/include -fPIE -DPIE -m64 -mcx16 > -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE > -Wstrict-prototypes -Wredundant-decls -Wall -Wundef -Wwrite-strings > -Wmissing-prototypes -fno-strict-aliasing -fno-common -fwrapv > -Wendif-labels -Wno-missing-include-dirs -Wempty-body -Wnested-externs > -Wformat-security -Wformat-y2k -Winit-self -Wignored-qualifiers > -Wold-style-declaration -Wold-style-definition -Wtype-limits > -fstack-protector-strong -I/usr/include/p11-kit-1 > -I/usr/include/libpng12 -I/usr/include/spice-server > -I/usr/include/spice-1 -I$(SRC_PATH)/capstone/include -c -o > config-temp/qemu-conf.o config-temp/qemu-conf.c > c++ -D__STDC_LIMIT_MACROS -Werror -pthread -I/usr/include/glib-2.0 > -I/usr/lib/x86_64-linux-gnu/glib-2.0/include -fPIE -DPIE -m64 -mcx16 > -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -Wall -Wundef > -Wwrite-strings -fno-strict-aliasing -fno-common -fwrapv > -Wendif-labels -Wno-missing-include-dirs -Wempty-body > -Wformat-security -Wformat-y2k -Winit-self -Wignored-qualifiers > -Wtype-limits -fstack-protector-strong -I/usr/include/p11-kit-1 > -I/usr/include/libpng12 -I/usr/include/spice-server > -I/usr/include/spice-1 -I$(SRC_PATH)/capstone/include -o > config-temp/qemu-conf.exe config-temp/qemu-conf.cxx > config-temp/qemu-conf.o -Wl,--warn-common -Wl,-z,relro -Wl,-z,now -pie > -m64 -g > aarch64-linux-gnu-gcc -o config-temp/qemu-conf.exe > config-temp/qemu-conf.c -static > aarch64-linux-gnu-gcc -Werror -o config-temp/qemu-conf.exe > config-temp/qemu-conf.c -static > arm-linux-gnueabihf-gcc -o config-temp/qemu-conf.exe > config-temp/qemu-conf.c -static > arm-linux-gnueabihf-gcc -Werror -o config-temp/qemu-conf.exe > config-temp/qemu-conf.c -static > powerpc-linux-gnu-gcc -o config-temp/qemu-conf.exe > config-temp/qemu-conf.c -static > powerpc-linux-gnu-gcc -Werror -o config-temp/qemu-conf.exe > config-temp/qemu-conf.c -static > 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 > 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>. > ====== > > The configure options for this build are > '--cc=ccache gcc' '--enable-debug' '--python=python3' > > Looks like something is mishandling CC. > > On FreeBSD I get an odd linker error: > > LINK qemu-ga > /usr/bin/ld: cannot open linker script file ../config-host.ld: No such > file or directory > c++: error: linker command failed with exit code 1 (use -v to see invocation) > gmake: *** [Makefile:688: qemu-ga] Error 1 > > thanks > -- PMM -- Alex Bennée
On Wed, Jun 20, 2018 at 11:43:57AM +0100, Peter Maydell wrote: > On 19 June 2018 at 16:43, Alex Bennée <alex.bennee@linaro.org> 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 :|
On 19 June 2018 at 16:43, Alex Bennée <alex.bennee@linaro.org> 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 ;-) > Also, I've just noticed that your commit messages are full of the 'below-the-fold' info about changes since v3, v4, etc. That shouldn't be in commits in pull requests, please. thanks -- PMM
Peter Maydell <peter.maydell@linaro.org> writes: > On 19 June 2018 at 16:43, Alex Bennée <alex.bennee@linaro.org> 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 ;-) >> > > Also, I've just noticed that your commit messages are > full of the 'below-the-fold' info about changes since > v3, v4, etc. That shouldn't be in commits in pull requests, > please. Doh... let me add that to my pullreq script... > > thanks > -- PMM -- Alex Bennée