Re: [Xen-devel] [PATCH] tools/check: check for headers and libra

Roger Pau Monne writes ("[Xen-devel] [PATCH] tools/check: check for headers and 
libraries in EXTRA_INCLUDES and EXTRA_LIB"):
> tools/check: check for headers and libraries in EXTRA_INCLUDES and EXTRA_LIB

Thanks, but:

>  check-build:
> -     PYTHON=$(PYTHON) LIBXENAPI_BINDINGS=$(LIBXENAPI_BINDINGS) ./chk build
> +     PYTHON=$(PYTHON) LIBXENAPI_BINDINGS=$(LIBXENAPI_BINDINGS) 
> EXTRA_INCLUDES=$(EXTRA_INCLUDES) EXTRA_LIB=$(EXTRA_LIB) ./chk build

Perhaps this would be more easily achieved with "export" ?

>  has_header() {
> +     check_sys_root || return 1
> +

The check_sys_root stuff seems unrelated ?  Or at least, it's not
mentioned in your commit message.  I'm not sure if it should be
disentangled but it should definitely be mentioned.  (I'm not much of
an expert on tools/check.)

Also, I should probably wait with applying this until after your
revised USER_* flags variables patch, since you probably want to do a
similar job for USER_ ?

Ian.

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

<Prev in Thread]

Current Thread

[Next in Thread>

Previous by Date:

Re: [Xen-users] Re: [Xen-devel] Xen document day (Oct 12 or 26), Lars Kurth

Next by Date:

[Xen-devel] [PATCH] powernow: don't read never initialoized structure member, Jan Beulich

Previous by Thread:

[Xen-devel] [PATCH] tools/check: check for headers and libraries in EXTRA_INCLUDES and EXTRA_LIB, Roger Pau Monne

Next by Thread:

Re: [Xen-devel] [PATCH] tools/check: check for headers and libraries in EXTRA_INCLUDES and EXTRA_LIB, Roger Pau Monné

Indexes:

[Date] [Thread] [Top] [All Lists]

WARNING - OLD ARCHIVES

xen-devel

Re: [Xen-devel] [PATCH] tools/check: check for headers and libraries in