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

Re: svn commit: r32292 - branches/issue-2843-dev/subversion/libsvn_wc

From: Karl Fogel <kfogel_at_red-bean.com>
Date: Sun, 17 Aug 2008 18:56:50 -0400

firemeteor_at_tigris.org writes:
> * subversion/libsvn_wc/update_editor.c
> (complete_directory): Use the full target path when retrieving adm_access.

Just to let you know, I reviewed this (resulting in r32514, and in
http://subversion.tigris.org/servlets/ReadMsg?list=dev&msgNo=141928).

-Karl

> --- branches/issue-2843-dev/subversion/libsvn_wc/update_editor.c
> +++ branches/issue-2843-dev/subversion/libsvn_wc/update_editor.c
> @@ -495,12 +495,14 @@ complete_directory(struct edit_baton *eb
> entry = apr_hash_get(entries, eb->target, APR_HASH_KEY_STRING);
> if (entry && entry->depth == svn_depth_exclude)
> {
> + char * full_target;
> entry->depth = svn_depth_infinity;
> SVN_ERR(svn_wc__entries_write(entries, adm_access, pool));
> /* There is a small chance that the target is gone in the
> repository. We'd better get rid of the exclude flag now. */
> + full_target = svn_path_join(eb->anchor, eb->target, pool);
> SVN_ERR(svn_wc__adm_retrieve_internal
> - (&target_access, eb->adm_access, eb->target, pool));
> + (&target_access, eb->adm_access, full_target, pool));
> if (!target_access)
> {
> int log_number = 0;
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: svn-unsubscribe_at_subversion.tigris.org
> For additional commands, e-mail: svn-help_at_subversion.tigris.org

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-08-18 00:57:07 CEST

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.