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

Re: [sparse-directories] Depth parameter for the editor API

From: Daniel Rall <dlr_at_collab.net>
Date: 2007-04-05 01:23:41 CEST

On Sat, 24 Mar 2007, Karl Fogel wrote:

> Vlad Georgescu <vgeorgescu@gmail.com> writes:
> > What happens now if a pre-1.5 client tries to do a non-recursive
> > checkout? If I'm reading the code right, it looks like we set the
> > default depth to infinity, expecting the client to set the real depth
> > later via set_path(). However, old clients don't know about the new
> > set_path(), so it looks like they'd be stuck with a depth of infinity.
>
> That's on my list to look at. The client should still be sending the
> non-recurse flag, and the new code should still pay attention to it
> (by setting default_depth to svn_depth_files).

If you call the old svn_repos_begin_report() API directly, things
should work as expected, with the report_baton_t->default_depth being
used when the old set_path()/link_path() don't supply an overriding
depth. However, it appears that all the RA code uses the new
svn_repos_begin_report2() API, which will have the problem described
by Vlad (it always uses svn_depth_infinity).

  • application/pgp-signature attachment: stored
Received on Thu Apr 5 01:24:09 2007

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.