Re: [PATCH] Make sparse-directories checkouts/updates work with old servers

Vlad Georgescu <vgeorgescu@gmail.com> writes:
> Hmm, I'm not sure it's something as simple as a bugfix. Suppose we
> checkout a greek tree at depth=empty, and then do a depth=infinity
> update. How is the (1.5+) server supposed to know it should send A/ and
> iota? All it sees is the top-level set-path with a depth of infinity,
> the same as a fully complete working copy might send.

Why would the top-level set_path() send depth=infinity? Wouldn't it
send depth=empty? I thought the whole point of the depth parameter to
set_path() was to accurately describe the working copy state.

The *desired* depth, infinity, should be sent by other means. If
we've conflated those two things, then we've made a mistake.

...and, I think maybe we have made that mistake, haven't we? :-)

> With old servers it gets more complicated. You'd probably have to call
> delete-path on each file or directory that's missing, except that a
> working copy doesn't actually have that information.
>
> And, of course, whatever solution we come up with must simultaneously
> work for both old and new servers, because we don't currently
> distinguish between them.

Ugh. This is a deep problem. I'm pondering.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sun Apr 22 19:42:53 2007