[libvirt] [PATCH] build: set --without-firewalld-zone in configure commandline for Fedora 30

Laine Stump posted 1 patch 4 years, 11 months ago
Test syntax-check passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/20190411170319.18165-1-laine@laine.org
libvirt.spec.in | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[libvirt] [PATCH] build: set --without-firewalld-zone in configure commandline for Fedora 30
Posted by Laine Stump 4 years, 11 months ago
The firewalld package in Fedora 30 didn't get support for rich rule
priorities, which is required by the libvirt zonefile that's installed
when the build is configured with --with-firewalld-zone, so we need to
set --without-firewalld-zone for that version of Fedora. The needed
feature is already upstream in firewalld, so it just needs another
upstream release to be there. Let's be optimistic and assume that will
happen prior to F31.

Signed-off-by: Laine Stump <laine@laine.org>
---
 libvirt.spec.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libvirt.spec.in b/libvirt.spec.in
index 8ea27c91a3..18f0a0e217 100644
--- a/libvirt.spec.in
+++ b/libvirt.spec.in
@@ -137,7 +137,7 @@
 
 %define with_firewalld 1
 
-%if 0%{?fedora} >= 30 || 0%{?rhel} > 7
+%if 0%{?fedora} >= 31 || 0%{?rhel} > 7
     %define with_firewalld_zone 0%{!?_without_firewalld_zone:1}
 %endif
 
-- 
2.20.1

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH] build: set --without-firewalld-zone in configure commandline for Fedora 30
Posted by Eric Garver 4 years, 11 months ago
On Thu, Apr 11, 2019 at 01:03:19PM -0400, Laine Stump wrote:
> The firewalld package in Fedora 30 didn't get support for rich rule
> priorities, which is required by the libvirt zonefile that's installed
> when the build is configured with --with-firewalld-zone, so we need to
> set --without-firewalld-zone for that version of Fedora. The needed
> feature is already upstream in firewalld, so it just needs another
> upstream release to be there. Let's be optimistic and assume that will
> happen prior to F31.
> 
> Signed-off-by: Laine Stump <laine@laine.org>
> ---
>  libvirt.spec.in | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/libvirt.spec.in b/libvirt.spec.in
> index 8ea27c91a3..18f0a0e217 100644
> --- a/libvirt.spec.in
> +++ b/libvirt.spec.in
> @@ -137,7 +137,7 @@
>  
>  %define with_firewalld 1
>  
> -%if 0%{?fedora} >= 30 || 0%{?rhel} > 7
> +%if 0%{?fedora} >= 31 || 0%{?rhel} > 7
>      %define with_firewalld_zone 0%{!?_without_firewalld_zone:1}
>  %endif
>  

Acked-by: Eric Garver <eric@garver.life>

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH] build: set --without-firewalld-zone in configure commandline for Fedora 30
Posted by Daniel P. Berrangé 4 years, 11 months ago
On Thu, Apr 11, 2019 at 01:03:19PM -0400, Laine Stump wrote:
> The firewalld package in Fedora 30 didn't get support for rich rule
> priorities, which is required by the libvirt zonefile that's installed
> when the build is configured with --with-firewalld-zone, so we need to
> set --without-firewalld-zone for that version of Fedora. The needed
> feature is already upstream in firewalld, so it just needs another
> upstream release to be there. Let's be optimistic and assume that will
> happen prior to F31.
> 
> Signed-off-by: Laine Stump <laine@laine.org>
> ---
>  libvirt.spec.in | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>


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 :|

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list