Re: [PATCH v2] POSIX ACL support

[Eli Zaretskii]

> From: Michael Albinus <address@hidden>
> Cc: Eli Zaretskii <address@hidden>,  address@hidden
> Date: Fri, 07 Dec 2012 09:19:09 +0100
> 
> Romain Francoise <address@hidden> 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.  */)

My take of the last sentence is below:

  The latter can happen for local files if Emacs was not compiled with
  ACL support, or for remote files if the file handler returns nil for
  the file's ACL entries.