Re: [PATCH v2] POSIX ACL support

[Romain Francoise <romain@orebokech.com>]

Hi Eli,

Eli Zaretskii <eliz@gnu.org> writes:

> Suggest an additional entry "@cindex access control list" here.

Ok.

> We don't mention "ACL" or explain it anywhere else in the manual.  So
> this acronym should be explained here, and its first use should use
> @dfn, as we do with any new terminology.

Makes sense. I'll try to come up with a description general enough to
apply to all potential platforms.

>> +DEFUN ("file-acl", Ffile_acl, Sfile_acl, 1, 1, 0,
>> +       doc: /* Return ACL entries of file named FILENAME, as a string.
>> +Return nil if file does not exist or is not accessible, or if Emacs was
>> +not compiled with ACL support.  */)

> The last sentence of the doc string is inaccurate, because remote
> files are supported even if Emacs was not built with ACL support.

Yes... if there's an implementation of file-acl for the remote handler.
This bit was inherited from the SELinux functions, which have a similar
docstring. I'm not sure how to phrase it better, though.

Thanks!