All of lore.kernel.org
 help / color / mirror / Atom feed
From: Ian Campbell <ian.campbell@citrix.com>
To: Wei Liu <wei.liu2@citrix.com>
Cc: Dave Scott <Dave.Scott@eu.citrix.com>,
	Wen Congyang <wency@cn.fujitsu.com>,
	George Dunlap <george.dunlap@eu.citrix.com>,
	Jonathan Ludlam <Jonathan.Ludlam@eu.citrix.com>,
	xen-devel@lists.xen.org, Ian Jackson <ian.jackson@citrix.com>,
	Yang Hongyang <yanghy@cn.fujitsu.com>
Subject: Re: [PATCH RFC 2/4] build: Download and build extnernal blktap2.5 repo
Date: Mon, 30 Mar 2015 15:41:25 +0100	[thread overview]
Message-ID: <1427726485.13935.303.camel@citrix.com> (raw)
In-Reply-To: <20150330143322.GH25911@zion.uk.xensource.com>

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 <george.dunlap@eu.citrix.com>
> > ---
> > 
> > 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.

Ian.

  reply	other threads:[~2015-03-30 14:41 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-03-26 12:46 [PATCH RFC 0/4] tools: Upstream blktap "2.5" as an external repo George Dunlap
2015-03-26 12:46 ` [PATCH RFC 1/4] build: Reorganize and briefly document "external repo" template in tools/Makefile George Dunlap
2015-03-30  9:07   ` Ian Campbell
2015-03-26 12:46 ` [PATCH RFC 2/4] build: Download and build extnernal blktap2.5 repo George Dunlap
2015-03-26 12:55   ` Ian Campbell
2015-03-26 14:04     ` George Dunlap
2015-03-30 14:33   ` Wei Liu
2015-03-30 14:41     ` Ian Campbell [this message]
2015-03-30 14:46       ` George Dunlap
2015-03-30 14:50       ` Andrew Cooper
2015-03-30 14:43     ` George Dunlap
2015-03-30 15:38       ` Wei Liu
2015-03-31  9:55         ` George Dunlap
2015-03-31 10:40           ` Stefano Stabellini
2015-03-31 11:10             ` George Dunlap
2015-03-31 12:02               ` Ian Campbell
2015-03-31 12:03               ` Ian Campbell
2015-03-26 12:46 ` [PATCH RFC 3/4] libxl: Use XenServer's blktap2.5 George Dunlap
2015-03-26 12:46 ` [PATCH RFC 4/4] tools: Remove in-tree blktap2 George Dunlap
2015-03-26 13:50 ` [PATCH RFC 0/4] tools: Upstream blktap "2.5" as an external repo George Dunlap
2015-03-26 15:47 ` Jon Ludlam
2015-03-26 16:08   ` George Dunlap
2015-03-26 16:25     ` Felipe Franciosi
2015-03-26 17:05       ` George Dunlap
2015-03-26 17:12         ` Andrew Cooper
2015-03-26 17:23           ` George Dunlap
2015-03-26 18:06             ` Felipe Franciosi
2015-03-26 16:42     ` Jon Ludlam

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1427726485.13935.303.camel@citrix.com \
    --to=ian.campbell@citrix.com \
    --cc=Dave.Scott@eu.citrix.com \
    --cc=Jonathan.Ludlam@eu.citrix.com \
    --cc=george.dunlap@eu.citrix.com \
    --cc=ian.jackson@citrix.com \
    --cc=wei.liu2@citrix.com \
    --cc=wency@cn.fujitsu.com \
    --cc=xen-devel@lists.xen.org \
    --cc=yanghy@cn.fujitsu.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.