Re: [libvirt PATCH] rpm: more fixes for disabling features

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

 



On Mon, Nov 02, 2020 at 02:13:41PM +0100, Pavel Hrdina wrote:
> On Mon, Nov 02, 2020 at 12:45:21PM +0000, Daniel P. Berrangé wrote:
> > The %meson macro sets "--auto-features=enabled", so it is not enough to
> > disable the driver options, we must also disable any library options
> > which the drivers depend on.
> > 
> > Signed-off-by: Daniel P. Berrangé <berrange@xxxxxxxxxx>
> > ---
> >  libvirt.spec.in | 16 ++++++++--------
> >  1 file changed, 8 insertions(+), 8 deletions(-)
> > 
> > diff --git a/libvirt.spec.in b/libvirt.spec.in
> > index 06afd0dab8..1b6a9596d8 100644
> > --- a/libvirt.spec.in
> > +++ b/libvirt.spec.in
> > @@ -1026,15 +1026,15 @@ exit 1
> >  %endif
> 
> This fixes the current issue with RPM dependencies so I'm willing to
> give it an ACK, however, I would like to propose to get rid of the
> library specific options if the library is used only by single driver,
> the library is mandatory for that driver.
> 
> Historically we had all of the libraries with autoconf because it was
> possible to provide a custom path to a library using the build option
> in addition to enable/disable state. With meson it is no longer
> possible and users have to use different ways to provide custom path
> for libraries.

I tend to have a preference for being able to disable individual libraries,
while then letting the drivers them auto-select enabled/disabled based on
whether the libraries they need are present. IOW, I think the meson options
for libraries are good - they just clash with fact that RPM forces all
auto-options to enabled, so we need this explicit hammer to turn them off
again.

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