Re: [PATCH] Fix a regression from 1.4 in trunk and branch

Of interest (to me, at least) is how we managed to get ahold of the
non-canonicalized path that we currently write to the entries file. I mean,
the first thing our command-line client does with the paths provided to
subcommands is canonicalize them, I thought. What gives?

Augie Fackler wrote:
> I have a simple one-line patch that fixes the issue for me. It appears
> the problem occurred when the repo root was also the server root the
> repo root was declared in entries to be "http://svn.example.com/" but
> should have been "http://svn.example.com" according to
> svn_path_canonicalize(). I just tested this against svn.perian.org and
> the checkout worked as expected from there.
> Peace,
> Augie
>
> [[[
> Fix a regression from 1.4 where corrupt WC entries would be generated if
> the
> server root and the repository root were the same.
>
> * subversion/libsvn_wc/entries.c
> (write_entry): Call svn_path_canonicalize() on the repository root
> path before
> writing to entries.
>
> Patch by: Augie Fackler <durin42_at_gmail.com>
> ]]]
>
> Index: subversion/libsvn_wc/entries.c
> ===================================================================
> --- subversion/libsvn_wc/entries.c (revision 29086)
> +++ subversion/libsvn_wc/entries.c (working copy)
> @@ -1538,9 +1539,9 @@
> || (this_dir->repos == NULL
> || (entry->repos
> && strcmp(this_dir->repos, entry->repos) != 0))))
> - valuestr = entry->repos;
> + valuestr = svn_path_canonicalize(entry->repos, pool);
> else
> valuestr = NULL;
> write_str(buf, valuestr, pool);
>
> /* Schedule. */
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
> For additional commands, e-mail: dev-help_at_subversion.tigris.org
>

-- 
C. Michael Pilato <cmpilato_at_collab.net>
CollabNet   <>   www.collab.net   <>   Distributed Development On Demand