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.
This commit is contained in:
Daniel Stenberg 2004-04-23 08:38:43 +00:00
parent b053ae6a65
commit daced8041d
1 changed files with 1 additions and 4 deletions

View File

@ -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";
}
}
#######################################################################