db/pgbackrest
1
0
Fork 0
mirror of https://github.com/pgbackrest/pgbackrest.git synced 2025-07-01 00:25:06 +02:00
Code Issues Releases Activity

Don't perform valgrind when requested.

Browse Source 
The --no-valgrind flag was not being honored.  It's not clear if this flag ever worked, but it does now.
This commit is contained in:
David Steele
2018-09-13 19:12:40 -04:00
parent fd14ceb399
commit aeb1fa3dfb
2 changed files with 6 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
doc/xml/release.xml
View File

@ -105,6 +105,10 @@
<p>Make Valgrind return an error even when a non-fatal issue is detected. Update some minor issues discovered in the tests as a result.</p>
</release-item>
<release-item>
<p>Don't perform valgrind when requested.</p>
</release-item>
<release-item>
<p>Fix typo in unit test error messages, EXECTED => EXPECTED.</p>
</release-item>

4
test/lib/pgBackRestTest/Common/JobTest.pm
View File

@ -236,8 +236,8 @@ sub run
$strCommand =
'docker exec -i -u ' . TEST_USER . " ${strImage} bash -l -c '" .
"cd $self->{strGCovPath} && " .
"make -s 2>&1 && " .
($self->{oTest}->{&TEST_VM} ne VM_CO6 ?
"make -s 2>&1 &&" .
($self->{oTest}->{&TEST_VM} ne VM_CO6 && $self->{bValgrindUnit}?
" valgrind -q --gen-suppressions=all --suppressions=$self->{strBackRestBase}/test/src/valgrind.suppress" .
" --leak-check=full --leak-resolution=high --error-exitcode=25" : '') .
" ./test 2>&1'";