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

[Laine Stump]

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(-)
>