From mboxrd@z Thu Jan 1 00:00:00 1970 From: cardoe@cardoe.com (Doug Goldstein) Date: Sat, 6 Feb 2016 14:18:46 -0600 Subject: [Cocci] [Xen-devel] [PATCH 2/9] Use gnutls_priority_set_direct() to deprecate gnutls_*_set() In-Reply-To: <20160206040322.GB25240@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> <20160206014454.GA25240@wotan.suse.de> <20160206034554.GA14513@localhost.localdomain> <20160206040322.GB25240@wotan.suse.de> Message-ID: <56B65526.3060002@cardoe.com> To: cocci@systeme.lip6.fr List-Id: cocci@systeme.lip6.fr On 2/5/16 10:03 PM, Luis R. Rodriguez wrote: > 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" >>>>>>> >>>>>>> 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... >> >> And it looks like you didn't CC the maintainer. > > mcgrof at garbanzo ~/devel/xen (git::staging)$ ./scripts/get_maintainer.pl ../xen-fixes/0002-proper-qemu-trad-gnutls.patch > Ian Campbell > Ian Jackson > Jan Beulich > Keir Fraser > Tim Deegan > xen-devel at lists.xen.org > > 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 > I'd personally love to know what's the real minimum that needs to be supported by Xen master/staging. I tried [1] to put a line in the sand but that died. I'm also in the process on standing up some basic build CI [2] and as part of that I would build on new stuff and build on the oldest set that's supported. [1] http://lists.xenproject.org/archives/html/xen-devel/2016-01/msg03577.html [2] http://lists.xenproject.org/archives/html/xen-devel/2016-02/msg00984.html -- Doug Goldstein -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 959 bytes Desc: OpenPGP digital signature URL: From mboxrd@z Thu Jan 1 00:00:00 1970 From: Doug Goldstein Subject: Re: [PATCH 2/9] Use gnutls_priority_set_direct() to deprecate gnutls_*_set() Date: Sat, 6 Feb 2016 14:18:46 -0600 Message-ID: <56B65526.3060002@cardoe.com> 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> <20160206014454.GA25240@wotan.suse.de> <20160206034554.GA14513@localhost.localdomain> <20160206040322.GB25240@wotan.suse.de> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============5347859381311018722==" Return-path: In-Reply-To: <20160206040322.GB25240@wotan.suse.de> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: "Luis R. Rodriguez" , Konrad Rzeszutek Wilk Cc: xen-devel@lists.xensource.com, ian.campbell@citrix.com, "Luis R. Rodriguez" , ian.jackson@eu.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 This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --===============5347859381311018722== Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="PBefNlB1aORNOKpO2vuINw55QkgIGoMie" This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --PBefNlB1aORNOKpO2vuINw55QkgIGoMie Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: quoted-printable On 2/5/16 10:03 PM, Luis R. Rodriguez wrote: > 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 wro= te: >>>>>> 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... >> >> And it looks like you didn't CC the maintainer. >=20 > mcgrof@garbanzo ~/devel/xen (git::staging)$ ./scripts/get_maintainer.pl= ../xen-fixes/0002-proper-qemu-trad-gnutls.patch=20 > Ian Campbell > Ian Jackson > Jan Beulich > Keir Fraser > Tim Deegan > xen-devel@lists.xen.org >=20 > I missed Ian Jackson, Keir and Tim. >=20 >> Also - how does this work if you have an older version of SuSE, >> say SLES10? >=20 > Beats me. I just dealt with getting this to compile with what is curren= t. > 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. >=20 > Luis >=20 I'd personally love to know what's the real minimum that needs to be supported by Xen master/staging. I tried [1] to put a line in the sand but that died. I'm also in the process on standing up some basic build CI [2] and as part of that I would build on new stuff and build on the oldest set that's supported. [1] http://lists.xenproject.org/archives/html/xen-devel/2016-01/msg03577.html= [2] http://lists.xenproject.org/archives/html/xen-devel/2016-02/msg00984.html= --=20 Doug Goldstein --PBefNlB1aORNOKpO2vuINw55QkgIGoMie Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG/MacGPG2 v2.0 iQJ8BAEBCgBmBQJWtlUrXxSAAAAAAC4AKGlzc3Vlci1mcHJAbm90YXRpb25zLm9w ZW5wZ3AuZmlmdGhob3JzZW1hbi5uZXRBNTM5MEQ2RTNFMTkyNzlCNzVDMzIwOTVB MkJDMDNEQzg3RUQxQkQ0AAoJEKK8A9yH7RvUz2oP/1x0qg0bHV5bkUv0U+4yPZZ4 dL76YWtdbkuGlRVZ6Mfd8WVe+qEHGJm8qliDVyMKMLbNgIZ8HUddLid8fpsVD+4m q9+tPSxl8WwnEafyuSPTh1u6ssPivigcpZX1kfxNnIXyE0kd2MaCKYJqKDZfjEZP Kxa+MC2+33gKNhpVBPHvk5RJapz0ulxJIHtgPnrYQxorcAAPSjcxM5evA4XLqNCt LoymU2BkEyg32aAg9M0ddyB9CyYvkkbOwN9+LuWgHKcaTKvGfhL9T8f49QaVgr8r LUtB+5djkcSUkQwjTSf7mpYS2briL/6csMCJjo+VBxHhlwI3s02AQfzVzZLwxOYj ubElsGJSsE5DTwCrAzv3hvLKPiy06uwQXcMVwD97xgesxGM24rYnnW/HhqrBN36Z W6fVLC30Hg4Z84dqtYeh2HhfSTadvlwT4mnkP1LQ0ZBltRVF0WVL7SQFTK+3W/cK feVEg6mGeunW9C3p5LzJeCfZEbTu+M9Fx8yg74rBPgDmATzrfqW2/b8Lou0E38eg RQ9uE+8g3ECcwbEz/aDVQPPRxjl3evj76gMlIPpQuOT7X/VGqmKZO7uxITQViRd8 ATOe5eBhnCh6/M16yD4BzIWNmNFAZscLscXDEEUBxKMlF0rGPY8uV4DtmfwLd1j8 eY4yDhD7xcsVyJdecyqa =+6ep -----END PGP SIGNATURE----- --PBefNlB1aORNOKpO2vuINw55QkgIGoMie-- --===============5347859381311018722== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel --===============5347859381311018722==--