Re: svn commit: r9085 - in trunk/subversion: include libsvn_subr libsvn_wc

On Tue, 2004-03-16 at 18:31, Ben Reser wrote:
> > If we want to enable shared working directories (in 1.1), perhaps the
> > best option is to introduce a function private to libsvn_wc, which
> > invokes svn_io_set_file_read_write only on Windows.

> Actually there is code in other sections that would need this same
> behavior. I think we should just add a function in 1.1 that gives us a
> place to put things that need to be done before a file is removed.

I feel like "prep file for removal" is not a good candidate for a real
API. I'd be a bit happier seeing an API for "remove a file which we
expect is read-only," which could perhaps be called
svn_io_remove_read_only_file().

(Alternatively, we could treat it as a bug that svn_io_remove_file()
doesn't work on read-only files under Windows, and just make it do so,
but that might affect our performance if we start issuing set-attr calls
before every file deletion on Windows.)

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Wed Mar 17 00:39:11 2004