On Wed, Jun 16, 2010 at 11:21 AM, Philip Martin
<philip.mar...@wandisco.com> wrote:
> hwri...@apache.org writes:
>
>> Author: hwright
>> Date: Tue Jun 15 11:35:36 2010
>> New Revision: 954830
>
>> - if verbose_mode and quiet_mode:
>> - sys.stderr.write("ERROR: 'verbose' and 'quiet' are incompatible\n")
>> - sys.exit(1)
>> + if not appended:
>> + parser.error("invalid test number, range of numbers, " +
>> + "or function '%s'\n" % arg)
>
> In the past I could run './basic_tests.py list' now I get
>
> ../../../../src/subversion/tests/cmdline/basic_tests.py list
> Traceback (most recent call last):
> File "../../../../src/subversion/tests/cmdline/basic_tests.py", line 2571,
> in <module>
> svntest.main.run_tests(test_list)
> File "/home/pm/sw/subversion/src/subversion/tests/cmdline/svntest/main.py",
> line 1439, in run_tests
> parser.error("invalid test number, range of numbers, " +
> NameError: global name 'parser' is not defined
Using './basic_tests.py list' has been deprecated for a very long
time, replaced by './basic_tests.py --list'. My rewrite of the
options parsing removed the hack that supported the non-optioned
version of '--list'.
The error message could be a bit more friendly, thought, and that's
what I did in r955354.
-Hyrum
