[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: Evgeny Kotkov <evgeny.kotkov_at_visualsvn.com>
Date: Tue, 10 May 2016 18:26:00 +0300

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?

Regards,
Evgeny Kotkov
Received on 2016-05-10 17:26:33 CEST

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

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.