ivan_at_apache.org writes:
> Author: ivan
> Date: Wed Jan 23 11:55:27 2013
> New Revision: 1437354
>
> URL: http://svn.apache.org/viewvc?rev=1437354&view=rev
> Log:
> When removing old repositories before re-creating them for tests, we don't
> need to be clean about it, so just recursively remove the directory.
>
> * subversion/tests/svn_test_fs.c
> (svn_test__create_repos): Just remove the target directory, rather than
> just the repository.
>
> Modified:
> subversion/trunk/subversion/tests/svn_test_fs.c
>
> Modified: subversion/trunk/subversion/tests/svn_test_fs.c
> URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/tests/svn_test_fs.c?rev=1437354&r1=1437353&r2=1437354&view=diff
> ==============================================================================
> --- subversion/trunk/subversion/tests/svn_test_fs.c (original)
> +++ subversion/trunk/subversion/tests/svn_test_fs.c Wed Jan 23 11:55:27 2013
> @@ -213,7 +213,10 @@ svn_test__create_repos(svn_repos_t **rep
> if (apr_stat(&finfo, name, APR_FINFO_TYPE, pool) == APR_SUCCESS)
> {
> if (finfo.filetype == APR_DIR)
> - SVN_ERR(svn_repos_delete(name, pool));
> + SVN_ERR_W(svn_io_remove_dir2(name, TRUE, NULL, NULL, pool),
> + apr_psprintf(pool,
> + "cannot create repos '%s' there is already "
> + "a directory of that name", name));
> else
> return svn_error_createf(SVN_ERR_TEST_FAILED, NULL,
> "there is already a file named '%s'", name);
>
>
That was the only call to svn_repos_delete in the testsuite. We should
have a proper regression test.
--
Certified & Supported Apache Subversion Downloads:
http://www.wandisco.com/subversion/download
Received on 2013-01-23 13:43:37 CET