From 735de4e393eb28e0520adee72ff08ea0b6cda9bb Mon Sep 17 00:00:00 2001 From: Kalev Lember Date: Wed, 4 Apr 2012 01:00:00 +0300 Subject: [PATCH] Revert the testsuite changes from previous commit Can't BuildRequire wine without breaking the build on arches where wine is unavailable. As such, the testsuite should only be used for local testing and not enabled in official builds. Even on secondary arches, all the builds are done from a single source RPM that gets constructed on the primary koji instance. Because of the single SRPM issue, using build time conditionals won't work for conditionally selecting BuildRequires -- the SRPM requires will be chosen depending on where the SRPM happens to get built and the same set of requires will also be used an all other arches. --- mingw-gcc.spec | 16 ++-------------- 1 file changed, 2 insertions(+), 14 deletions(-) diff --git a/mingw-gcc.spec b/mingw-gcc.spec index 393533c..3fa8ada 100644 --- a/mingw-gcc.spec +++ b/mingw-gcc.spec @@ -13,7 +13,7 @@ %global enable_libgomp 1 # Run the testsuite -%global enable_tests 1 +%global enable_tests 0 %global snapshot_date 20120322 @@ -62,9 +62,7 @@ BuildRequires: mingw64-pthreads %endif %endif %if 0%{enable_tests} -# The x86_64 build repo doesn't contain the i686 wine packages -# so we can't BR: wine here, only wine-wow :( -BuildRequires: wine-wow +BuildRequires: wine BuildRequires: autogen BuildRequires: dejagnu BuildRequires: sharutils @@ -280,10 +278,6 @@ export WINEPREFIX=/tmp/.wine_gcc_testsuite rm -rf $WINEPREFIX mkdir $WINEPREFIX -%ifarch x86_64 -export WINELOADER=/usr/bin/wine64 -%endif - # The command below will fail, but that's intentional # We only have to call a wine binary which triggers # the generation and population of a wine prefix @@ -309,7 +303,6 @@ cp %{mingw32_bindir}/pthreadGC2.dll $SYSTEM32_DIR cp build_win32/i686-w64-mingw32/libgomp/.libs/libgomp-1.dll $SYSTEM32_DIR %endif -%ifarch x86_64 SYSTEM64_DIR=$WINEPREFIX/drive_c/windows/system32 cp build_win64/x86_64-w64-mingw32/libquadmath/.libs/libquadmath-0.dll $SYSTEM64_DIR cp build_win64/x86_64-w64-mingw32/libgfortran/.libs/libgfortran-3.dll $SYSTEM64_DIR @@ -325,7 +318,6 @@ cp %{mingw64_bindir}/pthreadGC2.dll $SYSTEM64_DIR %endif cp build_win64/x86_64-w64-mingw32/libgomp/.libs/libgomp-1.dll $SYSTEM64_DIR %endif -%endif # According to Kai Tietz (of the mingw-w64 project) it's recommended # to set the environment variable GCOV_PREFIX_STRIP @@ -333,7 +325,6 @@ export GCOV_PREFIX_STRIP=1000 # Run the testsuite # Code taken from the native Fedora GCC package to collect testsuite results -%ifarch i386 i586 i686 pushd build_win32 make -k check %{?_smp_mflags} || : echo ====================TESTING WIN32========================= @@ -347,9 +338,7 @@ pushd build_win32 | uuencode testlogs-%{mingw32_target}.tar.bz2 || : rm -rf testlogs-%{mingw32_target}-%{version}-%{release} popd -%endif -%ifarch x86_64 pushd build_win64 make -k check %{?_smp_mflags} || : echo ====================TESTING WIN64========================= @@ -363,7 +352,6 @@ pushd build_win64 | uuencode testlogs-%{mingw64_target}.tar.bz2 || : rm -rf testlogs-%{mingw64_target}-%{version}-%{release} popd -%endif %endif