[libvirt] [PATCH 2/4] configure: Make ACL mandatory when building the QEMU driver
Andrea Bolognani
abologna at redhat.com
Tue Feb 14 16:14:33 UTC 2017
On Tue, 2017-02-14 at 16:00 +0000, Daniel P. Berrange wrote:
> Your platform has libacl available so it is not difficult to fix that
> by building with libacl support. We print out the configure summary
> precisely so users can see if there's any libraries they forgot to
> install which might be useful.
>
> Mandating libacl will prevent use of the QEMU driver on platforms lacking
> the libacl library. I see various bug reports indicating portability
> problems for libacl wrt other platforms, in particular OS-X. So IMHO it
> is not acceptable to make it a mandatory requirement, when there's no
> good reason for that aside beyond helping people who forget to install
> -devel library packages
FreeBSD and other have <sys/acl.h>, it's just not part of a
separate package but of libc itself. Not sure about macOS,
but it being a BSD derivative I expect it wouldn't be too
different.
On the other hand, we really only care about having the ACL
APIs when we are isolating QEMU, which only happens of Linux
due to the namespaces requirement... So maybe we could have
it as a strict requirement on Linux only, and as an optional
dependency on other platforms?
--
Andrea Bolognani / Red Hat / Virtualization
More information about the libvir-list
mailing list