[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

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

From: Brian Denny <brian_at_briandenny.net>
Date: 2003-05-14 01:37:04 CEST

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...


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

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.