Re: [Qemu-devel] [PATCH v4 00/22] Clean up around qmp() and hmp()

2017-08-07 Thread Philippe Mathieu-Daudé 
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()

2017-08-07 Thread Fam Zheng 
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()

2017-08-07 Thread Fam Zheng 
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()

2017-08-07 Thread Fam Zheng 
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()

2017-08-04 Thread Fam Zheng 
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()

2017-08-04 Thread Eric Blake 
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()

2017-08-03 Thread no-reply 
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