Re: svn commit: rev 1893 - trunk/subversion/libsvn_subr

Branko Čibej <brane@xbc.nu> writes:
> Exactly. Sorry, I should've written that to the log.

Oh, you still can if you want :-).

> What actually triggered the problem for me was updating on Windows
> after the expat-lite dir was removed from the repo. The update failed,
> because it couldn't remove the various read-only files in
> expat-lite/.svn. I had to remove them by hand, "svn cleanup" and "svn
> up" again to get it working.
>
> This fix solves this particular problem, which also showed up in
> python tests on Win32.

Gotcha, thanks.

> On another note: I feel we should rename that funciton to
> svn_dir_remove. If we want to get something into APR, I'd rather see
> an apr_dir_recurse that takes a callback, and implement svn_dir_remove
> on top of that. IIRC we discussed that some time ago. Thoughts?

I think it's fine. We're not lurching very hard in the direction of
getting it incorporated into apr, and it's not essential that we do
so.

-K

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Tue May 7 18:35:25 2002