Re: svn commit: r11509 - branches/1.1.x

On Wed, Oct 20, 2004 at 08:00:51AM -0500, lundblad@tigris.org wrote:
> Author: lundblad
> Date: Wed Oct 20 08:00:43 2004
> New Revision: 11509
>
> Modified:
> branches/1.1.x/STATUS
> Log:
> * STATUS: Veto r11490, but vote for r11501.
>
>
> Modified: branches/1.1.x/STATUS
> Url: http://svn.collab.net/viewcvs/svn/branches/1.1.x/STATUS?view=diff&rev=11509&p1=branches/1.1.x/STATUS&r1=11508&p2=branches/1.1.x/STATUS&r2=11509
> ==============================================================================
> --- branches/1.1.x/STATUS (original)
> +++ branches/1.1.x/STATUS Wed Oct 20 08:00:43 2004
> @@ -52,8 +52,11 @@
> It used to work. It should work now.
> Notes:
> r11490 is just cleanups. But makes it merge cleanly.
> + OTOH, r11490 breaks file://localhost//machine/repos unnecessarily.

What? You want a URL that points to a remote machine to accept a
hostname of localhost? Your desired behavior of such a URL is bogus.

Windows itself won't even accept that URL. It'll take the following
URLS though:
file:////machine/path
file://machine/path

Personally I believe the only one we should support is the later.
Anytime the hostname is blank or localhost we should interpret it as a
local name and that means that duplicate path separators are no-ops.

So I'd veto any implementation that supports
file://localhost//machine/repos
or
file:////machine/repos

So open a new issue requesting this feature. We don't have a way to
refer to UNC paths via a URL and we never have. Obviously this is
desireable but is a new feature and belongs in 1.2.

-- 
Ben Reser <ben@reser.org>
http://ben.reser.org
"Conscience is the inner voice which warns us somebody may be looking."
- H.L. Mencken
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org