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

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

From: Lieven Govaerts <svnlgo_at_mobsol.be>
Date: Sun, 29 Jun 2008 22:54:33 +0200

kameshj_at_tigris.org wrote:
> Author: kameshj
> Date: Fri Jun 27 07:56:04 2008
> New Revision: 31900
>
> Log:
> Fix issue #3067, 'subtrees that don't exist at the start or end of a merge
> range shouldn't break the merge'
>
> Fixes the core problem of issue #3076: Don't try to describe invalid
> subtrees to the merge report editor. Also fixes the ancillary problem
> described in http://subversion.tigris.org/issues/show_bug.cgi?id=3067#desc34
> where subtrees with empty mergeinfo don't get their mergeinfo updated
> properly.
>
> * subversion/libsvn_client/merge.c
> (init_rangelist, push_range): New helpers.
> (prepare_subtree_ranges): New helper for filter_merged_revisions() when
> operating on subtrees.
> (filter_merged_revisions): Use new prepare_subtree_ranges() helper to
> address various issue #3067 problems. Do away with entry argument
> and instead pass a svn_client__merge_path_t representing the working copy
> target path, along with another svn_client__merge_path_t representing the
> path's parent if the path is a subtree. Cascade some additional args from
> calculate_remaining_ranges().
> (calculate_remaining_ranges): Improve doc string. Tweak signature to get
> svn_client__merge_path_t for target and its nearest parent if target is a
> subtree. Update call to filter_merged_revisions().
> (populate_remaining_ranges, do_file_merge): Update calls to
> calculate_remaining_ranges().
> (get_mergeinfo_walk_cb): Stop doing any 'preemptive' elision on subtrees.
> This leaves a local mod on the subtree and the subtree is to be deleted by
> the requested merge,the local mod will cause the subtree to be skipped.
> Just add subtrees with empty mergeinfo and let prepare_subtree_ranges()
> handle it.
> (get_mergeinfo_paths): Note in doc string that we always include children
> with empty mergeinfo.
>
This commit seems to break 4 merge tests over ra_serf only.

The error message for each of this is:

svn: Premature EOF seen from server

I don't know what specifically in your patch can cause this situation,
and I don't have a lot of time right now to look into it. If anyone else
can, great. Otherwise I'll use this as a todo reminder mail.

Lieven

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-06-29 22:54:49 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.