Re: [libvirt] [dbus PATCH 5/8] maint: set the first minimal version that will be released

On Mon, Mar 19, 2018 at 01:42:11PM +0000, Daniel P. Berrangé wrote: > On Mon, Mar 19, 2018 at 10:30:49AM +0100, Pavel Hrdina wrote: > > We will require libvirt to have at least the same version as > > libvirt-dbus. The 1.2.12 version is the lowest one which we will > > support since it introduced virDomainDefineXMLFlags() and we don't > > have to support the non-flags APIs. > > > > Signed-off-by: Pavel Hrdina <phrdina(a)redhat.com&gt; > > --- > > configure.ac | 4 ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/configure.ac b/configure.ac > > index df1a375..ba397ca 100644 > > --- a/configure.ac > > +++ b/configure.ac > > @@ -1,4 +1,4 @@ > > -AC_INIT([libvirt-dbus], [0.0.1], [libvir-list(a)redhat.com], [], [http://libvirt.org]) > > +AC_INIT([libvirt-dbus], [1.2.12], [libvir-list(a)redhat.com], [], [http://libvirt.org]) > > > > AC_CONFIG_SRCDIR(src/main.c) > > AC_CONFIG_AUX_DIR([build-aux]) > > @@ -11,7 +11,7 @@ AC_USE_SYSTEM_EXTENSIONS > > > > AM_SILENT_RULES([yes]) > > > > -LIBVIRT_REQUIRED=1.2.8 > > +LIBVIRT_REQUIRED=AC_PACKAGE_VERSION > > This feels a bit odd to me - what will you set the AC_INIT version to > if you want todo 5 releases of libvirt-dbus, without bumping the min > required version of libvirt. > > I can understand having the libvirt-dbus versions be set in lockstep to > libvirt versions in AC_INIT, if you plan to always do a libvirt-dbus > release on the 1st of the month at same time as libvirt, but even then > I doubt you'd want to set min required libvirt to match. The reasoning behind it is that I would like to avoid using #if LIBVIR_CHECK_VERSION() like we do in libvirt-python and strictly depend on libvirt version that libvirt-dbus will be written for.

The other solution is to not use XML files for interface but have the interface specified directly in the code in order to be able to generate correct introspect data depending on what was compiled in. Since this is more or less a binding, I think it would be better to follow libvirt versions like we do with other bindings. This was the first idea that I had and now I realize that it's not good one. I can change it back, however we need to make sure to properly update the libvirt required version once we implement new libvirt APIs.