On 5/13/20 5:13 PM, no-re...@patchew.org wrote:
Patchew URL: https://patchew.org/QEMU/20200513133629.18508-1-eyal.moscov...@oracle.com/Hi, This series failed the docker-mingw@fedora build test. Please find the testing commands and their output below. If you have Docker installed, you can probably reproduce it locally. === TEST SCRIPT BEGIN === #! /bin/bash export ARCH=x86_64 make docker-image-fedora V=1 NETWORK=1 time make docker-test-mingw@fedora J=14 NETWORK=1 === TEST SCRIPT END === CC x86_64-softmmu/ioport.o CC x86_64-softmmu/qtest.o /tmp/qemu-test/src/qemu-img.c: In function 'cvtnum_full': /tmp/qemu-test/src/qemu-img.c:488:63: error: format '%ld' expects argument of type 'long int', but argument 3 has type 'int64_t' {aka 'long long int'} [-Werror=format=] error_report("Invalid %s specified. Must be between %ld bytes " ~~^ %lld "to %ld bytes.", name, min, max);
patchew is correct; printing int64_t values requires "%"PRId64 rather than "%ld". I'm fine with touching that up in my queue, unless you'd like to submit v4.
-- Eric Blake, Principal Software Engineer Red Hat, Inc. +1-919-301-3226 Virtualization: qemu.org | libvirt.org
