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

Re: svn: Can't remove 'db/transactions/1-1.txn': Directory not empty

From: Artem Egorkine <arteme_at_gmail.com>
Date: 2006-06-22 10:45:20 CEST

On 6/21/06, Garrett Rooney <rooneg@electricjellyfish.net> wrote:
> On 6/21/06, Artem Egorkine <arteme@gmail.com> wrote:
> > More on the issue:
> >
> > Current solution for MacOS features a rewinddir() after removing each
> > item in the directory. Isn't that a bit of an overkill? I propose a
> > different approach:
> >
> > 1. do readdir(),
> > 2. if there's nothing to delete, we're done
> > 2. remove everything in the directory
> > 3. do rewinddir()
> > 4. goto 1.
> >
> > This approach features much less calls to rewinddir() while still
> > making sure everything is deleted.
> >
> > It can also be used on every OS, not just MacOS to be on the safe side.
> Are you actually sure that the problem is the same? It seems kind of
> unlikely that two different operating systems have the exact same bug
> that can be worked around in exactly the same way...
> -garrett

No, to tell the truth, I don't know if the problem is really the same.
While I'm waiting for TextDrive folks to respond to my request to let
me compile and test subversion on their box, I have written a quick
test in perl to see if readdir() does really return less entries than
it should. That test didn't have a problem - it returned correct
number of elements. While that doesn't yet mean that FreeBSD has the
same problem as MacOS, it doesn't prove that it doesn't have that
problem either.

To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Thu Jun 22 10:45:53 2006

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