Re: [Xen-devel] [PATCH 2 of 2] build: append YAJL_CLFAGS and YAJL_LDFLAGS to libxl build

[Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>]

Roger Pau Monné writes ("Re: [Xen-devel] [PATCH 2 of 2] build: append 
YAJL_CLFAGS and YAJL_LDFLAGS to libxl build"):
> 2011/10/6 Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>:
> > I think this should be done by setting EXTRA_INCLUDES and EXTRA_LIB in
> > Config.mk or .config, and shouldn't need an extra pair of variables.
> 
> The problem is that EXTRA_INCLUDES and EXTRA_LIB is added to the whole
> compilation process, and if I have old xen header files in this
> folders together with the yajl headers for example, they get included
> instead of the new ones, and the build fails.

Ah.  I don't think the approach of only using these flags when
compiling libxl is a good one, then, as it would build libxl against
these putative old Xen headers.

I think the problem then is that the ordering of the various -I and -L
options is wrong.  The EXTRA_* should come near the end, after all of
the -I and -L specified by the build system to refer to other parts of
the Xen tree.

Would you like to look into how to make that work :-) ?

Ian.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel