On 12-07-12 14:32, Daniel P. Berrange wrote:
On Thu, Jul 12, 2012 at 02:20:11PM +0200, Christophe Fergeau wrote:
Hey,
On Wed, Jul 11, 2012 at 03:42:20PM +0100, Daniel P. Berrange wrote:
On Wed, Jul 11, 2012 at 10:23:21AM -0400, Cole Robinson wrote:
I think this comes from rbd support, from configure.ac:
LIBRBD_LIBS="-lrbd -lrados -lcrypto"
from this commit:
http://libvirt.org/git/?p=libvirt.git;a=commit;h=74951eadef85e2d100c7dc7bd9ae1093fbda722f
What's missing here? RPM BuildRequires, configure change, ...?
If rbd is include '-lcrypto' in its pkgconfig line, then its
RPM is responsible for ensuring BuildRequires is set in its
own -devel package. So sounds like there's BR missing there.
ceph-devel does not seem to come with a pkgconfig file, and the LIBRBD_LIBS
line Cole quoted comes from libvirt's own configure.ac. I've checked with
ldd that nor librbd nor librados link with libcrypto.
Ah, I see that now, in configure.ac we have
LIBRBD_LIBS="-lrbd -lrados -lcrypto"
I vote for just removing the '-lcrypto' line from configure.ac then.
If libcrypto is needed for rbd support, then libvirt should check itself
that libcrypto is present. All of this would indeed be much simpler if
ceph shipped a pkgconfig file...
Yeah, I despair everytime I find a modern library that does not use
pkgconfig :-(
I'll open a issue for this at the Ceph guys, we can probably sort that out.
Wido
Daniel
--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list