[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] POSIX ACL support
From: |
Stefan Monnier |
Subject: |
Re: [PATCH] POSIX ACL support |
Date: |
Tue, 20 Nov 2012 13:15:10 -0500 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.2 (gnu/linux) |
> However, I think Glenn was also referring to the additional argument to
> `copy-file', which doesn't preserve the modes.
Yes, this also needs to be handled in a similar way (tho I'm not sure
what you mean by "which doesn't preserve the modes" since the docstring
says "This function always sets the file modes of the output file to match
the input file.").
> But maybe we can change the existing `preserve-selinux-context'
> argument into a more general `preserve-platform-attributes' argument
> and do both ACL and SELinux? Any caller that wants to preserve the
> SELinux context probably wants to preserve the ACL entries as
> well, anyway.
Actually, I'd go even further and merge PRESERVE-UID-GID
PRESERVE-SELINUX-CONTEXT into a single argument.
> Ideally we'd get rid of `file-modes', `file-selinux-context' and
> `file-posix-acl' and just add the info to `file-attributes', which already
> returns a list of 12 different attributes. Then `set-file-attributes'
> would be created to set all those different attributes in one go. But that
> would be a lot of work...
I'm not sure I'm ready to add the info to file-attributes.
But we could start by providing a new file-extended-attributes which
returns a self-descriptive list of attributes, and then
set-file-extended-attributes which takes that list and applies it.
Stefan
- [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, 2012/11/19
- 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 <=
- 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
- Re: [PATCH] POSIX ACL support, Achim Gratz, 2012/11/21
- Re: [PATCH] POSIX ACL support, Eli Zaretskii, 2012/11/21
- Re: [PATCH] POSIX ACL support, Achim Gratz, 2012/11/21
- Re: [PATCH] POSIX ACL support, Paul Eggert, 2012/11/19