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

[PATCH] Fix a bug in ra_serf (was: Fix a regression from 1.4 in trunk and branch)

From: Augie Fackler <durin42_at_gmail.com>
Date: Wed, 30 Jan 2008 16:43:19 -0600

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.
>
> Augie

[[[
Fix a bug in ra_serf where the computed repo root was not canonical.
* subversion/libsvn_ra_serf/serf.c
   (svn_ra_serf__get_repos_root): Canonicalize the repo root URL before
    returning.
]]]

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;
  }

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

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.