From: Ian Campbell <Ian.Campbell@citrix.com>
To: Ian Jackson <Ian.Jackson@eu.citrix.com>
Cc: jfehlig@suse.com, Wei Liu <wei.liu2@citrix.com>, xen-devel@lists.xen.org
Subject: Re: [PATCH] libxl: provide xenlight.pc
Date: Thu, 15 Jan 2015 09:33:33 +0000 [thread overview]
Message-ID: <1421314413.19103.302.camel@citrix.com> (raw)
In-Reply-To: <21686.41014.686596.666172@mariner.uk.xensource.com>
On Wed, 2015-01-14 at 16:58 +0000, Ian Jackson wrote:
> Ian Campbell writes ("Re: [PATCH] libxl: provide xenlight.pc"):
> > On Fri, 2015-01-09 at 14:32 +0000, Wei Liu wrote:
> > > A pkg-config file for libxl. It also contains two variables
> > > (xenfirmwaredir and libexec_bin) so that tools that are very keen on
> > > knowing the locations of Xen binaries (say, libvirt) can use them to
> > > determine the location of the binaries.
> ...
> > This seems like a reasonable thing to provide and the manner which you
> > do so seems sane. I've not actually read any pkg-config specs to check
> > the content, but it looks plausible:
>
> Thanks, Ian. (I haven't checked the supplied .pc file.)
>
> > Acked-by: Ian Campbell <ian.campbell@citrix.com>
> >
> > We ought to consider if there are any other libraries we should do this
> > for, I think non-stable API ones like libxenctrl/guest should be
> > explicitly excluded.
>
> The obvious one is xenstore. I'm not sure how much we want to
> consider that to have a stable API.
>
> Also, what about libxl users who want to use xentoollog ?
I think both of those should be candidates for a .pc file, but that
needn't block this patch IMHO.
xentoollog could do with moving out of tools/libxc, but ISTR trying that
years ago and it turned into a yakk of some sort.
Ian.
next prev parent reply other threads:[~2015-01-15 9:33 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-01-09 14:32 [PATCH] libxl: provide xenlight.pc Wei Liu
2015-01-13 12:11 ` Ian Campbell
2015-01-13 12:19 ` Olaf Hering
2015-01-13 12:56 ` Wei Liu
2015-01-13 14:25 ` Ian Campbell
2015-01-14 16:58 ` Ian Jackson
2015-01-15 9:33 ` Ian Campbell [this message]
2015-01-19 17:00 ` Ian Campbell
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=1421314413.19103.302.camel@citrix.com \
--to=ian.campbell@citrix.com \
--cc=Ian.Jackson@eu.citrix.com \
--cc=jfehlig@suse.com \
--cc=wei.liu2@citrix.com \
--cc=xen-devel@lists.xen.org \
/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.