Re: [PATCH v2] POSIX ACL support

[Michael Albinus <michael.albinus@gmx.de>]

Romain Francoise <romain@orebokech.com> writes:

> Hi Eli,

Hi,

>>> +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.

| Return nil if file does not exist or is not accessible, or if Emacs is
| not able to determine the ACL entries.  The latter happens, if Emacs was
| not compiled with ACL support, or a remote file handler returns nil
| ACL entries.  */)

Similar wording could be applied to file-selinux-context and
set-file-selinux-context.

Once the patch is installed in the trunk (i.e., the interface is
stabilized), I will try to implement corresponding file name handlers.

> Thanks!

Best regards, Michael.