[libvirt] [PATCH] Fix detection of GnuTLS 1.x.y
Eric Blake
eblake at redhat.com
Wed Aug 3 14:48:33 UTC 2011
On 08/03/2011 08:22 AM, Matthias Bolte wrote:
> Detection based on gnutls_session doesn't work because GnuTLS 2.x.y
> comes with a compat.h that defines gnutls_session to gnutls_session_t.
>
> Instead detect this based on LIBGNUTLS_VERSION_MAJOR.
> ---
> configure.ac | 22 +++++++++++++++-------
> 1 files changed, 15 insertions(+), 7 deletions(-)
ACK for correctness.
However, I wonder if it is the most efficient. We aren't using this as
a Makefile conditional, and LIBGNUTLS_VERSION_MAJOR is available at
compile time. That is, I think that we could skip the configure check
altogether, and just move the #if LIBGNUTLS_VERSION_MAJOR < 2 check into
a common header used by all files that want to use gnutls in the first
place, for a slightly smaller and faster configure.
--
Eric Blake eblake at redhat.com +1-801-349-2682
Libvirt virtualization library http://libvirt.org
More information about the libvir-list
mailing list