Re: [Qemu-devel] [PATCH 0/8] Small IPMI (and other) fixes

[Fam Zheng <famz@redhat.com>]

On Sat, 12/23 08:36, Corey Minyard wrote:
> On 12/21/2017 05:10 PM, no-reply@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.
> 
> The error here doesn't seem to have anything to do with these patches:
> 
> 
> >    CC      aarch64-softmmu/hw/virtio/virtio.o
> >    CC      aarch64-softmmu/hw/virtio/virtio-balloon.o
> > /tmp/qemu-test/src/hw/i386/vmport.c: In function 'vmport_ioport_read':
> > /tmp/qemu-test/src/hw/i386/vmport.c:81:9: error: implicit declaration of function 'qemu_log_mask' [-Werror=implicit-function-declaration]
> >           qemu_log_mask(LOG_UNIMP, "vmport: unknown command %x\n", command);
> >           ^~~~~~~~~~~~~
> > /tmp/qemu-test/src/hw/i386/vmport.c:81:9: error: nested extern declaration of 'qemu_log_mask' [-Werror=nested-externs]
> > /tmp/qemu-test/src/hw/i386/vmport.c:81:23: error: 'LOG_UNIMP' undeclared (first use in this function)
> >           qemu_log_mask(LOG_UNIMP, "vmport: unknown command %x\n", command);
> >                         ^~~~~~~~~
> > /tmp/qemu-test/src/hw/i386/vmport.c:81:23: note: each undeclared identifier is reported only once for each function it appears in
> > cc1: all warnings being treated as errors
> > make[1]: *** [hw/i386/vmport.o] Error 1
> > make: *** [subdir-x86_64-softmmu] Error 2
> > make: *** Waiting for unfinished jobs....
> > /tmp/qemu-test/src/rules.mak:66: recipe for target 'hw/i386/vmport.o' failed
> > Makefile:387: recipe for target 'subdir-x86_64-softmmu' failed
> 
> Am I missing something here?
> 

It's a false positive because the master was broken at the time your series was
tested. Now it's fixed. Sorry for the noise. Patchew is being worked on to only
report errors when the merge base passes the same tests.

Fam