saz / puppet-sudo

Manage sudo with Puppet on Debian-, RedHat- and SUSE-based linux distributions and some BSDs
Other
107 stars 215 forks source link

4.0.0 broken on ArchLinux - params::package_ldap undefined #173

Closed jantman closed 7 years ago

jantman commented 7 years ago

I only discovered this because I'm using this module in one of my own, and depend on >=3.0.0 <4.0.0 but left my .fixtures.yml with just the git repo, so it's pulling in master.

It appears that, on Archlinux, 4.0.0 now fails because init.pp has $package_ldap = $sudo::params::package_ldap but params.pp doesn't define $package_ldap in the Archlinux block.

saz commented 7 years ago

Thanks for pointing this out. Can you give the current master a try, if it's working as expected?

jantman commented 7 years ago

Looks good to me, thanks so much!