[PATCH] rpm: Require dmidecode on more architectures

Andrea Bolognani posted 1 patch 2 weeks ago
libvirt.spec.in | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[PATCH] rpm: Require dmidecode on more architectures
Posted by Andrea Bolognani 2 weeks ago
It's not only used on x86_64 these days. See virSysinfoRead().

Technically we should include loongarch64 in the list as well,
but Fedora hasn't been bootstrapped on the architecture yet,
and when the time comes several more changes are going to be
necessary anyway.

Signed-off-by: Andrea Bolognani <abologna@redhat.com>
---
 libvirt.spec.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libvirt.spec.in b/libvirt.spec.in
index c3dc6e21c0..f26de04553 100644
--- a/libvirt.spec.in
+++ b/libvirt.spec.in
@@ -19,7 +19,7 @@
 %define arches_x86              %{ix86} x86_64
 
 %define arches_systemtap_64bit  %{arches_64bit}
-%define arches_dmidecode        %{arches_x86}
+%define arches_dmidecode        %{arches_x86} aarch64 riscv64
 %define arches_xen              %{arches_x86} aarch64
 %if 0%{?fedora}
     %define arches_xen          x86_64 aarch64
-- 
2.46.2
Re: [PATCH] rpm: Require dmidecode on more architectures
Posted by Ján Tomko 2 weeks ago
On a Friday in 2024, Andrea Bolognani wrote:
>It's not only used on x86_64 these days. See virSysinfoRead().
>
>Technically we should include loongarch64 in the list as well,
>but Fedora hasn't been bootstrapped on the architecture yet,
>and when the time comes several more changes are going to be
>necessary anyway.
>
>Signed-off-by: Andrea Bolognani <abologna@redhat.com>
>---
> libvirt.spec.in | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>

Reviewed-by: Ján Tomko <jtomko@redhat.com>

Jano