On 03/15/2011 09:25 AM, Alon Levy wrote:
On Tue, Mar 15, 2011 at 08:44:27AM -0500, Anthony Liguori wrote:
On 03/15/2011 07:42 AM, Jes Sorensen wrote:
On 03/14/11 17:40, Alon Levy wrote:
On Mon, Mar 14, 2011 at 04:20:22PM +0100, Jes Sorensen wrote:
ok, here is a note where I kinda ignored my own wishes but I want
to be very clear on them:
libcacard should not be part of qemu.
it is here because I once thought it would speed things up.
So I'm not taking it out or anything - it's fine with me that it
goes into qemu, just as long as it's understood that I'm now maintaining
another copy of it for usage outside of qemu, in the spice client (or
any other client for that matter - it will be the same when we do vnc
support for this).
Hi Alon,
This bit is somewhat problematic. If QEMU is maintaining a copy of
libcacard, then that has to comply with the QEMU way of doing things.
QEMU cannot rely on various portions in the tree behaving in different
ways. Otherwise it really should be an external library requirement
pulled in by the build.
I am not sure what is the best way, if it stays in QEMU people will
eventually start making modifications to it, without looking at the
other copy that is being maintained.
Two copies is not really practical. QEMU should be the place that
owns it and things should be consuming a .so from QEMU.
My bad - I thought you didn't want this. I can do a patch to make qemu
build an .so file if configure gets a "--libs", how does that sound?
Yeah, we probably should target being able to do something like:
configure --target-list="" --disable-tools --enable-libs
And it just build libs.
right now that would build just libcacard, I guess libqmp too later?
or perhaps have a separate Makefile (Makefile.libs)? Have you given this
any thought?
I was half considering seeing if I could get away with using libtool ;-)
We can do something pretty simple for now and only build libs when GCC
is available and we're on Linux.
Regards,
Anthony Liguori
Regards,
Anthony Liguori
Alternatively the external apps that build against it should be taught
to link with the QEMU version.
Cheers,
Jes