Re: Issue 1954

Philipp Marek wrote:
> And in subversion/libsvn_fs/fs-loader.c I see this function:
> svn_error_t *
> svn_fs_make_file(svn_fs_root_t *root, const char *path, apr_pool_t *pool)
> {
> SVN_ERR(svn_path_check_valid(path, pool));
> return root->vtable->make_file(root, path, pool);
> }
> which makes me believe that this should happen for the repository, too.

Good lord. I must have been sleeping when somebody snuck that rubbish into
the codebase. [Eyes kfogel suspiciously.] Sorry for inadvertantly
misleading you, Philipp. FWIW, I strongly disagree with this change to
libsvn_fs. If someone wanted to protect the repository, they should have
done so by introducing a wrapper in libsvn_repos.

-- 
C. Michael Pilato <cmpilato@collab.net>
CollabNet   <>   www.collab.net   <>   Distributed Development On Demand