Re: [Issue 2959] [sparse-directories] depth upgrade against old servers is broken

Vlad Georgescu <vgeorgescu@gmail.com> writes:
> When I suggested a server_supports_depth() function, I was thinking of
> putting it in the reporter vtable, so that svn_wc_crawl_revisions
> wouldn't have to be revved to take a 'depth_compatibility_trick'
> argument. Instead, it would call reporter->server_supports_depth().

For everyone, here's what Vlad and I discussed in IRC:

<kfogel> I thought that having a generic capabilities querying
              mechanism would be best. However, the reporter could
              *use* that, so that svn_wc_crawl_revisions3() could just
              ask the reporter, and not have to take a new
              'depth_compatibility_trick' argument.

<kfogel> IOW, a new RA function, but also a new reporter
              function.

<vgeorgescu_> sure

<kfogel> ok

<kfogel> first let me finish this ra_serf code, and shelve a
              working patch with 'depth_compatibility_trick', then
              tackle the reporter vtable next

<vgeorgescu_> ok

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sat Oct 13 01:46:25 2007