Re: [libvirt] [Xen-devel] [libvirt test] 50401: regressions - FAIL

On Wed, 2015-04-15 at 14:16 +0100, Daniel P. Berrange wrote: FYI our bisector has now tripped over another related problem, http://lists.xen.org/archives/html/xen-devel/2015-04/msg01900.html http://lists.xen.org/archives/html/xen-devel/2015-04/msg01968.html in particular: http://logs.test-lab.xenproject.org/osstest/logs/50460/build-armhf-libvir...: qemu/qemu_domain.c: In function 'qemuDomainSupportsBlockJobs': qemu/qemu_domain.c:3067:11: error: declaration of 'sync' shadows a global declaration [-Werror=shadow] cc1: all warnings being treated as errors Essentially another instance of the same class of issue, in this case arising from: commit 1eccac1d2da7bbe97e1df25fd0ddac6e71b0794a Author: Peter Krempa <pkrempa(a)redhat.com&gt; Date: Tue Mar 31 17:29:35 2015 +0200 qemu: domain: Add helper to check block job support Given the code in autoconf to enable essentially all warnings I'm a bit surprised -Wshadow isn't being enabled for everyone -- I thought it was quite a long standing warning. I'm also not sure why we are having a sudden rash of these now. But perhaps it is worth adding Wshadow to the set of excluded options until either it is more widely supported or the libvirt code base has been audited? I can send such a patch if you think that's desirable. Ian.