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

Re: svn switch and URL:s with colons

From: Daniel Rall <dlr_at_collab.net>
Date: 2006-06-30 20:32:10 CEST

On Fri, 30 Jun 2006, Julian Foad wrote:

> Daniel Rall wrote:
> >On Thu, 29 Jun 2006, Fredrik Arnerup wrote:
> >>In 1.3.1, svn switch does not seem to consider "svn://server:/foobar"
> >>and "svn://server/foobar" to be the same repo.
> [...]
> >>Can anyone confirm this? Should I file a bug report?
> >
> >Many common tools (Firefox, CVS, etc.) handle this format. Section
> >3.2.2 and 3.2.3 of RFC 2616 are slightly at odds with each other on
> >this topic, suggesting first that the colon should only be included
> >with a port number, then going on to say that a colon which lacks a
> >port number should be treated identically as one without for
> >comparison purposes.
>
> I don't see any inconsistency.

I was referring to the BNF, which makes it looks like both are
necessary:

 http_URL = "http:" "//" host [ ":" port ] [ abs_path [ "?" query ]]

> Both sections talk about when the "port ... is empty or not given",
> and in the syntax notation the "port" in '[ ":" port ]' is defined
> as zero or more digits if you follow the reference to RFC2396.
...

+1, agreed.

We could crib some URL parsing code from httpd to handle this for both
ra_svn and ra_dav as necessary, instead of doing straight strcmp()'s.

  • application/pgp-signature attachment: stored
Received on Fri Jun 30 20:33:45 2006

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.