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

Re: [PATCH] Repository root relative support for the command-line client

From: Troy Curtis Jr <troycurtisjr_at_gmail.com>
Date: Tue, 29 Apr 2008 10:50:49 -0500

On Tue, Apr 29, 2008 at 10:33 AM, Daniel Shahaf <d.s_at_daniel.shahaf.co.il> wrote:
> Julian Foad wrote on Tue, 29 Apr 2008 at 15:36 +0100:
>
> > Daniel Shahaf wrote:
> > > Troy Curtis Jr wrote on Tue, 29 Apr 2008 at 04:49 -0500:
> > >
> > > > Yep sure enough. My patch can't accept PEG revs on relative url
> > > > targets. Doh! I don't have time for a patch right now, I'll do it
> > > > later tonight unless someone beats me to it.
> >
> > Oops. I missed that. Thanks Troy if you can fix that. I won't be around to
> > help apply it for the next 10 days, so please Daniel or someone else do so if
> > you can.
> >
>
> Okay.
>
>
> >
> > > Your patch also seems to require a trailing slash:
> > >
> > > 0:% ./subversion/svn/svn info '^'
> > > subversion/libsvn_client/info.c:266: (apr_err=150000)
> > > svn: '^' is not under version control
> > > 1:% ./subversion/svn/svn info '^'/ | grep URL
> > > URL: https://svn.collab.net/repos/svn
> > > 0:%
> > > Is this intentional?
> >
> > That was completely intentional. It's quite likely that a future development
> > might support more syntaxes beginning with "^", and then we would no longer
> > want a lone "^" to be interpreted as "repos-root-relative" because there would
> > be other just-as-likely meanings for it.
> >
>
> Thanks Julian. I wouldn't have thought about this myself, but I agree.
>
> Troy, is the syntax/semantics of ^ URLs documented anywhere? I didn't
> find anything obvious in the log message.
>

From the log message:
 "...This allows the user to use '^/' in front of any target to mean
the repository root url...."

I wrote this to mean that the repository root url syntax is "^/" not
"^". Of course you do have to read into it a bit to get that. Julian
mentioned that eventually this syntax would likely be documented in
the svnbook and I guess we should explicitly mention the bare
repository root case.

>
>
> > >
> > > Thanks,
> > >
> > > Daniel
> > > (who wishes he didn't have to quote ^ every time)
> >
> > Well, yes.
> >
> > - Julian
> >
>

Troy

-- 
"Beware of spyware. If you can, use the Firefox browser." - USA Today
Download now at http://getfirefox.com
Registered Linux User #354814 ( http://counter.li.org/)
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-04-29 17:51:12 CEST

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.