WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

Re: [Xen-devel] [PATCH] Fix tools makefile to properly check for externa

To: xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] Fix tools makefile to properly check for external tools
From: Jerone Young <jyoung5@xxxxxxxxxx>
Date: Tue, 04 Oct 2005 14:04:41 -0500
Delivery-date: Tue, 04 Oct 2005 19:05:02 +0000
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <1128451606.15111.10.camel@thinkpad>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <1128451606.15111.10.camel@thinkpad>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Ok ,thinking about the comment in tools/check/Makefile is so that one
could install stuff in "dist" directory. But our networking scripts have
no checks at all to see if (for example) "brctl" is even there...it just
silently installs. So given this when would one check that we have
everything need installed. Also I will submit a patch to add a check to
the network script to see if the tools are actually on the system.

On Tue, 2005-10-04 at 13:46 -0500, Jerone Young wrote:
> Signed-off-by: Jerone Young <jyoung5@xxxxxxxxxx>
> 
> Fix tools makefile so that extrnal tools checks are run to ensure
> extrnal tools  are actually installed.
> 
> diff -r 00037ba13f0b -r cc75d51355cb tools/Makefile
> --- a/tools/Makefile    Tue Oct  4 17:23:58 2005
> +++ b/tools/Makefile    Tue Oct  4 16:28:18 2005
> @@ -33,7 +33,7 @@
>         done
>         $(MAKE) ioemu
> 
> -install: check
> +install: check_install
>         @set -e; for subdir in $(SUBDIRS); do \
>                 $(MAKE) -C $$subdir $@; \
>         done
> @@ -48,6 +48,9 @@
>  check:
>         $(MAKE) -C check
> 
> +check_install:
> +       $(MAKE) -C check install
> +
>  check_clean:
>         $(MAKE) -C check clean
> 
> 
-- 
Jerone Young
IBM Linux Technology Center
jyoung5@xxxxxxxxxx
512-838-1157 (T/L: 678-1157)


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

<Prev in Thread] Current Thread [Next in Thread>