On Mar 29 12:58, Bruno Haible via Cygwin wrote:
> Corinna Vinschen wrote:
> > Btw., while I was testing test-file-has-acl.sh, I found two more bugs,
> > one in test-file-has-acl.sh, and one (a problem of account handling in
> > Windows) in Cygwin's setfacl(1). Together with the above change to
> > acl_extended_file(), fixing the bug in setfacl(1) is sufficient to run
> > test-file-has-acl.sh successfully.
> >
> >
> > However:
> >
> > I ran the script with VERBOSE=1 and this is what is printed:
> >
> > [...]
> > + chmod 600 tmpfile0
> > + acl_flavor=none
> > + acl_flavor=linux
> > [...]
> >
> > Oops, acl_flavor=linux?
> >
> > Turns out, the script checks the output for a --set-file option,
> > which is supported by Cygwin's setfacl since commit ed4d919c24646
> > ("setfacl: Rename the option --file to --set-file, as on Linux").
>
> This should have been fixed two years ago already:
> https://git.savannah.gnu.org/gitweb/?p=gnulib.git;a=commitdiff;h=2d4adbff973ee2cc186cb6256b61d246c254fe93
> You must be running an old copy of test-file-has-acl.sh.
Oh drat. I did a `git fetch' but missed the `git merge' afterwards.
Sigh. Let me recheck with the latest version...
Corinna
