Mailing-List: contact dev-help@subversion.apache.org; run by ezmlm
Precedence: bulk
MIME-Version: 1.0
In-Reply-To: <20160510135300.527453A0216@svn01-us-west.apache.org>
References: <20160510135300.527453A0216@svn01-us-west.apache.org>
From: Evgeny Kotkov <evgeny.kotkov@visualsvn.com>
Date: Tue, 10 May 2016 18:26:00 +0300
Message-ID: <CAP_GPNgW8WrGvEqdZchiqYj1mJ8UgNhCwTZYUViKWM2SS-rw_g@mail.gmail.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
To: Stefan Fuhrmann <stefan2@apache.org>
Cc: Subversion Development <dev@subversion.apache.org>
Content-Type: text/plain; charset=UTF-8
archived-at: Tue, 10 May 2016 15:26:33 -0000

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