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

Re: Issue 1509 - was WC Permissions

From: Philip Martin <philip_at_codematters.co.uk>
Date: 2003-12-11 17:43:03 CET

John Peacock <jpeacock@rowman.com> writes:

> What I propose is that svn_io_set_file_read_write() itself be extended
> to try a little harder. If the existing directory would permit the
> file to be renamed and recreated (i.e., the group ownership of the
> directory would permit it), then after the first attempt to change
> READONLY fails, the code could then attempt to rename and
> copy/recreate the file with the correct perms.

No, I don't think that's the way to go. The thing about
svn_io_set_file_read_write is that it is really a Windows function,
it's not needed on Unix. Windows doesn't allow us to delete read-only
files (I don't know if it allows us to rename them either), so we have
to set them read-write first. As I recall, there are very few places
where the Unix client *needs* to set read-write, most of the time it
just slows us down, and leads to the ownership problems. APR doesn't
provide platform independence here.

A better plan would be to identify those places where the set
read-write call is only needed on Windows, and make those platform
specific. I believe there is an issue that covers this.

Philip Martin
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Thu Dec 11 17:44:24 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.