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

Re: svn commit: r1785734 - /subversion/trunk/subversion/tests/libsvn_fs/fs-test.c

From: Daniel Shahaf <d.s_at_daniel.shahaf.name>
Date: Mon, 6 Mar 2017 20:53:13 +0000

stefan2_at_apache.org wrote on Mon, Mar 06, 2017 at 19:05:45 -0000:
> +static svn_error_t *
> +test_rep_sharing_strict_content_check(const svn_test_opts_t *opts,
> + apr_pool_t *pool)
> +{
> + /* Bail (with success) on known-untestable scenarios */
> + if (strcmp(opts->fs_type, SVN_FS_TYPE_BDB) == 0)
> + return svn_error_create(SVN_ERR_TEST_SKIPPED, NULL,
> + "this will not test BDB repositories");

Minor point: the log message^W^W skip message should answer "Why", not
"What". Suggestion:

    "bdb does not have a rep-cache"
Received on 2017-03-06 21:58:02 CET

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.