remove the empty case for OpenBSD in package.pp installation,

saz / puppet-sudo

Manage sudo with Puppet on Debian-, RedHat- and SUSE-based linux distributions and some BSDs

Other

105 stars 215 forks source link

remove the empty case for OpenBSD in package.pp installation, #131

Closed buzzdeee closed 7 years ago

buzzdeee commented 9 years ago

but enhance the default case, to allow installation of package also on OpenBSD.

reason for that is that OpenBSD recently dropped sudo from base and moved it to ports/packages. At some point in time, a default in param.pp could be added. I tried with versioncmp on operationgsystemrelease, but I only have 5.7-snapshots, or 5.8-beta hosts around, that versioncmp messed totally up with those in order to get it right :(

while there, I added/enhanced spec tests for OpenBSD package installation.

buzzdeee commented 9 years ago

reworked a bit, in order to add default package name in params.pp for 5.7 to '', and 5.8 to 'sudo', based on ::kernelversion. ::operatingsystemrelease is not really suitable for use with OpenBSD, since it usually has -stable, -snapshot, -beta etc. suffixed, which complicates matter with versioncmp()