[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r1743183 - in /subversion/trunk/subversion/tests/cmdline: svnrdump_tests.py svntest/actions.py svntest/main.py svntest/sandbox.py svntest/verify.py svntest/wc.py

From: Stefan Fuhrmann <stefan2_at_apache.org>
Date: Fri, 13 May 2016 05:33:29 +0200

On 10.05.2016 17:26, Evgeny Kotkov wrote:
> Stefan Fuhrmann <stefan2_at_apache.org> writes:
>
>> * subversion/tests/cmdline/svntest/actions.py
>> (run_and_verify_dump): If we accept any output, "None" is the better
>> option as it works with both Python versions.
> [...]
>
>> exit_code, output, errput = run_and_verify_svnadmin(
>> - verify.AnyOutput, [],
>> + None, [],
>> 'dump', '--quiet', repo_dir, *args)
> None and AnyOutput aren't the same, since AnyOutput guards against
> unexpected empty output.
>
> Are we okay with this behavior change for compatibility purposes?
The behavioural change was not intended.
'any' is not the most precise word in the English language ...

Turns out that the other patches already achieved
whatever this change was supposed to do and the
original code got re-instantiated in r1743257.

-- Stefan^2.
Received on 2016-05-13 05:33:35 CEST

This is an archived mail posted to the Subversion Dev mailing list.