Re: [Xen-devel] [PATCH] Avoid relative XEN_ROOT specification

[Keir Fraser <keir.xen@xxxxxxxxx>]

On 07/10/2011 08:51, "Ian Campbell" <Ian.Campbell@xxxxxxxxxx> wrote:

> On Fri, 2011-10-07 at 05:42 +0100, Juergen Gross wrote:
>> XEN_ROOT must be set to an absolute path to avoid out of the tree references
> 
> How about:
> 
> # HG changeset patch
> # User Ian Campbell <ian.campbell@xxxxxxxxxx>
> # Date 1317973741 -3600
> # Node ID d0327c39dff8abdc224e3c19f31e11f0fbeb5c16
> # Parent  f17f0df2a2ede01a8c35dc4768a532a4438b1c02
> tools: error out on non-absolute $(XEN_ROOT)
> 
> Signed-off-by: Ian Campbell <ian.campbell@xxxxxxxxxx>

Why not put this in root Config.mk? The restriction applies outside tools/
as well.

 -- Keir

> diff -r f17f0df2a2ed -r d0327c39dff8 tools/Rules.mk
> --- a/tools/Rules.mk Wed Oct 05 10:43:41 2011 +0100
> +++ b/tools/Rules.mk Fri Oct 07 08:49:01 2011 +0100
> @@ -3,6 +3,10 @@
>  # `all' is the default target
>  all:
>  
> +ifeq ($(filter /%,$(XEN_ROOT)),)
> +$(error XEN_ROOT must be absolute)
> +endif
> +
>  include $(XEN_ROOT)/Config.mk
>  
>  export _INSTALL := $(INSTALL)
> 
> 
> 
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@xxxxxxxxxxxxxxxxxxx
> http://lists.xensource.com/xen-devel



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