AllPlayers/puppet-acl — pull request #2 by dobbymoodge (2012-03-15 15:44:54+00:00)
Title: Fix behavior of :recursive parameter for acl type

Fixed bug where getfacl/setfacl commands in posixacl.rb were being called with empty string arguments when acl param recursive was false. This triggered a syntax error in the command because of how execve passes args to programs.

* Part of this fix was making the setfacl/getfacl calls dumber using conditionals. There may be a more clever way to implement this but I just learned enough Ruby to make this work.

Fixed acl :recursive param in type/acl.rb to use :true and :false, and check_recursive in posixacl.rb to return boolean true or false