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

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

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

[[[
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

Received on 2009-11-10 18:22:50 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.