Skip to content

Commit

Permalink
mesontest: Use test setup name in logfiles
Browse files Browse the repository at this point in the history
When using a setup, use the setup name as the namebase for the logfile
instead of the wrapper. The wrapper may not be set, or it may be shared
between test setups.

Also don't try to use the wrapper if it's an empty list.

Closes mesonbuild#1371
  • Loading branch information
nirbheek committed Feb 19, 2017
1 parent d1bc5c3 commit 5bf4338
Showing 1 changed file with 9 additions and 6 deletions.
15 changes: 9 additions & 6 deletions mesontest.py
Original file line number Diff line number Diff line change
Expand Up @@ -415,15 +415,18 @@ def open_log_files(self):
if not self.options.logbase or self.options.verbose:
return None, None, None, None

namebase = None
logfile_base = os.path.join(self.options.wd, 'meson-logs', self.options.logbase)

if self.options.wrapper is None:
logfilename = logfile_base + '.txt'
jsonlogfilename = logfile_base + '.json'
else:
if self.options.wrapper:
namebase = os.path.split(self.get_wrapper()[0])[1]
logfilename = logfile_base + '-' + namebase.replace(' ', '_') + '.txt'
jsonlogfilename = logfile_base + '-' + namebase.replace(' ', '_') + '.json'
elif self.options.setup:
namebase = self.options.setup

if namebase:
logfile_base += '-' + namebase.replace(' ', '_')
logfilename = logfile_base + '.txt'
jsonlogfilename = logfile_base + '.json'

jsonlogfile = open(jsonlogfilename, 'w')
logfile = open(logfilename, 'w')
Expand Down

0 comments on commit 5bf4338

Please sign in to comment.