Inconsistent setting of readonly attribute in 1.7 ?

[Corinna Vinschen]

On Aug 26 20:37, Christian Franke wrote:
> If ACLs are used, Cygwin 1.7 chmod() does never set R/O attribute, but  
> open() sets it if a R/O file is created:
> [...]
> This change might be enough (or not):
>
> fhandler_base::open (int flags, mode_t mode)
> ...
>      -if (!(mode & (S_IWUSR | S_IWGRP | S_IWOTH)))
>      +if (!pc.has_acls () && !(mode & (S_IWUSR | S_IWGRP | S_IWOTH)))
>         file_attributes |= FILE_ATTRIBUTE_READONLY;

I applied the change plus a comment.  Thanks for catching.


Corinna

-- 
Corinna Vinschen                  Please, send mails regarding Cygwin to
Cygwin Project Co-Leader          cygwin AT cygwin DOT com
Red Hat

--
Problem reports:       http://cygwin.com/problems.html
FAQ:                   http://cygwin.com/faq/
Documentation:         http://cygwin.com/docs.html
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple