rpm spec: don't assume %{fedora} exists as a macro

[libvirt] [PATCH] rpm spec: don't assume %{fedora} exists as a macro

Daniel P. Berrangé posted 1 patch 5 years, 2 months ago

Download mbox

Test syntax-check passed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/20190121140637.28806-1-berrange@redhat.com

libvirt.spec.in | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

Expand all Fold all

[libvirt] [PATCH] rpm spec: don't assume %{fedora} exists as a macro

Posted by Daniel P. Berrangé 5 years, 2 months ago

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

Pushed under the "fix the previous build breaker fix" rule :-( Sorry.

 libvirt.spec.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libvirt.spec.in b/libvirt.spec.in
index 567721f424..823f0753ae 100644
--- a/libvirt.spec.in
+++ b/libvirt.spec.in
@@ -110,7 +110,7 @@
 %endif
 
 # Ceph dropping support for 32-bit hosts
-%if 0%{fedora} >= 30
+%if 0%{?fedora} >= 30
     %ifarch %{arm} %{ix86}
         %define with_storage_rbd 0
     %endif
-- 
2.20.1

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

Re: [libvirt] [PATCH] rpm spec: don't assume %{fedora} exists as a macro

Posted by Erik Skultety 5 years, 2 months ago

On Mon, Jan 21, 2019 at 02:06:37PM +0000, Daniel P. Berrangé wrote:
> Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
> ---
>
> Pushed under the "fix the previous build breaker fix" rule :-( Sorry.

That would explain the ubuntu build failure, thanks.

Erik

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