[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [PATCH] svn_client_commit[ 23] broken on trunk

From: Karl Fogel <kfogel_at_red-bean.com>
Date: 2007-10-22 03:22:35 CEST

John Peacock <john.peacock@havurah-software.org> writes:
> I am resending this with a new subject line, because I don't think it was
> obvious enough before. Even though the breakage showed up in the
> Perl bindings tests, it actually affects any code that uses any of
> the svn_client_commit calls except for *_commit4.

+1 here, and I think SVN_DEPTH_INFINITY_OR_FILES() is the right way to
go. Sorry for forgetting to update this in r26241.

-Karl

> [[[
> New depth argument for svn_client_commit4 wasn't promoted to
> svn_client_commit3, thus breaking all older commit API calls.
>
> * subversion/libsvn_client/commit.c
> (svn_client_commit3): synthesize depth argument from old recurse argument
> ]]]
>
> Index: subversion/libsvn_client/commit.c
> ===================================================================
> --- subversion/libsvn_client/commit.c (revision 27297)
> +++ subversion/libsvn_client/commit.c (working copy)
> @@ -1770,7 +1770,9 @@
> svn_client_ctx_t *ctx,
> apr_pool_t *pool)
> {
> - return svn_client_commit4(commit_info_p, targets, recurse, keep_locks,
> + svn_depth_t depth = SVN_DEPTH_INFINITY_OR_FILES(recurse);
> +
> + return svn_client_commit4(commit_info_p, targets, depth, keep_locks,
> FALSE, NULL, ctx, pool);
> }

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Mon Oct 22 03:22:51 2007

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.