Merge pull request #2050 from minrk/xunit-quotes

only add quotes around xunit-file on Windows

because it uses os.system. We can trust Popen to handle spaces in the string on other platforms.
Min RK 14 years ago
commit e57052b8d8

@ -352,7 +352,13 @@ class IPTester(object):
raise ValueError("Section not found", self.params)
if '--with-xunit' in self.call_args:
self.call_args.append('--xunit-file="%s"' % path.abspath(sect+'.xunit.xml'))
self.call_args.append('--xunit-file')
# FIXME: when Windows uses subprocess.call, these extra quotes are unnecessary:
xunit_file = path.abspath(sect+'.xunit.xml')
if sys.platform == 'win32':
xunit_file = '"%s"' % xunit_file
self.call_args.append(xunit_file)
if '--with-xml-coverage' in self.call_args:
self.coverage_xml = path.abspath(sect+".coverage.xml")

Loading…
Cancel
Save