Re: [Xen-devel] [PATCH] pass $($*.o-cflags) first to gcc/g++

[Stefano Stabellini <stefano.stabellini@xxxxxxxxxxxxx>]

On Thu, 10 Jul 2014, Paolo Bonzini wrote:
> Il 09/07/2014 23:59, Stefano Stabellini ha scritto:
> > On Wed, 9 Jul 2014, Paolo Bonzini wrote:
> > > What package is it that has the conflicting utils.h file?  Any chance to
> > > get
> > > it fixed in your distro?  Here I get:
> > > 
> > > $ find /usr/include/ -name utils.h
> > > /usr/include/libnl3/netlink/utils.h
> > > /usr/include/libnl3/netlink/cli/utils.h
> > > /usr/include/id3/utils.h
> > > /usr/include/octave-3.6.4/octave/utils.h
> > > 
> > > but none of them have the path in -I.
> > 
> > It's Xen: when building QEMU as part of the Xen build process,
> > tools/Makefile uses --extra-cflags to add the local Xen directories to
> > the QEMU include path. However one of the Xen header files is named
> > utils.h, conflicting with utils.h from disas/libvixl.
> > It seems to be that --extra-cflags should come after the QEMU include
> > paths.
> 
> I think the bug is in Xen then.  I have an oldish checkout and I see:
> 
> $ find . -name utils.h
> ./tools/console/daemon/utils.h
> ./tools/xenstore/utils.h

this is the one


> Could you move more headers to tools/include/ and avoid polluting QEMU_CFLAGS?

I'll see what I can do



> That said, for this particular case not using foo.o-cflags can be a stopgap
> solution too.

Thanks, I'll try to work on the Xen side to fix the problem. I'll come
back to this patch only if strictly necessary :-)

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel