[prev in list] [next in list] [prev in thread] [next in thread] 

List:       subversion-dev
Subject:    Re: PATCH: BOOK: Windows drive letter in "file:" URL
From:       Julian Foad <julianfoad () btopenworld ! com>
Date:       2003-08-30 21:41:55
[Download RAW message or body]

Francois Beausoleil wrote:
> 
> I mostly agree with the patch.  The appc.xml change is correct, but the
> ch02.xml patch is incorrect.  Here's what I get on Win2K SP4, 0.27.0
> (r6715):
> 
> K:\>svnadmin create repos
> H:\>svn co "file:///K|/repos" w
> svn: Bogus URL
> svn: URL `file:///K|/repos' is not properly URI-encoded

Oh.  Thank you very much for reviewing this.  That's the same as I get on Linux, but \
I assumed it would work on Windows.  The existing text, saying that either a ":" or a \
"|" can be used, was inserted just over six months ago in r5057 by cmpilato.  C-Mike, \
do you know what's up with this?

Now I would like someone to say whether this is supposed to work but presently \
suffering from a bug, or not supposed to work, before I go further.

- Julian


> H:\>svn co "file:///K:/repos" wc
> Checked out revision 0.
> 
> So, this part of the diff should be dropped:
> -$ svn checkout file:///X|/path/to/repos
> +C:\> svn checkout "file:///X|/path/to/repos"
> 
> And in the following paragraphs, the first one should be removed too:  
> +        <para>In the second syntax, you need to quote the URL so that
> +          the vertical bar character is not interpreted as a
> pipe.</para>
> +
> +        <para>Note that a URL uses ordinary slashes even though the
> native
> +          (non-URL) form of a path on Windows uses backslashes.</para>
> +


> On Fri, 29 Aug 2003 17:39:43 +0100, "Julian Foad" <julianfoad@btopenworld.com> \
> said: 
> > Please could a Windows user check and confirm this (particularly the two
> > examples in ch02.xml) for me.
> > 
> > I am not running Windows and cannot check it.  I believe this is a better
> > patch than the one attached to issue 1306 by Francois Beausoleil
> > (Francois, do you agree?).  I will commit it and close issue 1306 if
> > someone confirms it.
> > 
> > Log:
> > [[[
> > Book: update the information on using a Windows drive letter in a "file:"
> > URL.
> > 
> > * doc/book/book/appc.xml
> > Replace the FAQ about working around the inability to specify a drive
> > letter with one pointing to the section on how to do it.
> > * doc/book/book/ch02.xml
> > Correct, update and explain the examples of using a drive letter in a
> > URL.
> > ]]]


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org


[prev in list] [next in list] [prev in thread] [next in thread]