From: Daniel Stenberg <daniel@haxx.se>
Date: Fri, 23 Apr 2004 08:38:43 +0000 (+0000)
Subject: No longer uses the valgrind option '--logfile-fd', we use the --logfile
X-Git-Tag: curl-7_11_2~16
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=daced8041d5564de280591822af9c1173f659c94;p=curl

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.
---

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