search

ghoneycutt / puppet-module-pam

Puppet module to manage PAM
Other
18 stars 78 forks source link

Update init.pp #84

Closed pmiles closed 9 years ago

pmiles commented 9 years ago

In our organisation, we don't need all of the functionality of the puppet nsswitch module.

This change wraps a condition around the nsswitch include, and if nsswitch module doesn't exist, silently ignore the include instead.

I believe this change is backward compatible.

ptionally include nsswitch module if its installed.

ghoneycutt commented 9 years ago

instead of using if defined, this should be a parameter that defaults to true for the inclusion of the class

Here's a similar example to base your tests off of

https://github.com/ghoneycutt/puppet-module-puppet/pull/89/files

ghoneycutt commented 9 years ago

Closing this due to inactivity. Opened issue #89 with an example, should you want to pursue this.