Per IRC, sending this patch with a log message and so forth. Patch and
log message after the conversation.
On Jan 30, 2008, at 4:03 PM, Augie Fackler wrote:
> On Jan 30, 2008, at 3:17 PM, Augie Fackler wrote:
>> On Jan 30, 2008, at 3:12 PM, C. Michael Pilato wrote:
>>> Augie Fackler wrote:
>>>> Ah, that information comes from the RA layer (I was just trying
>>>> to figure this out) - I'll go inspect them all and make sure they
>>>> DTRT for this case. Should I keep the original part of the patch
>>>> and include it with the work on the RA layer?
>>> We shouldn't need the original patch, but I won't complain if we
>>> include that code. Should make us more resilient in the case of
>>> some other unseen programmer error.
>> Acutally, in looking more - it's a bug in libsvn_ra_serf anyway. If
>> I switch my RA layer to neon and re-run the checkout without my
>> patch, it works. I'm working on a fix for serf now.
> This is the simplest fix I can come up with for serf:
> Index: subversion/libsvn_ra_serf/serf.c
> --- subversion/libsvn_ra_serf/serf.c (revision 29079)
> +++ subversion/libsvn_ra_serf/serf.c (working copy)
> @@ -992,6 +992,7 @@
> *url = session->repos_root_str;
> + *url = svn_path_canonicalize(*url, pool);
> return SVN_NO_ERROR;
> Anything else will require a little fiddling around inside serf to
> make some of the code in update.c work correctly.
Fix a bug in ra_serf where the computed repo root was not canonical.
(svn_ra_serf__get_repos_root): Canonicalize the repo root URL before
--- subversion/libsvn_ra_serf/serf.c (revision 29079)
+++ subversion/libsvn_ra_serf/serf.c (working copy)
@@ -992,6 +992,7 @@
*url = session->repos_root_str;
+ *url = svn_path_canonicalize(*url, pool);
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-01-30 23:43:32 CET