[vdagent-win PATCH] RFC spec: run tests during RPM build if possible

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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 <fziglio@xxxxxxxxxx>
---
 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@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]