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

[Fam Zheng]

On Sat, 12/23 08:36, Corey Minyard wrote:
> On 12/21/2017 05:10 PM, address@hidden 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