Re: setfacl(2.4.0.15): for next year !!!!!

[Corinna Vinschen <corinna-cygwin at cygwin dot com>]

On Dec 23 12:10, Houder wrote:
> On 2015-12-23 11:50, Corinna Vinschen wrote:
> >On Dec 22 15:42, Houder wrote:
> >>The
> >>difference is that 'ls -l' on FC19 shows an additional plus sign.
> >
> >This is a problem in ls itself.  The reason is that with the start of
> >reimplementing the ACL handling (back in August 2014), the definition of
> >MIN_ACL_ENTRIES changed from 4 to 3.
> >
> >I recall having a discussion with eblake (coreutils maintainer) via IRC
> >in 2014 where we discussed this.  At that time the mask entry was only
> >fasked, so we came up with the fact that there's never an aclent_t with
> >4 entries, so ls is still using the old definition to maintain backward
> >compat.
> >
> >With the new code in 2.4.0 it's probably time to drop this Cygwin-specific
> >workaround in coreutils (but it doesn't hurt much either).
> 
> Does't hurt?
> 
> Well, without the plus sign, the output of 'ls -l' looks suspicious/weird to
> me in case of the example above ...

Now that you mention it... I agree.  It might hurt.  But it's a rather
seldom border case, so the confusion should be rather limited :)

> Perhaps Eric agrees ...

Yes, he very likely agrees.


Corinna

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

Attachment: signature.asc
Description: PGP signature