Re: [libvirt PATCH v2 0/2] stop using netcf for the interface driver backend

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

 



Unless there are any objections to it, I'm planning to push these two patches later today or tomorrow. (Dan - I've Cc'ed you since you were the only one to respond to V1)

After that, I'm wondering if it would be okay to backport it into Fedora Rawhide so that it makes it into Fedora 34 (since it is still enabled by default unless you do an rpm build (as long as netcf-devel is installed, which will be the case on anyone's dev machine), the only way to get more exposure is in Fedora, and since it's so sparsely used (if at all) I don't think having it just in Rawhide is going to be of any practical use). Also, this is disabling netcf for Fedora > 33, and if we fail to actually get the change in before F34 we'll either have to change that, or deal with it changing for virt-preview and anyone's local builds...


On 1/24/21 1:44 AM, Laine Stump wrote:
V1 here: https://www.redhat.com/archives/libvir-list/2021-January/msg00922.html

A short version of the cover letter from V1: this is a followup to my
proposal to stop using netcf for the interface driver backend in
Fedora/RHEL/CentOS builds and use the udev backend instead (it turns
out Ubuntu already disabled netcf in 2018).

Changes in V2:

* removed the patch that made the default netcf=disabled even when
   netcf-devel was found on the host. If someone has netcf-devel
   installed and still wants to build without netcf, then can add
   "-Dnetcf=disabled" on the meson commandline.

* Made the specfile changes more intelligent:

   * instead of hardcoding -Dnetcf=disabled, we now have a variable
     %{with_netcf} that is set to 1 for current Fedora (< 34) and current
     RHEL (< 9) but will be set to 0 for future Fedora/RHEL. This way the
     behavior on current OS releases will remain the same even for future
     libvirt.

   * it is possible to for netcf support off even in current/older OS
     releases by adding "--without netcf" to the rpmbuild commandline.

I think at this point I would be comfortable pushing these patches, unless someone has misgivings about it...

Laine Stump (2):
   build: support explicitly disabling netcf
   rpm: disable netcf for the interface driver in rpm build on new
     targets

  libvirt.spec.in | 22 +++++++++++++++++-----
  meson.build     | 10 ++++++----
  2 files changed, 23 insertions(+), 9 deletions(-)





[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