On Wed, Jan 23, 2013 at 4:42 PM, Philip Martin <philip.mar...@wandisco.com> wrote: >> 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. >> [.. ] >> 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. > Done in r1437414.
-- Ivan Zhakov
