On Wed, Oct 30, 2013 at 10:18:35AM +0100, Attila Nagy wrote: > On 10/30/13 09:56, Stefan Sperling wrote: > >I believe it's the stupid code replaced below, which I wrote in r880420. > >Because of it we end up setting perms based on umask upon every commit, > >and end up expanding restrictive file permissions. > > > >This patch should fix the problem. > Indeed, the file remains at 600 after commit. Can I expect it in the next > release?
Yes. I'll commit it now and nominate it for backport to 1.8.5. > >Note that Subversion users cannot and couldn't ever rely on per-file > >permission flags to keep files in a working copy hidden from other > >users. This is because Subversion often resets permission bits when > >it re-creates working files from temporary files. There is no way to > >set bits of those files to something other than the current umask. > If the original file is there, subversion could copy its permissions. > Of course, if it's lost, it can't do it. That's what Subversion tries to do and what it does in most cases. But in some cases that's impossible (I don't recall which in detail). Thanks for your report! If you find more problems let us know.
