[PATCH RESEND v3 8/8] libvirt.spec: drop no longer required build dependencies

Andrea Bolognani abologna at redhat.com
Wed May 26 07:50:44 UTC 2021


On Tue, May 25, 2021 at 10:19:28PM -0400, Neal Gompa wrote:
> On Tue, May 25, 2021 at 11:34 AM Andrea Bolognani <abologna at redhat.com> wrote:
> > On Tue, May 25, 2021 at 05:06:41PM +0200, Michal Prívozník wrote:
> > > Wasn't the whole idea to drop dependencies?
> >
> > The point was to drop **build** dependencies, specifically
> > problematic ones such as ZFS. When it comes to runtime dependencies,
> > our RPMs generally try to be as featureful as possible, so adding a
> > dependency on scrub is the right thing to do IMO.
>
> Why is ZFS problematic? The zfs-fuse package is totally fine for us to
> depend on...

I'm not too familiar with the ZFS situation in Fedora / RHEL, but at
least in Debian we're patching meson.build specifically to enable ZFS
support without needing a ZFS implementation available at build time.

In general, using the build time availability of programs to guess
whether features should be enabled is a good, user-friendly behavior,
but we shouldn't second-guess the information provided by the user:
if they passed -Dstorage_zfs=enabled to Meson, then we should enable
ZFS support whether or not ZFS happens to be currently installed.

-- 
Andrea Bolognani / Red Hat / Virtualization





More information about the libvir-list mailing list