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

Re: 'svn info URL'

From: Peter N. Lundblad <peter_at_famlundblad.se>
Date: 2005-02-27 22:01:46 CET

On Thu, 24 Feb 2005, Ben Collins-Sussman wrote:

> On Feb 24, 2005, at 6:13 PM, sussman@tigris.org wrote:
>
> > Author: sussman
> > Date: Thu Feb 24 18:13:51 2005
> > New Revision: 13144
> >
> > Log:
> > Make 'svn info' work on URLs as well as working-copy items.
> >
> >
>
> So we have a problem here.
>
...
> But there's a shortcoming in svn_ra_get_dir(): it doesn't return a '.'
> entry. This means we're fetching parent entries not just for a file,
> but for every directory as well! This has two yucky consequences:
>
> 1. 'svn info repos_root_URL' fails. There's no way to get the
> svn_dirent_t of '/'!
>
> 2. It's inefficient. A directory may have a whole lot of children.
>
> So I'm wondering... should we add a new RA API to fetch a single
> svn_dirent_t?
>
Since I already revised every RA function, now is our chance to fix such
mistakes without further revision... I'd say just make it return an entry
for "" and also add a depth argument (we can start with supporting 0 and
1). In libsvn_ra/wrapper_template.h, you can jsut filter out that entry.

We need to extend the protocols as well. For ra_svn, we can just add a
depth argument. No depth means the old behaviour: depth 1 and no "" entry.
...and you're the DAV expert.

Regards,
//Peter

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sun Feb 27 22:00:58 2005

This is an archived mail posted to the Subversion Dev mailing list.