From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Luis R. Rodriguez" Subject: Re: [Xen-devel] [PATCH 2/9] Use gnutls_priority_set_direct() to deprecate gnutls_*_set() Date: Sat, 6 Feb 2016 02:44:54 +0100 Message-ID: <20160206014454.GA25240@wotan.suse.de> References: <1448041672-3986-1-git-send-email-mcgrof@do-not-panic.com> <1448041672-3986-3-git-send-email-mcgrof@do-not-panic.com> <20151125145322.GA24630@x230.dumpdata.com> <20151125193651.GH9528@wotan.suse.de> <20151125204420.GA20267@x230.dumpdata.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <20151125204420.GA20267@x230.dumpdata.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: cocci-bounces@systeme.lip6.fr Errors-To: cocci-bounces@systeme.lip6.fr To: Konrad Rzeszutek Wilk Cc: xen-devel@lists.xensource.com, ian.campbell@citrix.com, pryorm09@gmail.com, cfergeau@redhat.com, JBeulich@suse.com, samuel.thibault@ens-lyon.org, cocci@systeme.lip6.fr List-Id: xen-devel@lists.xenproject.org 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" > > > > > > > > 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 None of these are merged yet... Luis