Re: svn commit: rev 5814 - in trunk/subversion: libsvn_wc tests/clients/cmdline

On Tue, May 13, 2003 at 10:21:54PM +0100, Philip Martin wrote:
>
> It's a problem already.
>
> svnadmin create repo
> svn co file://`pwd`/repo wc
> svn mkdir wc/bar
> rm -rf wc/bar
> svn st wc
> ! wc/bar
> svn mkdir file://`pwd`/repo/foo
> svn up wc
> A wc/foo
> D wc/bar
> Updated to revision 1.
>
> I don't think the wc/bar entry should be removed.

I agree. Pre-r5814 behavior is a "working copy not locked" error, which
is presumably not ideal either. I presume that the best thing would be
to leave the entry untouched?

...

As far as the fragility of this solution, basically we have to think
of all the possible cases under which it might be wrong to delete a
missing directory after a successful update. If we forget one, bad
news. But I don't understand how that's different from, e.g., having
to determine the set of conditions under which the crawler needs to
call reporter->delete_path...

-brian

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Wed May 14 01:35:20 2003