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] libxl, buildmakevars2file: Do not relink libxl e

To: Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] libxl, buildmakevars2file: Do not relink libxl each time
From: Keir Fraser <keir.fraser@xxxxxxxxxxxxx>
Date: Mon, 20 Sep 2010 18:41:58 +0100
Cc:
Delivery-date: Mon, 20 Sep 2010 10:42:58 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <19607.37461.506269.556285@xxxxxxxxxxxxxxxxxxxxxxxx>
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/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: ActY5PRWDCJTJ2JzSmmXKOy9ttPM9QABis02
Thread-topic: [Xen-devel] [PATCH] libxl, buildmakevars2file: Do not relink libxl each time
User-agent: Microsoft-Entourage/12.26.0.100708
On 20/09/2010 17:56, "Ian Jackson" <Ian.Jackson@xxxxxxxxxxxxx> wrote:

> Ian Jackson writes ("[Xen-devel] [PATCH] libxl, buildmakevars2file: Do not
> relink libxl each time"):
>> Make Config.mk's definition of buildmakevars2file and
>> tools/libxl/Makefile's rule for _libxl_paths.h generate their files to
>> temporary files and only rename them into place if the intended file
>> has changed.  This gets rid of unnecessarily compilation and link
>> steps.
> 
> Looks like *.tmp isn't in the .hgignore!  If the patch meets with
> approval I'll add that too.

Fine by me.

Acked-by: Keir Fraser <keir.fraser@xxxxxxxxxx>

 -- Keir



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

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