On Mon, Mar 06, 2017 at 05:22:14PM +0100, Stefan Sperling wrote:
> On Mon, Mar 06, 2017 at 04:06:44PM +0000, Daniel Shahaf wrote:
> > I can reproduce this using the FS API only (without the contrib/ script):
> 
> Hmm, looks like the contrib script is using the API wrong.
> Check out what 'svnadmin unlock' does:
> [[[
>   /* Open the repos/FS, and associate an access context containing
>      USERNAME. */
>   SVN_ERR(open_repos(&repos, opt_state->repository_path, opt_state, pool));
>   fs = svn_repos_fs(repos);
>   SVN_ERR(svn_fs_create_access(&access, username, pool));
>   SVN_ERR(svn_fs_set_access(fs, access));
> 
>   SVN_ERR(svn_utf_cstring_to_utf8(&lock_path_utf8, lock_path, pool));
>   if (opt_state->bypass_hooks)
>     SVN_ERR(svn_fs_unlock(fs, lock_path_utf8, lock_token,
>                           FALSE, pool));
>   else
>     SVN_ERR(svn_repos_fs_unlock(repos, lock_path_utf8, lock_token,
>                                 FALSE, pool));
> ]]]
> 
> The script should probably be doing something like this, too.

This problem with 'svnadmin load' might also be caused by wrong use
of the API: https://issues.apache.org/jira/browse/SVN-4030
I have not checked the details, though. But if anybody is fixing the script,
consider taking a look at issue #4030 as well while the context is fresh.

Reply via email to