[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [libvirt] [PATCH v3 1/3] Add virXMLValidateAgainstSchema helper method

On Fri, Jan 16, 2015 at 01:58:27PM +0000, Ian Campbell wrote:
> Hello,
> On Tue, 2015-01-13 at 17:00 +0000, Daniel P. Berrange wrote:
> > +#  define VIR_WARNINGS_NO_PRINTF \
> > +    _Pragma ("GCC diagnostic push") \
> > +    _Pragma ("GCC diagnostic ignored \"-Wsuggest-attribute=format\"")
> Xen automated tests are failing to build on all architectures with:
>         util/virxml.c: In function 'catchRNGError':
>         util/virxml.c:1094:9: error: unknown option after '#pragma GCC 
> diagnostic' kind [-Werror=pragmas]
> which I think must be down to one of these additions.
> (helpful of gcc not to print the unknown option in question!)
> test overview:
> http://www.chiark.greenend.org.uk/~xensrcts/logs/33443/
> specific failure log:
> http://www.chiark.greenend.org.uk/~xensrcts/logs/33443/build-amd64-libvirt/5.ts-libvirt-build.log
> We use Debian Wheezy's gcc, which is 4.6.3 AFAIK.

The configure logs show

checking whether C compiler handles -Wsuggest-attribute=const... yes
checking whether C compiler handles -Wsuggest-attribute=format... no
checking whether C compiler handles -Wsuggest-attribute=noreturn... yes
checking whether C compiler handles -Wsuggest-attribute=pure... yes

So, can someone with a Debian machine check if it helps to modify

    _Pragma ("GCC diagnostic ignored \"-Wsuggest-attribute=format\"")

To be just

    _Pragma ("GCC diagnostic ignored \"-Wsuggest-attribute\"")

THough, I guess some very old gcc might not support -Wsuggest-attribute
at all, so perhaps we need to check this fully

|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|

Xen-devel mailing list



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.