You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by ko...@apache.org on 2015/10/02 17:39:01 UTC
svn commit: r1706432 - /subversion/trunk/build/run_tests.py
Author: kotkov
Date: Fri Oct 2 15:39:01 2015
New Revision: 1706432
URL: http://svn.apache.org/viewvc?rev=1706432&view=rev
Log:
Following up on r1700215, unbreak the test runner's --bin option.
For instance, win-tests.py --bin=some\path used to error out like this:
Traceback (most recent call last):
File "win-tests.py", line 1025, in <module>
failed = th.run(tests_to_run)
File "C:\Projects\Subversion\trunk\build\run_tests.py", line 319, in run
self._init_py_tests(list(py_basedir)[0])
File "C:\Projects\Subversion\trunk\build\run_tests.py", line 219,
in _init_py_tests
cmdline.append('--bin=%s', self.opts.svn_bin)
TypeError: append() takes exactly one argument (2 given)
* build/run_tests.py
(TestHarness._init_py_tests): Use % operator to format the --bin path.
Modified:
subversion/trunk/build/run_tests.py
Modified: subversion/trunk/build/run_tests.py
URL: http://svn.apache.org/viewvc/subversion/trunk/build/run_tests.py?rev=1706432&r1=1706431&r2=1706432&view=diff
==============================================================================
--- subversion/trunk/build/run_tests.py (original)
+++ subversion/trunk/build/run_tests.py Fri Oct 2 15:39:01 2015
@@ -214,7 +214,7 @@ class TestHarness:
else:
cmdline.append('--parallel-instances=%d' % self.opts.parallel)
if self.opts.svn_bin is not None:
- cmdline.append('--bin=%s', self.opts.svn_bin)
+ cmdline.append('--bin=%s' % self.opts.svn_bin)
if self.opts.url is not None:
cmdline.append('--url=%s' % self.opts.url)
if self.opts.fs_type is not None: