On 13.03.2014 13:55, Ivan Zhakov wrote:
> On 13 March 2014 16:46, Branko Čibej <brane_at_wandisco.com> wrote:
>> On 13.03.2014 13:37, Ivan Zhakov wrote:
>>
>> On 12 March 2014 18:17, Gareth McCaughan
>> <Gareth.McCaughan_at_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. brane_at_wandisco.com
Received on 2014-03-13 14:09:30 CET