Ben Collins-Sussman <sussman@collab.net> writes:
> On Aug 23, 2005, at 2:30 PM, kfogel@collab.net wrote:
> >
> > Am I missing something, or would this solve our problem totally?
>
>
> When folks pointed out that mod_dav_svn already has a 'private'
> syntax for version-urls (!svn/bc/NNNNN/path), Greg Stein's reaction
> was, "hey, let's just make a public variant of that syntax -- !svn/
> rev/NNNNN/path -- it's a one line code change. And we can promise to
> never change it."
>
> But I like Karl's "path@REV" syntax proposal better, because
>
> 1. it's easier to remember.
>
> 2. it matches the existing svn commandline syntax.
>
>
> The great thing about #2 is that it prevents confusion. In earlier
> threads, there was confusion about people typing "http://host/repos/!
> svn/bc/NNNNN/path" in their web browser, and then going to the prompt
> and feeding the same URL to 'svn cat'. We want to avoid such mix-
> ups. Having identical syntax avoids the confusion.
Nonono. Please *do not* do this to public URL namespace. It's one
thing to bind revision numbers to URLs in our command-line client
syntax for the sake of UI and path/rev association simplicity (and
yes, remember that this URL@REV syntax is not Subversion-wide -- it's
only in our one little commandline client). But this change has no
place trickling into mod_dav_svn.
I much prefer the !svn/rev/NNNNN/path syntax, because besides not
destroying the integrity of our existing URL namespace (what happens
if you have a file with '@[0-9]+' at the end of it under version
control?), it allows for relative URLs in HTML documents served
directory from mod_dav_svn to continue to work. An index.html with a
URL of http://.../!svn/4500/index.html can reference "./logo.png" and
the browser will know to use http://.../!svn/4500/logo.png to fetch
that image.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Tue Aug 23 22:58:36 2005
