Greg Stein <gst...@gmail.com> writes:

> Sorry I wasn't clear. The function is only useful for situations like:
>
> $ svn rm path/some-subdir
> $ rm -r path/some-subdir

It needs an f: 'rm -rf'

> ... and then calling the function on path/some-subdir. It will record
> (in path/.svn/) that some-subdir is "not-present" in the working copy.
>
> The function is used as part of the commit code. Post-commit, we are
> allowed to mark that subdir as "not-present" in the new revision of
> path/... since we know it was deleted as part of the commit. We don't
> mind that the subdir is missing because that recording is in the stub
> located in path/.svn/
>
> That said, the function is bogus because this kind of recording should
> happen during commit processing naturally. And it should bump the
> revision on that directory, rather than just alter the presence value

Which revision/directory?  The deleted directory doesn't have a
revision any more and the parent might not be changing.

> stashed away in the stub. We should be able to eliminate the function
> today, and fold it into commit processing (and then it *may* further
> simplify during the single-db move).

Once we have a singe-db then this sort of deleted directory stops
being special, it's just a normal deleted directory.  We will have to
keep the function because it's part of the public API.

-- 
Philip

Reply via email to