validate:launcher: Handle not redirecting valgrind output
authorThibault Saunier <thibault.saunier@osg.samsung.com>
Tue, 30 May 2017 20:15:19 +0000 (16:15 -0400)
committerThibault Saunier <thibault.saunier@osg.samsung.com>
Thu, 1 Jun 2017 20:57:50 +0000 (16:57 -0400)
validate/launcher/baseclasses.py

index 297d14e..353591a 100644 (file)
@@ -381,8 +381,6 @@ class Test(Loggable):
                         "--args"] + self.command
 
     def use_valgrind(self):
-        vglogsfile = self.logfile + '.valgrind'
-        self.extra_logfiles.append(vglogsfile)
 
         vg_args = []
 
@@ -394,11 +392,15 @@ class Test(Loggable):
                 #       and all false positives should be added to suppression files.
                 ('errors-for-leak-kinds', 'definite'),
                 ('num-callers', '20'),
-                ('log-file', vglogsfile),
                 ('error-exitcode', str(VALGRIND_ERROR_CODE)),
                 ('gen-suppressions', 'all')]:
             vg_args.append("--%s=%s" % (o, v))
 
+        if not self.options.redirect_logs:
+            vglogsfile = self.logfile + '.valgrind'
+            self.extra_logfiles.append(vglogsfile)
+            vg_args.append("--%s=%s" % ('log-file', vglogsfile))
+
         for supp in self.get_valgrind_suppressions():
             vg_args.append("--suppressions=%s" % supp)