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

Re: svn commit: r26339 - in trunk/subversion: libsvn_client libsvn_ra_neon tests/cmdline

From: Daniel Rall <dlr_at_finemaltcoding.com>
Date: 2007-08-27 21:49:36 CEST

On Aug 27, 2007, at 12:12 PM, kameshj@tigris.org wrote:
...
> Fix issue 2889 - mergeinfo REPORT requests are ignoring peg revisions.
>
> Following bug is exposed by the fix to #2889.
> a)To 'get_wc_or_repos_mergeinfo' we pass ra_session which is not
> related
> to Working copy target on which we merge.
>
> * subversion/libsvn_ra_neon/mergeinfo.c
> (svn_ra_neon__get_mergeinfo): Run the merge-info REPORT request on a
> baseline url.
> * subversion/libsvn_client/merge.c
> (do_merge, do_single_file_merge): reparent the ra_session
> corresponding to
> working copy target before calling 'get_wc_or_repos_mergeinfo'.
> * subversion/tests/cmdline/merge_tests.py
> (merge_catches_nonexistent_target): Fix the wrong comment about
> revision
> number.
> (test_list): Remove XFail marker from
> 'merge_fails_if_subtree_is_deleted_on_src'.
...
> --- trunk/subversion/libsvn_client/merge.c (original)
> +++ trunk/subversion/libsvn_client/merge.c Mon Aug 27 12:12:22 2007
> @@ -2100,6 +2100,7 @@
> svn_boolean_t is_root_of_noop_merge = FALSE;
> apr_size_t target_count, merge_target_count;
> apr_pool_t *subpool;
> + svn_boolean_t is_same_repos = FALSE;
>
> ENSURE_VALID_REVISION_KINDS(initial_revision1->kind,
> initial_revision2->kind);
> @@ -2141,13 +2142,26 @@
>
> if (notify_b.same_urls)
> {
> + /* ### TODO: Should we use from_same_repos? I don't like its
> behaviour
> + ### for dry-run case. */
> + const char *src_repos_root;
> + SVN_ERR(svn_ra_get_repos_root(ra_session, &src_repos_root,
> pool));
> + if (svn_path_is_ancestor(src_repos_root, entry->url))
> + is_same_repos = TRUE;
> + }
> + if (notify_b.same_urls && is_same_repos)
> + {
> apr_array_header_t *requested_rangelist;
>
> + /* Reparent ra_session to WC target url. */
> + svn_ra_reparent(ra_session, entry->url, pool);
> SVN_ERR(get_wc_or_repos_mergeinfo(&target_mergeinfo, entry,
> &indirect, FALSE,
> svn_mergeinfo_inherited,
> ra_session,
> target_wcpath, adm_access,
> ctx, pool));
> + /* Reparent ra_session back to initial_URL1. */
> + svn_ra_reparent(ra_session, initial_URL1, pool);
>
> is_rollback = (merge_type == merge_type_rollback);
> SVN_ERR(svn_client__path_relative_to_root(&rel_path,
> initial_URL1, NULL,
...

Kamesh, can you say more about what's objectionable about
from_same_repos? Could is_same_repos and from_same_repos be merged
into a single variable? If not, it would be good to rename
is_same_repos to something which better differentiates itself from
from_same_repos.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Tue Aug 28 03:21:59 2007

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