Re: [Qemu-devel] [PATCH v4 00/22] Clean up around qmp() and hmp()
On Mon, Aug 7, 2017 at 4:33 AM, Fam Zheng wrote: > On Mon, 08/07 14:43, Fam Zheng wrote: >> On Fri, 08/04 20:10, Fam Zheng wrote: >> > On Fri, 08/04 06:50, Eric Blake wrote: >> > > On 08/03/2017 08:54 PM, no-re...@patchew.org wrote: >> > > > Hi, >> > > > >> > > > This series failed automatic build test. Please find the testing >> > > > commands and >> > > > their output below. If you have docker installed, you can probably >> > > > reproduce it >> > > > locally. >> > > >> > > When will patchew have a syntax for stating dependencies? (Of course, I >> > > should actually mention those dependencies in my cover letter, not after >> > > the fact). >> > >> > I can give it a try this weekend, if it works, it will be announced on this >> > list. >> >> Hijack! As an experiment of the newly pushed "base" tag handling on the >> server: >> >> Based-on: 20170802201900.11890-1-ebl...@redhat.com :) > > It worked. Now, restarted the tests with the "rebased" branch: > > https://github.com/patchew-project/qemu/commits/patchew/20170804012551.2714-1-eblake%40redhat.com > > Fam >
Re: [Qemu-devel] [PATCH v4 00/22] Clean up around qmp() and hmp()
On Thu, 08/03 18:54, no-re...@patchew.org wrote: > Hi, > > This series failed automatic build test. Please find the testing commands and > their output below. If you have docker installed, you can probably reproduce > it > locally. As said in the sub-thread of this message, this is a false positive because patchew didn't know the dependency. With that fixed, this test and the other falsely failed one both pass. Thanks. Fam
Re: [Qemu-devel] [PATCH v4 00/22] Clean up around qmp() and hmp()
On Mon, 08/07 14:43, Fam Zheng wrote: > On Fri, 08/04 20:10, Fam Zheng wrote: > > On Fri, 08/04 06:50, Eric Blake wrote: > > > On 08/03/2017 08:54 PM, no-re...@patchew.org wrote: > > > > Hi, > > > > > > > > This series failed automatic build test. Please find the testing > > > > commands and > > > > their output below. If you have docker installed, you can probably > > > > reproduce it > > > > locally. > > > > > > When will patchew have a syntax for stating dependencies? (Of course, I > > > should actually mention those dependencies in my cover letter, not after > > > the fact). > > > > I can give it a try this weekend, if it works, it will be announced on this > > list. > > Hijack! As an experiment of the newly pushed "base" tag handling on the > server: > > Based-on: 20170802201900.11890-1-ebl...@redhat.com It worked. Now, restarted the tests with the "rebased" branch: https://github.com/patchew-project/qemu/commits/patchew/20170804012551.2714-1-eblake%40redhat.com Fam
Re: [Qemu-devel] [PATCH v4 00/22] Clean up around qmp() and hmp()
On Fri, 08/04 20:10, Fam Zheng wrote: > On Fri, 08/04 06:50, Eric Blake wrote: > > On 08/03/2017 08:54 PM, no-re...@patchew.org wrote: > > > Hi, > > > > > > This series failed automatic build test. Please find the testing commands > > > and > > > their output below. If you have docker installed, you can probably > > > reproduce it > > > locally. > > > > When will patchew have a syntax for stating dependencies? (Of course, I > > should actually mention those dependencies in my cover letter, not after > > the fact). > > I can give it a try this weekend, if it works, it will be announced on this > list. Hijack! As an experiment of the newly pushed "base" tag handling on the server: Based-on: 20170802201900.11890-1-ebl...@redhat.com (Eric, the syntax is simple and strict for now: "Based-on: $MESSAGE_ID", if you find something serious lacking (e.g. human readability), please let me know). Fam
Re: [Qemu-devel] [PATCH v4 00/22] Clean up around qmp() and hmp()
On Fri, 08/04 06:50, Eric Blake wrote: > On 08/03/2017 08:54 PM, no-re...@patchew.org wrote: > > Hi, > > > > This series failed automatic build test. Please find the testing commands > > and > > their output below. If you have docker installed, you can probably > > reproduce it > > locally. > > When will patchew have a syntax for stating dependencies? (Of course, I > should actually mention those dependencies in my cover letter, not after > the fact). I can give it a try this weekend, if it works, it will be announced on this list. Fam
Re: [Qemu-devel] [PATCH v4 00/22] Clean up around qmp() and hmp()
On 08/03/2017 08:54 PM, no-re...@patchew.org wrote: > Hi, > > This series failed automatic build test. Please find the testing commands and > their output below. If you have docker installed, you can probably reproduce > it > locally. When will patchew have a syntax for stating dependencies? (Of course, I should actually mention those dependencies in my cover letter, not after the fact). > /tmp/qemu-test/src/tests/test-qga.c: In function ‘test_qga_set_time’: > /tmp/qemu-test/src/tests/test-qga.c:652: warning: implicit declaration of > function ‘qmp_fd’ This patch series depends on my earlier cleanup: https://lists.gnu.org/archive/html/qemu-devel/2017-08/msg00385.html [PATCH] test-qga: Kill broken and dead QGA_TEST_SIDE_EFFECT -- Eric Blake, Principal Software Engineer Red Hat, Inc. +1-919-301-3266 Virtualization: qemu.org | libvirt.org signature.asc Description: OpenPGP digital signature
Re: [Qemu-devel] [PATCH v4 00/22] Clean up around qmp() and hmp()
Hi, This series failed automatic build test. Please find the testing commands and their output below. If you have docker installed, you can probably reproduce it locally. Subject: [Qemu-devel] [PATCH v4 00/22] Clean up around qmp() and hmp() Message-id: 20170804012551.2714-1-ebl...@redhat.com Type: series === TEST SCRIPT BEGIN === #!/bin/bash set -e git submodule update --init dtc # Let docker tests dump environment info export SHOW_ENV=1 export J=8 time make docker-test-quick@centos6 time make docker-test-build@min-glib time make docker-test-mingw@fedora === TEST SCRIPT END === Updating 3c8cf5a9c21ff8782164d1def7f44bd888713384 From https://github.com/patchew-project/qemu t [tag update] patchew/1501692241-23310-1-git-send-email-peter.mayd...@linaro.org -> patchew/1501692241-23310-1-git-send-email-peter.mayd...@linaro.org * [new tag] patchew/20170804012551.2714-1-ebl...@redhat.com -> patchew/20170804012551.2714-1-ebl...@redhat.com Switched to a new branch 'test' 25d1d3e326 libqtest: Rename qmp_cmd() to qmp() 09d1c5f657 libqtest: Drop now-unused qmp() 0524247e82 tests/libqos/pci: Clean up string interpolation into QMP input 04c8567b62 libqtest: Add qmp_args_dict() helper a919f621a5 tests/libqos/usb: Clean up string interpolation into QMP input ef30c8a4db libqtest: Add qmp_args() helper f3ab2876ae libqtest: Add qmp_cmd() helper 8de753188c libqtest: Delete qtest_qmp() wrappers 70107888b6 libqtest: Separate qmp_discard_response() from command f1b80c24d0 libqtest: Add qmp_raw() 5b2e91aad2 libqtest: Change qmp_fd_send() to drop varargs d2734fc6f7 test-qga: Simplify command construction 9afca5aa84 libqtest: Skip round-trip through QObject df5c79a57b qtest: Document calling conventions a4f21de576 qtest: Avoid passing raw strings through hmp() 92906b7187 numa-test: Use hmp() 19b14cb945 qobject: Perform %% interpolation in qobject_from_jsonf() 5461f67f74 qobject: Simplify qobject_from_jsonv() f01023f518 tests: Add assertion for no qmp("") c21259c314 tests/libqtest: Clean up how we read the QMP greeting ecc0b4c2ee tests: Clean up wait for event 817e62311e qobject: Accept "%"PRId64 in qobject_from_jsonf() === OUTPUT BEGIN === Submodule 'dtc' (git://git.qemu-project.org/dtc.git) registered for path 'dtc' Cloning into '/var/tmp/patchew-tester-tmp-he2066gt/src/dtc'... Submodule path 'dtc': checked out '558cd81bdd432769b59bff01240c44f82cfb1a9d' BUILD centos6 make[1]: Entering directory '/var/tmp/patchew-tester-tmp-he2066gt/src' ARCHIVE qemu.tgz ARCHIVE dtc.tgz COPYRUNNER RUN test-quick in qemu:centos6 Packages installed: SDL-devel-1.2.14-7.el6_7.1.x86_64 bison-2.4.1-5.el6.x86_64 ccache-3.1.6-2.el6.x86_64 epel-release-6-8.noarch flex-2.5.35-9.el6.x86_64 gcc-4.4.7-18.el6.x86_64 git-1.7.1-8.el6.x86_64 glib2-devel-2.28.8-9.el6.x86_64 libfdt-devel-1.4.0-1.el6.x86_64 make-3.81-23.el6.x86_64 package g++ is not installed pixman-devel-0.32.8-1.el6.x86_64 tar-1.23-15.el6_8.x86_64 zlib-devel-1.2.3-29.el6.x86_64 Environment variables: PACKAGES=libfdt-devel ccache tar git make gcc g++ flex bison zlib-devel glib2-devel SDL-devel pixman-devel epel-release HOSTNAME=62a8e8853e87 TERM=xterm MAKEFLAGS= -j8 HISTSIZE=1000 J=8 USER=root LS_COLORS=rs=0:di=01;34:ln=01;36:mh=00:pi=40;33:so=01;35:do=01;35:bd=40;33;01:cd=40;33;01:or=40;31;01:mi=01;05;37;41:su=37;41:sg=30;43:ca=30;41:tw=30;42:ow=34;42:st=37;44:ex=01;32:*.tar=01;31:*.tgz=01;31:*.arj=01;31:*.taz=01;31:*.lzh=01;31:*.lzma=01;31:*.tlz=01;31:*.txz=01;31:*.zip=01;31:*.z=01;31:*.Z=01;31:*.dz=01;31:*.gz=01;31:*.lz=01;31:*.xz=01;31:*.bz2=01;31:*.tbz=01;31:*.tbz2=01;31:*.bz=01;31:*.tz=01;31:*.deb=01;31:*.rpm=01;31:*.jar=01;31:*.rar=01;31:*.ace=01;31:*.zoo=01;31:*.cpio=01;31:*.7z=01;31:*.rz=01;31:*.jpg=01;35:*.jpeg=01;35:*.gif=01;35:*.bmp=01;35:*.pbm=01;35:*.pgm=01;35:*.ppm=01;35:*.tga=01;35:*.xbm=01;35:*.xpm=01;35:*.tif=01;35:*.tiff=01;35:*.png=01;35:*.svg=01;35:*.svgz=01;35:*.mng=01;35:*.pcx=01;35:*.mov=01;35:*.mpg=01;35:*.mpeg=01;35:*.m2v=01;35:*.mkv=01;35:*.ogm=01;35:*.mp4=01;35:*.m4v=01;35:*.mp4v=01;35:*.vob=01;35:*.qt=01;35:*.nuv=01;35:*.wmv=01;35:*.asf=01;35:*.rm=01;35:*.rmvb=01;35:*.flc=01;35:*.avi=01;35:*.fli=01;35:*.flv=01;35:*.gl=01;35:*.dl=01;35:*.xcf=01;35:*.xwd=01;35:*.yuv=01;35:*.cgm=01;35:*.emf=01;35:*.axv=01;35:*.anx=01;35:*.ogv=01;35:*.ogx=01;35:*.aac=01;36:*.au=01;36:*.flac=01;36:*.mid=01;36:*.midi=01;36:*.mka=01;36:*.mp3=01;36:*.mpc=01;36:*.ogg=01;36:*.ra=01;36:*.wav=01;36:*.axa=01;36:*.oga=01;36:*.spx=01;36:*.xspf=01;36: CCACHE_DIR=/var/tmp/ccache EXTRA_CONFIGURE_OPTS= V= SHOW_ENV=1 MAIL=/var/spool/mail/root PATH=/usr/lib/ccache:/usr/lib64/ccache:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin PWD=/ LANG=en_US.UTF-8 TARGET_LIST= HISTCONTROL=ignoredups SHLVL=1 HOME=/root TEST_DIR=/tmp/qemu-test LOGNAME=root LESSOPEN=||/usr/bin/lesspipe.sh %s FEATURES= dtc DEBUG= G_BROKEN_FILENAMES=1 CCACHE_HASHDIR= _=/usr/bin/env Configure options: --enable-werror --target-list=x86_64-softmmu,aarch64-softmmu --