subversion-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Evgeny Kotkov <evgeny.kot...@visualsvn.com>
Subject 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
Date Tue, 10 May 2016 15:26:00 GMT
Stefan Fuhrmann <stefan2@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

Mime
View raw message