Re: [Xen-devel] [PATCH 2/9] Use gnutls_priority_set_direct() to deprecate gnutls_*_set()

["Luis R. Rodriguez" <mcgrof@xxxxxxxx>]

On Fri, Feb 05, 2016 at 10:45:54PM -0500, Konrad Rzeszutek Wilk wrote:
> On Sat, Feb 06, 2016 at 02:44:54AM +0100, Luis R. Rodriguez wrote:
> > On Wed, Nov 25, 2015 at 03:44:30PM -0500, Konrad Rzeszutek Wilk wrote:
> > > On Wed, Nov 25, 2015 at 08:36:51PM +0100, Luis R. Rodriguez wrote:
> > > > On Wed, Nov 25, 2015 at 09:53:27AM -0500, Konrad Rzeszutek Wilk wrote:
> > > > > On Fri, Nov 20, 2015 at 09:47:45AM -0800, Luis R. Rodriguez wrote:
> > > > > > From: "Luis R. Rodriguez" <mcgrof@xxxxxxxx>
> > > > > > 
> > > > > > Using deprecate gnutls_*_set() triggers a failure to compile
> > > > > > with gnutls30-3.4.4, used on OpenSUSE factory:
> > > > > > 
> > > > > > ../libqemu_common.a(vnc.o): In function `vnc_start_tls':
> > > > > > ~/devel/xen/tools/qemu-xen-traditional-dir/vnc.c:2164: undefined 
> > > > > > reference to `gnutls_kx_set_priority'
> > > > > > ~/devel/xen/tools/qemu-xen-traditional-dir/vnc.c:2171: undefined 
> > > > > > reference to `gnutls_certificate_type_set_priority'
> > > > > > ~/devel/xen/tools/qemu-xen-traditional-dir/vnc.c:2178: undefined 
> > > > > > reference to `gnutls_protocol_set_priority'
> > > > > > 
> > > > > > This compilation issue can be fixed by using the new routine
> > > > > > gnutls_priority_set_direct() which replaces the deprecated calls
> > > > > > which also simplifies the code considerably.
> > > > > 
> > > > > 
> > > > > Thanks for posting that! It certainly fixes that issue.
> > > > 
> > > > Acked-by?
> > > 
> > > Tested-by: Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx>
> > 
> > None of these are merged yet...
> 
> And it looks like you didn't CC the maintainer.

mcgrof@garbanzo ~/devel/xen (git::staging)$ ./scripts/get_maintainer.pl 
../xen-fixes/0002-proper-qemu-trad-gnutls.patch 
Ian Campbell <ian.campbell@xxxxxxxxxx>
Ian Jackson <ian.jackson@xxxxxxxxxxxxx>
Jan Beulich <jbeulich@xxxxxxxx>
Keir Fraser <keir@xxxxxxx>
Tim Deegan <tim@xxxxxxx>
xen-devel@xxxxxxxxxxxxx

I missed Ian Jackson, Keir and Tim.

> Also - how does this work if you have an older version of SuSE,
> say SLES10?

Beats me. I just dealt with getting this to compile with what is current.
I kind of doubt what is in xen master or staging will end up in SLE10's
version of Xen too. If that had to happen people porting this to an old
library could just add a helper gnutls_priority_set_direct() wrapper to
do things the old way.

  Luis

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel