From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrew Cooper Subject: Re: [PATCH RFC 2/4] build: Download and build extnernal blktap2.5 repo Date: Mon, 30 Mar 2015 15:50:59 +0100 Message-ID: <551962D3.1050300@citrix.com> References: <1427373970-25399-1-git-send-email-george.dunlap@eu.citrix.com> <1427373970-25399-3-git-send-email-george.dunlap@eu.citrix.com> <20150330143322.GH25911@zion.uk.xensource.com> <1427726485.13935.303.camel@citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1427726485.13935.303.camel@citrix.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Ian Campbell , Wei Liu Cc: Dave Scott , Wen Congyang , George Dunlap , Jonathan Ludlam , xen-devel@lists.xen.org, Ian Jackson , Yang Hongyang List-Id: xen-devel@lists.xenproject.org On 30/03/15 15:41, Ian Campbell wrote: > On Mon, 2015-03-30 at 15:33 +0100, Wei Liu wrote: >> On Thu, Mar 26, 2015 at 12:46:08PM +0000, George Dunlap wrote: >>> Download and build XenServer's blktap as an external tree, similar to >>> qemu-xen. >>> >>> As of this patch we just download and build it, but don't install or >>> use it. >>> >>> Signed-off-by: George Dunlap >>> --- >>> >>> FIXME: Directly use the XenServer github repo for now, while we're >>> discussing things. If we decide to take this series, we'll have to >>> clone the tree on xenbits and remove the FIXME line. >>> >> IMHO we need to support --with-system-blktap= in configure in case >> distro wants to package blktap separately. Not sure if in practice this >> makes sense since AIUI blktap is only used by Xen. > I think you are right, there are at least some distros which ship a > blktap package not from Xen. e.g. Debian ship, or have in the past > shipped, the blktap2.5 packages. > > blktap is a bit different to some of the other similar cases though in > that there is a circular dependency, although George indicated last week > that he thought that might be an easily fixed one. I seem to recall that some bits of blktap need xen-devel (depending on exactly what is built). libxl will need blktap-devel, so these dependences can be resolved. ~Andrew