C. Michael Pilato wrote on Wed, 23 Jun 2010 at 22:43 -0000: > Daniel Shahaf wrote: > >> - SVN_ERR(svn_ra_svn_write_cmd(conn, pool, "get-locks", "c", path)); > >> + /* Figure out the repository abspath from PATH. */ > >> + abs_path = svn_path_url_add_component2(sess->url, path, pool); > >> + SVN_ERR(svn_ra_get_path_relative_to_root(session, &abs_path, > >> + abs_path, pool)); > > > > I think this change means that, in build.conf, libsvn_ra should have > > been added as a dependency to [libsvn_ra_svn]. (This patch added it only > > to [svnserve].) > > > > Unless objections, I'll make this change (while also committing the > > ra_svn protocol bits noted on IRC and in the issue). > > Fine with me. (The change I made was sufficient to fix the problem I was > seeing in my build.) >
Looking further, the patch added svn_ra_get_path_relative_to_root() to all network-based RA layers. However, when I try to add libsvn_ra in build.conf as suggested above, I just get errors from configure/make about circular dependencies :-( I'm not sure what's going on here. But if it breaks in the future, hopefully this thread is going to be useful...
