Re: [libvirt PATCH] remote: fix driver name check for libxl driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, May 05, 2020 at 12:46:16PM +0200, Pavel Hrdina wrote:
> On Tue, May 05, 2020 at 12:37:58PM +0200, Andrea Bolognani wrote:
> > On Mon, 2020-05-04 at 12:10 -0600, Jim Fehlig wrote:
> > > On 5/4/20 11:43 AM, Daniel P. Berrangé wrote:
> > > > Can we changes the driver tables to use "Xen" too, now that we got
> > > > rid of the old Xen driver. I can't remember if the driver tables
> > > > names leak out anywhere important ?
> > > 
> > > It doesn't appear to be the case, but I only looked quickly and could have 
> > > missed something.
> > > 
> > > I'll send a patch to change 'name' in the driver tables. With the old Xen driver 
> > > gone Andrea and I also discussed renaming the libxl driver to xen, but I'm not 
> > > sure it is worth the effort. It seems difficult to remove all references to 
> > > libxl. E.g. changing the '--with-libxl' configure option would break external 
> > > users/scripts.
> > 
> > I think it's fine to make changes to configure options, as long as
> > passing the old options results in a failure rather than silently
> > (not) enabling the driver. Our API stability guarantee thankfully
> > does *not* extend to configure :)
> > 
> > On the other hand, Any Minute Now™ we're going to switch to Meson,
> > at which point *all* of the existing options will break and all of
> > the scripts that exist will have to be updated. Maybe we can avoid
> > doing anything to autotools and just make sure the corresponding
> > Meson option is named correctly from day one?
> 
> I would probably change the name beforehand in configure to make the
> transition to Meson as direct as possible for the sake of reviewers.
> It can be done within the same release as the switch to meson to
> minimize impact to all building scripts.

Yeah, that makes sense to me.


Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|





[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux