On Wed, 2011-03-02, Noorul Islam K M wrote:
> Please find attached patch for issue 3826. All tests pass using 'make
> check'
[...]
> Index: subversion/svn/diff-cmd.c
> ===================================================================
> --- subversion/svn/diff-cmd.c (revision 1076214)
> +++ subversion/svn/diff-cmd.c (working copy)
> @@ -324,8 +324,11 @@
> return svn_error_createf(SVN_ERR_CL_ARG_PARSING_ERROR, NULL,
> _("Path '%s' not relative to base
> URLs"),
> path);
> + if (! svn_dirent_is_absolute(path))
> + path = svn_relpath_canonicalize(path, iterpool);
> + else
> + path = svn_dirent_canonicalize(path, iterpool);
>
> - path = svn_relpath_canonicalize(path, iterpool);
> if (svn_path_is_url(old_target))
> target1 = svn_path_url_add_component2(old_target, path,
> iterpool);
This doesn't look quite right. Here, "path" must be a relpath ...
> else
target1 = svn_dirent_join(old_target, path, iterpool);
... even though here it can be either a relpath or an absolute dirent.
- Julian
