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

[Xen-devel] Re: [PATCH] tools/check: check for yajl (needed by libxl)

To: Ian Campbell <Ian.Campbell@xxxxxxxxxxxxx>
Subject: [Xen-devel] Re: [PATCH] tools/check: check for yajl (needed by libxl)
From: Olaf Hering <olaf@xxxxxxxxx>
Date: Fri, 30 Sep 2011 14:15:42 +0200
Cc: Anthony Perard <anthony.perard@xxxxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Fri, 30 Sep 2011 05:16:43 -0700
Dkim-signature: v=1; a=rsa-sha1; c=relaxed/relaxed; t=1317384959; l=730; s=domk; d=aepfle.de; h=In-Reply-To:Content-Type:MIME-Version:References:Subject:Cc:To:From: Date:X-RZG-CLASS-ID:X-RZG-AUTH; bh=2csq1r85+9lDoJ77QuBBjyZx8Qs=; b=wg5rPjzsYeu0TXDdByvq9OXysrskpTcSCq54bILpTfOtYRiC4dcYbKsIliYIO8GoOkV GyVK5kRxlCPE+o7Kd1Um37bJuXQRGMSzCPwq1475krKlZrMQ/J3Z2fwBku3IgrJzm2hqt ifNjJZoQ/z+w5s0Yg5uEq47dwrI4uNSwnno=
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <1317384428.26672.269.camel@xxxxxxxxxxxxxxxxxxxxxx>
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>
References: <4b98868690218126b906.1317368189@xxxxxxxxxxxxxxxxxxxxxxxxx> <20110930120210.GA25618@xxxxxxxxx> <1317384428.26672.269.camel@xxxxxxxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mutt/1.5.21.rev5535 (2011-07-01)
On Fri, Sep 30, Ian Campbell wrote:

> On Fri, 2011-09-30 at 13:02 +0100, Olaf Hering wrote:
> > On Fri, Sep 30, Ian Campbell wrote:
> > 
> > > (note to committer, ensure the new file is executable)
> > 
> > Please dont, thats not patch friendly.
> > 
> > Shouldnt all script files be called with like 
> > '$interpreter $script' instead of './$script'?
> 
> In general I agree, but that isn't how it the infrastructure in
> tools/check/chk works right now.

There is tools/check/check_logging, which says it requires code with is
included in Python 2.3 and later.
The toplevel README states 2.3 or later is required, so
tools/check/check_logging could be removed and line 63 in
tools/check/chk be updated.

Olaf

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