Daniel P. Berrangé <berra...@redhat.com> writes: > On Thu, Dec 01, 2022 at 01:39:13PM +0100, Markus Armbruster wrote: >> Daniel P. Berrangé <berra...@redhat.com> writes: >> >> > On Thu, Dec 01, 2022 at 07:13:05AM +0100, Markus Armbruster wrote: >> >> HMP "info spice" has a bit of code to show channel type >> >> SPICE_CHANNEL_WEBDAV as "webdav", disabled since commit 7c6044a94e >> >> "hmp: info spice: take out webdav" (v2.3.0), because it compiles only >> >> with Spice versions 0.12.7 and later. Our minimum version is 0.12.5. >> > >> > Last version bump was 4 years ago >> > >> > commit 1b63665c2c0e0d52735e0dd5217f109fe0dd2322 >> > Author: Marc-André Lureau <marcandre.lur...@redhat.com> >> > Date: Wed Nov 28 19:59:32 2018 +0400 >> > >> > configure: bump spice-server required version to 0.12.5 >> > >> > ...snip.... >> > >> > According to repology, all the distros that are build target platforms >> > for QEMU include it: >> > >> > RHEL-7: 0.14.0 >> > Debian (Stretch): 0.12.8 >> > Debian (Jessie): 0.12.5 >> > FreeBSD (ports): 0.14.0 >> > OpenSUSE Leap 15: 0.14.0 >> > Ubuntu (Xenial): 0.12.6 >> > >> > We moved on from Debian and RHEL since then >> > >> > Debian 11: 0.14.3 >> > RHEL-8: 0.14.2 >> > FreeBSD (ports): 0.14.4 >> > Fedora 35: 0.14.0 >> > Ubuntu 20.04: 0.14.0 >> > OpenSUSE Leap 15.3: 0.14.3 >> > >> > IOW, we can bump to 0.14.0, and then revert the >> > webdav conditional commit. >> >> We need to bump spice-protocol, actually. > > Opps, I'm getting mixed up. The commit I mentioned was spice-server, > but the new versions I've listed just were indeed spice-protocol > >> Would you like me to bump spice-server as well? To which version? > > Yes, might as well, the spice-server versions are slightly different: > > Debian 11: 0.14.3 > RHEL-8: 0.14.3 > FreeBSD (ports): 0.15.0 > Fedora 35: 0.15.0 > Ubuntu 20.04: 0.14.2 > OpenSUSE Leap 15.3: 0.14.3 > > I think we might as well pick 0.14.0 for both protocol and server.
Makes sense, but it leads to another question. I found obvious version checks for spice-protocol, and dropped the outmoded ones, namely #if CHECK_SPICE_PROTOCOL_VERSION(0, 14, 0) For spice-server, I see a bunch of SPICE_INTERFACE_FOO_{MAJOR,MINOR} we check, and which ones become outmoded is not obvious to me. Help?