matthijs/cdist
1
0
Fork 0
forked from ungleich-public/cdist
Code Issues Pull requests Projects Releases Wiki Activity

__acl: drop Solaris because POSIX-draft ACL specification is not supported

Browse source
This commit is contained in:
ander 2019-04-24 00:36:53 +03:00
commit f586937614
2 changed files with 20 additions and 30 deletions
Download patch file Download diff file Expand all files Collapse all files

8
cdist/conf/type/__acl/gencode-remote
View file

@ -74,13 +74,6 @@ fi
if [ -f "$__object/parameter/remove" ] if [ -f "$__object/parameter/remove" ]
then then
if echo "$os" | grep -Fq 'solaris'
then
# Solaris setfacl behaves differently.
# We will not support Solaris for now, because no way to test it.
# But adding support should be easy (use -s instead of -m on modify).
echo "$os setfacl do not support -x flag for ACL remove" >&2
else
echo "$acl_is" | while read -r acl echo "$acl_is" | while read -r acl
do do
# Skip wanted ACL entries which already exist # Skip wanted ACL entries which already exist
@ -101,7 +94,6 @@ then
echo "$setfacl_exec -x \"$remove\" \"$acl_path\"" echo "$setfacl_exec -x \"$remove\" \"$acl_path\""
echo "removed '$remove'" >> "$__messages_out" echo "removed '$remove'" >> "$__messages_out"
done done
fi
fi fi
for acl in $acl_should for acl in $acl_should

6
cdist/conf/type/__acl/man.rst
View file

@ -10,11 +10,9 @@ DESCRIPTION
----------- -----------
ACL must be defined as 3-symbol combination, using ``r``, ``w``, ``x`` and ``-``. ACL must be defined as 3-symbol combination, using ``r``, ``w``, ``x`` and ``-``.
Fully supported on Linux (tested on Debian and CentOS). Fully supported and tested on Linux, partial support for FreeBSD.
Partial support for FreeBSD, OSX and Solaris. OpenBSD, NetBSD and Solaris support is not possible.
OpenBSD and NetBSD support is not possible.
See ``setfacl`` and ``acl`` manpages for more details. See ``setfacl`` and ``acl`` manpages for more details.