Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Directory +x / Files not +x #9

Closed
Poil opened this issue Aug 5, 2014 · 3 comments · Fixed by #61
Closed

Directory +x / Files not +x #9

Poil opened this issue Aug 5, 2014 · 3 comments · Fixed by #61
Labels
enhancement New feature or request

Comments

@Poil
Copy link

Poil commented Aug 5, 2014

Hi

It does not seem possible to do setfacl -R -m u:myuser:rwX myfolder

Any idea (except no-recurse/give all subfiles) ?

Best regards

@dobbymoodge
Copy link

This is a feature request. I agree "proper" recursion would be good, but it needs to behave like the File type. I don't maintain a puppet system any longer, and didn't really know ruby when I wrote this, but if you or anyone else wants to implement this I'd happily merge in the changes

@dobbymoodge
Copy link

So looking into this a bit further, it looks like the munging here will need to be updated: https://github.com/dobbymoodge/puppet-acl/blob/master/lib/puppet/type/acl.rb#L194

Also, either set_perm needs to test the path being set and adust perms or the perms passed to set_perm here need to be modified conditionally: https://github.com/dobbymoodge/puppet-acl/blob/master/lib/puppet/provider/acl/posixacl.rb#L114

@dobbymoodge dobbymoodge added the enhancement New feature or request label Oct 27, 2016
@towo
Copy link
Member

towo commented Aug 1, 2018

Bump. Still relevant.

pyslarvt added a commit to pyslarvt/puppet-posix_acl that referenced this issue Apr 8, 2019
jadestorm pushed a commit to ncstate-daniel/puppet-posix_acl that referenced this issue Jun 12, 2019
@jadestorm jadestorm mentioned this issue Jun 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants