RE: Possible bug regarding spaces in unc paths in 1.6.8

Stefan Küng wrote:

> On 15.06.2010 19:13, David Balažic wrote:
> > Stefan Küng wrote:
> >
> >> On 15.06.2010 18:58, David Balažic wrote:
[snip]
> >>
> >> Now try these:
> >> file:///C:/Program%2520files/nonexistent
> >> file:///C:/Program%20files/nonexistent
> >
> > In both cases I get an error dialog saying:
> > The specified path does not exist. Check the path, and
> then try again.
> >
> > Seems as expected?
>
> Yes.
> Another problem:
> the svn library converts file:/// urls to local paths and
> relative repo urls. But it doesn't unescape those urls for the local paths.
> So it ends up specifying a path like c:\program%20files
> with the windows APIs. And that path doesn't exist.

Looks like a bug in SVN library then.

file:///C:/Program%20files/ means "C:/Program files/"
If it is decoded to anything else, then that is clearly wrong handling of an URL.

So there is an URL file:///C:/foo/bar/baz and it gets divided into
file:///C:/foo/bar (local part)
/baz (repo part)

And the further file:///C:/foo/bar -> C:/foo/bar

Is that correct? Then in the last step, if it leaves the %20 as is, that is a bug.

Regards,
David

------------------------------------------------------
http://tortoisesvn.tigris.org/ds/viewMessage.do?dsForumId=4061&dsMessageId=2622087

To unsubscribe from this discussion, e-mail: [users-unsubscribe_at_tortoisesvn.tigris.org].
Received on 2010-06-15 19:22:30 CEST