Thanks for the confirmation, is there an issue number I can use to
track this?
Tom
On Oct 16, 2008, at 2:09 AM, Bert Huijben wrote:
> Hi Tom,
>
> The fix for this issue is in trunk. I noticed the same problem while
> building SharpSvn and using it from AnkhSVN 2.0, but this was just
> after 1.5 was branched.
>
> This method of using the path was unreliable before 1.5 as well L
> It all depends on your current directory related to the commit
> location. The used base directory is the common ancestor of the
> commit locations and your current working directory.
>
> In trunk/1.6 you will always receive an absolute path and optionally
> the prefix in a separate variable of svn_wc_notify_t. (The UI than
> has the option of extracting the prefix or leaving it to always real
> absolute or relative paths).
>
> The lock functions have the same problem in 1.5 and where fixed in
> the same way for 1.6. This change can’t be ported back to 1.5 as it
> changes the svn_wc_notify_t structure.
>
> Bert
>
> From: Tom Naughton [mailto:naughtont_at_gmail.com]
> Sent: Wednesday, October 15, 2008 11:42 PM
> To: dev_at_subversion.tigris.org
> Subject: absolute paths in svn_wc_notify_t
>
> I have an application that uses version 1.5.2 of the client library
> APIs. I've set up the notify_func2 in my svn_client_ctx_t to point
> to a callback.
>
> Before 1.5, I relied on 'path' in svn_wc_notify_t to point to an
> absolute path. Now for commit operations (and only commit
> operations), I'm getting partial paths that have had some prefix
> removed. Is there some way to determine what this prefix will be or
> even better, opt to receive absolute paths instead? According to the
> comments in svn_wc.h this path is "either absolute or relative to
> the current working directory". But I haven't found that to be the
> case. In some cases the prefix that is removed is just "/".
>
> Stepping through the subversion code I've been able to derive an
> algorithm for determining the prefix that will be removed:
>
> // Determine prefix stripped from the commit notify messages
> const char *display_dir = "";
> const char *base_dir = nil;
> svn_depth_t depth = svn_depth_infinity;
> apr_array_header_t *rel_targets;
> err = svn_path_condense_targets(&base_dir, &rel_targets,
> commit_targets, depth == svn_depth_infinity, pool);
> err = svn_path_get_absolute(&display_dir,display_dir, pool);
> display_dir = svn_path_get_longest_ancestor(display_dir,
> base_dir, pool);
>
> If the current directory is something like "/Objects/Debug" and all
> of the paths in commit_targets are outside of /Objects,
> svn_path_get_longest_ancestor returns "/", and that's what I'm
> seeing in my unit tests.
>
> Is there a better way to figure out this prefix?
>
> Tom
>
Received on 2008-10-17 04:50:16 CEST