Currently to run checks we need Wine for both 32 and 64 bit.
Some distros (like RHEL 7) don't seem to allow installing
both 32 and 64 bit versions so turn on these checks only
on recent Fedora versions.

Signed-off-by: Frediano Ziglio <fzig...@redhat.com>
---
 mingw-spice-vdagent.spec.in | 13 +++++++++++++
 1 file changed, 13 insertions(+)

This patch apply on top of PNG patches.
Is there a better way to test if is possible to installe both
Wine 32 and 64? Can we say try to install both but if it can
install only one (probably x64) use only this?

diff --git a/mingw-spice-vdagent.spec.in b/mingw-spice-vdagent.spec.in
index f874e66..6e62b10 100644
--- a/mingw-spice-vdagent.spec.in
+++ b/mingw-spice-vdagent.spec.in
@@ -2,6 +2,12 @@
 
 #define _version_suffix -e198
 
+%if "%{_build_arch}" == "x86_64" && (0%{?fedora} && 0%{?fedora} >= 24)
+    %define can_do_check 1
+%else
+    %define can_do_check 0
+%endif
+
 Name:           mingw-spice-vdagent
 Version:        @VERSION@
 Release:        1%{?dist}%{?extra_release}
@@ -22,6 +28,9 @@ BuildRequires:  mingw64-zlib-static
 BuildRequires:  mingw32-winpthreads-static
 BuildRequires:  mingw64-winpthreads-static
 BuildRequires:  pkgconfig
+%if %{can_do_check}
+BuildRequires:  ImageMagick wine-core(x86-32) wine-core(x86-64)
+%endif
 
 BuildArch:      noarch
 
@@ -73,6 +82,10 @@ Features:
 %mingw_configure --enable-debug
 %mingw_make %{?_smp_mflags} V=1
 
+%if %{can_do_check}
+%check
+%mingw_make check
+%endif
 
 %install
 %mingw_make_install DESTDIR=$RPM_BUILD_ROOT
-- 
2.13.3

_______________________________________________
Spice-devel mailing list
Spice-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/spice-devel

Reply via email to