Re: relative externals fail with spaces
Karl Fogel wrote:
> Stefan Küng <tortoisesvn_at_gmail.com> writes:
>> I'm wondering: this change is in libsvn_client, i.e., it's independent
>> of the ra layer. But the issue only shows for http(s), not for file or
>> svn. So what are ra_svn and ra_local doing differently?
>> I'm not familiar with that part of the svn code, but couldn't this lead
>> to double-escaping of urls if ra_svn/ra_local already do this themselves?
> What's going on is that file:// and svn:// do looser URL-parsing than
> http://. The error came from libsvn_ra_neon/session.c:parse_url(); in
> fact, it's possible that with ra_serf, there would have been no error or
> a different error. So file:// and svn:// weren't already doing the
> escaping themselves. The difference is just that they could handle a
> URL with spaces in it :-).
> (My patch passed 'make check' over all RA layers, and I checked svn://
> in GDB too -- the URL is not double-escaped there.)
Thanks for explaining!
oo // \\ "De Chelonian Mobile"
(_,\/ \_/ \ TortoiseSVN
\ \_/_\_/> The coolest Interface to (Sub)Version Control
/_/ \_\ http://tortoisesvn.net
Received on 2008-10-01 18:19:38 CEST
This is an archived mail posted to the Subversion Dev