From daced8041d5564de280591822af9c1173f659c94 Mon Sep 17 00:00:00 2001 From: Daniel Stenberg Date: Fri, 23 Apr 2004 08:38:43 +0000 Subject: [PATCH] No longer uses the valgrind option '--logfile-fd', we use the --logfile option instead (even though it appends the pid to the file name, making it harder to figure out its name to parse it after a test has run). Also made sure we only use valgrind for the actual test command command lines, not when for example running curl to detect if there are any already running servers are present etc. --- tests/runtests.pl | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/tests/runtests.pl b/tests/runtests.pl index 8de3c0690..2d9a97ed9 100755 --- a/tests/runtests.pl +++ b/tests/runtests.pl @@ -1024,7 +1024,7 @@ sub singletest { } if($valgrind) { - $CMDLINE = "exec 3>log/valgrind$testnum && $CMDLINE"; + $CMDLINE = "valgrind --leak-check=yes --logfile=log/valgrind$testnum -q $CMDLINE"; } $CMDLINE .= "$cmdargs >>$STDOUT 2>>$STDERR"; @@ -1476,9 +1476,6 @@ if($valgrind) { #print "Valgrind failure, disable it\n"; undef $valgrind; } - else { - $CURL="valgrind --leak-check=yes --logfile-fd=3 -q $CURL"; - } } #######################################################################