[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/3] crypto: require gnutls >= 3.1.18 for buildi
From: |
Daniel P . Berrangé |
Subject: |
Re: [Qemu-devel] [PATCH 1/3] crypto: require gnutls >= 3.1.18 for building QEMU |
Date: |
Mon, 6 Aug 2018 18:08:18 +0100 |
User-agent: |
Mutt/1.10.0 (2018-05-17) |
On Mon, Aug 06, 2018 at 11:58:41AM -0500, Eric Blake wrote:
> On 07/18/2018 07:03 AM, Daniel P. Berrangé wrote:
> > gnutls 3.0.0 was released in 2011 and all the distros that are build
> > target platforms for QEMU [1] include it:
> >
> > RHEL-7: 3.1.18
> > Debian (Stretch): 3.5.8
> > Debian (Jessie): 3.3.8
> > OpenBSD (ports): 3.5.18
> > FreeBSD (ports): 3.5.18
> > OpenSUSE Leap 15: 3.6.2
> > Ubuntu (Xenial): 3.4.10
> > macOS (Homebrew): 3.5.19
> >
> > Based on this, it is reasonable to require gnutls >= 3.1.18 in QEMU
> > which allows for all conditional version checks in the code to be
> > removed.
>
> Looks reasonable.
>
> >
> > [1] https://qemu.weilnetz.de/doc/qemu-doc.html#Supported-build-platforms
> >
>
> Any chance we can get http builds of the docs hosted directly on qemu.org
> someday? But unrelated to your patch.
>
> > Signed-off-by: Daniel P. Berrangé <address@hidden>
> > ---
> > @@ -2675,79 +2674,28 @@ fi
> > ##########################################
> > # GNUTLS probe
> > -gnutls_works() {
> > - # Unfortunately some distros have bad pkg-config information for gnutls
> > - # such that it claims to exist but you get a compiler error if you try
> > - # to use the options returned by --libs. Specifically, Ubuntu for
> > --static
> > - # builds doesn't work:
> > - # https://bugs.launchpad.net/ubuntu/+source/gnutls26/+bug/1478035
>
> This bug is still in state NEW, but targets Ubuntu 14.04 and "Package:
> libgnutls-dev 2.12.23-12ubuntu2.2", so is probably indeed old enough that it
> is hopefully a non-issue for gnutls > 3 in the versions of Ubuntu we
> specifically support. However, I'm not enough of an Ubuntu user myself to
> confirm whether things still work, so you'll probably want to collect
> additional R-b or Tested-by before accepting this for qemu 3.1.
FYI, I checked the pkg-config file with gnutls 3 on ubuntu and it did not
appear to have the bug that existed in gnutls 2, so I believe removing
this cruft is safe.
Regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|