[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] POSIX ACL support
From: |
Glenn Morris |
Subject: |
Re: [PATCH] POSIX ACL support |
Date: |
Mon, 19 Nov 2012 12:11:38 -0500 |
User-agent: |
Gnus (www.gnus.org), GNU Emacs (www.gnu.org/software/emacs/) |
Romain Francoise wrote:
> The following patch adds support for preserving POSIX ACL entries of
> files, which at the moment are lost when Emacs saves a buffer (unless
> `backup-by-copying' is set).
Thanks for writing this. It is definitely needed IMO.
> -(defun backup-buffer-copy (from-name to-name modes context)
> +(defun backup-buffer-copy (from-name to-name modes context acl)
[and several other such]
This is really a general comment, not an issue with your patch in
particular, so don't let it distract. I just wonder if it is possible to
find a different way to pass this information around, so that we don't
need to have an ever increasing list of function arguments and return
values (preserve-this, preserve-that, preserve-the-other, etc...).
I don't have a good suggestion for how to do that.
(I have a vague memory this was mentioned when SELinux was added.)
- [PATCH] POSIX ACL support, Romain Francoise, 2012/11/18
- Re: [PATCH] POSIX ACL support, Eli Zaretskii, 2012/11/18
- Re: [PATCH] POSIX ACL support,
Glenn Morris <=
- Re: [PATCH] POSIX ACL support, Eli Zaretskii, 2012/11/19
- Re: [PATCH] POSIX ACL support, Stefan Monnier, 2012/11/19
- Re: [PATCH] POSIX ACL support, Romain Francoise, 2012/11/19
- Re: [PATCH] POSIX ACL support, Stefan Monnier, 2012/11/20
- Re: [PATCH] POSIX ACL support, Romain Francoise, 2012/11/21
- Re: [PATCH] POSIX ACL support, Stefan Monnier, 2012/11/21
- Re: [PATCH] POSIX ACL support, Paul Eggert, 2012/11/21
- Re: [PATCH] POSIX ACL support, Romain Francoise, 2012/11/21
- Re: [PATCH] POSIX ACL support, Eli Zaretskii, 2012/11/21
- Re: [PATCH] POSIX ACL support, Paul Eggert, 2012/11/21