better explain the conditional BR on valgrind

This commit is contained in:
Kamil Dudka 2015-08-27 16:06:46 +02:00
parent bfaeb9dd10
commit 8b89049610
1 changed files with 6 additions and 1 deletions

View File

@ -52,7 +52,12 @@ BuildRequires: perl(Time::HiRes)
BuildRequires: perl(warnings)
BuildRequires: perl(vars)
# require valgrind to boost test coverage on i386 and x86_64
# The test-suite runs automatically trough valgrind if valgrind is available
# on the system. By not installing valgrind into mock's chroot, we disable
# this feature for production builds on architectures where valgrind is known
# to be less reliable, in order to avoid unnecessary build failures (see RHBZ
# #810992, #816175, and #886891). Nevertheless developers are free to install
# valgrind manually to improve test coverage on any architecture.
%ifarch %{ix86} x86_64
BuildRequires: valgrind
%endif