On 13.03.2014 13:55, Ivan Zhakov wrote: > On 13 March 2014 16:46, Branko Čibej <br...@wandisco.com> wrote: >> On 13.03.2014 13:37, Ivan Zhakov wrote: >> >> On 12 March 2014 18:17, Gareth McCaughan >> <gareth.mccaug...@lightblueoptics.com> wrote: >> >> Ivan Zhakov wrote: >> >> It looks like serious issue. Could you please file issue in Subversion >> issue tracker: http://subversion.tigris.org/issues >> >> Done. Issue #4478. >> >> Gareth, thanks a lot! >> >> It seems we have second reason to create temporary files in target WC >> directory, instead of .svn/tmp. Another problem we had before is >> inherited ACL on Windows discussed year ago: >> http://mail-archives.apache.org/mod_mbox/subversion-dev/201309.mbox/%3C20130928110059.d1bb8d007dfe7b26cbcb4f719cb77fd6.be88925bf6.wbe%40email16.secureserver.net%3E >> >> I've prepared stupid patch to create temporary files in WC, instead of >> .svn/tmp and it seems working fine. >> >> >> -1. This leaves us with no reliable way to clean up the working copy in case >> of an aborted operation. >> >> There should be a better way for managing ACLs on windows that does not >> require us to mess up the working copy. I'm perfectly happy with just >> documenting that we don't support different ACLs for .svn and the rest of >> the WC, at least for now. >> > Please note that problem reported is not about inherited ACL: now > users got NOT_INDEXED attribute on all WC files, because .svn marked > to by not indexed by Windows Search and tools like that.
I understand that; it simply means that when we install a file from .svn/tmp into the working copy proper, we have to change that attribute; either before or after the file is moved into place. See, e.g., SetFileInformationByHandle with FILE_BASIC_INFO, or SetFileAttributes. -- Brane -- Branko Čibej | Director of Subversion WANdisco // Non-Stop Data e. br...@wandisco.com