Re: svn commit: r1437354 - /subversion/trunk/subversion/tests/svn_test_fs.c
On Wed, Jan 23, 2013 at 4:42 PM, Philip Martin
>> 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.
>> 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));
>> 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.
Done in r1437414.
Received on 2013-01-23 15:44:08 CET
This is an archived mail posted to the Subversion Dev