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

Re: [PATCH] Replace svn_path_join() in libsvn_client/merge.c

From: Daniel Näslund <daniel_at_longitudo.com>
Date: Tue, 10 Nov 2009 18:37:18 +0100

Forgot! Make check passed.

On Tue, Nov 10, 2009 at 06:22:24PM +0100, Daniel Näslund wrote:
> [[[
> Remove use of the deprecated svn_path_join().
>
> * subversion/libsvn_client/merge.c
> (record_mergeinfo_for_dir_merge, find_unmerged_mergeinfo):
> Replace svn_path_join() with svn_uri_join(). All "base" parameters
> are repos relative paths.
> ]]]
>
> When joining source_repos_rel_path and target_repos_rel_path a leading
> '/' is stripped from the second component. svn_dirent_uri.h says that we
> should use svn_uri_basename() or svn_uri_strip_ancestor() when joining a
> dirent to an uri. But basename() only gives us the last part and
> strip_ancestor() depends on the two paths both beeing either relative or
> absolute. Just stripping the '/' seems easier to understand to me.
>
> /Daniel

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=2416240
Received on 2009-11-10 18:37:31 CET

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.