On Thu, Apr 26, 2012 at 4:07 PM, Greg Stein <gst...@gmail.com> wrote: > On Thu, Apr 26, 2012 at 17:04, <hwri...@apache.org> wrote: >>... >> +++ subversion/branches/ev2-export/subversion/libsvn_client/add.c Thu Apr 26 >> 21:04:42 2012 >> @@ -954,16 +954,18 @@ mkdir_urls(const apr_array_header_t *url >> base_relpath = svn_uri_skip_ancestor(repos_root, common, pool); >> >> /* Fetch RA commit editor */ >> - SVN_ERR(svn_ra__register_editor_shim_callbacks(ra_session, >> - svn_client__get_shim_callbacks(ctx->wc_ctx, NULL, >> - pool))); >> - SVN_ERR(svn_ra_get_commit_editor4(ra_session, &editor, >> - commit_revprops, >> - commit_callback, >> - commit_baton, >> - NULL, TRUE, /* No lock tokens */ >> - ctx->cancel_func, ctx->cancel_baton, >> - pool, pool)); >> + SVN_ERR(svn_ra__get_commit_ev2(&editor, ra_session, >> + commit_revprops, >> + commit_callback, >> + commit_baton, >> + NULL, TRUE, /* No lock tokens */ >> + svn_client__ra_provide_base, >> + svn_client__ra_provide_props, >> + svn_client__ra_get_copysrc_kind, >> + svn_client__ra_make_cb_baton(ctx->wc_ctx, >> + NULL, pool), >> + ctx->cancel_func, ctx->cancel_baton, >> + pool, pool)); > > The callbacks need the RELPATH_MAP. If you don't have one, then just > pass NULL for all callbacks and the baton. That tells RA "I don't have > any local information for you."
Fair enough. The tests didn't assert, so I figured the above was valid. Perhaps add an assert? -Hyrum -- uberSVN: Apache Subversion Made Easy http://www.uberSVN.com/
