voxpupuli / puppet-openssl

Puppet OpenSSL module
Apache License 2.0
38 stars 84 forks source link

use ensure_packages for package openssl #82

Closed zivis closed 7 years ago

zivis commented 7 years ago

should the resource Package[ 'openssl'] also be declared with ensure_packages?

raphink commented 7 years ago

Why?

zivis commented 7 years ago

Well openssl is a cross-cutting concern, and we were used to ensure_package openssl, in a more generic profile and were using additionally ::openssl, because it "came" as a dependency. But now we are using this module in our more generic profile. And it is no longer needed for us So feel free to close this PR…

raphink commented 7 years ago

OK. I'll just close cause I'd rather the module actually manages the package.