David Glasser wrote on Fri, 23 May 2008 at 10:00 -0700:
> On Fri, May 23, 2008 at 4:53 AM, Daniel Shahaf <d.s_at_daniel.shahaf.co.il> wrote:
> > David Glasser wrote on Fri, 23 May 2008 at 00:57 -0700:
> >> On Thu, May 22, 2008 at 11:43 PM, Daniel Shahaf <d.s_at_daniel.shahaf.co.il> wrote:
> >> > glasser_at_tigris.org wrote on Thu, 22 May 2008 at 17:00 -0700:
> >> >> @@ -30,8 +30,8 @@ class TestHarness:
> >> >> def __init__(self, abs_srcdir, abs_builddir, logfile,
> >> >> base_url=None, fs_type=None, http_library=None,
> >> >> server_minor_version=None, verbose=None,
> >> >> - cleanup=None, enable_sasl=None, parallel=None, list_tests=None,
> >> >> - svn_bin=None):
> >> >> + cleanup=None, enable_sasl=None, parallel=None, config_file=None,
> >> > ^^^^^^^^^^^
> >> >> + list_tests=None, svn_bin=None):
> >> >> '''Construct a TestHarness instance.
> >> >>
> >> >
> >> > Adding the new argument broke the Windows tests, because win-tests.py
> >> > calls TestHarness this way:
> >> >
> >> > import run_tests
> >> > th = run_tests.TestHarness(abs_srcdir, abs_builddir,
> >> > os.path.join(abs_builddir, log),
> >> > base_url, fs_type, http_library,
> >> > server_minor_version, 1, cleanup,
> >> > enable_sasl, parallel, list_tests,
> >> > ^^^^^^^^^^
> >> > svn_bin)
> >> >
> >> > Not sure how to fix it; should config_file be supported on Windows?
> >>
> >> Honestly I'd suggest rewriting that function to not take a million arguments.
> >>
> >
> > No opinion here, +1 to whatever you want to do to make it work again.
>
> I suggest that whoever maintains win-tests.py adds a None or
> something. I don't edit files that don't run on my platform if I can
> avoid it...
>
Done in r31392. I haven't checked that the config file is actually used
properly, though, just that the tests run and the argument is parsed.
If "whoever maintains win-tests.py" can, please look at it and make sure
I didn't break anything.
Thanks,
Daniel
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-05-23 19:37:16 CEST