On Thu, Apr 26, 2012 at 4:07 PM, Greg Stein <gstein_at_gmail.com> wrote:
> On Thu, Apr 26, 2012 at 17:04, Â <hwright_at_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/
Received on 2012-04-26 23:10:02 CEST