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