diff options
| author | Bill Weiss <bv@billweiss.net> | 2012-11-04 17:44:23 -0600 |
|---|---|---|
| committer | Bill Weiss <bv@billweiss.net> | 2012-11-04 17:44:23 -0600 |
| commit | 162a9fb7a305b5cc550497f01fece387d69f1060 (patch) | |
| tree | d9d5f488be0164664c251fcfd4b9dae2f590e97b | |
| parent | da121489208453158c9bdcac4596c5d8fed46ed8 (diff) | |
Trivial fix for style, and made the README and comment match up
Puppet-lint was complaining about "ensure found on line but it's not the
first attribute on line 109". Trivial fix (swapped the ensure and the
name).
Also, the README didn't mention Debian or CentOS, but the comment block
in init.pp did. I added both to the README. I happen to be using this
with Debian unstable, but I haven't tested it extensively enough to say
"yep, this works". It, however, works for me.
| -rw-r--r-- | README.markdown | 3 | ||||
| -rw-r--r-- | manifests/init.pp | 2 |
2 files changed, 3 insertions, 2 deletions
diff --git a/README.markdown b/README.markdown index bc6476a..a1e168a 100644 --- a/README.markdown +++ b/README.markdown @@ -6,7 +6,8 @@ This module has been built and tested using Puppet 2.6.x # Platforms # - * Enterprise Linux 5 + * Enterprise Linux 5 (and CentOS 5.4) * Ubuntu 10.04 Lucid * Amazon Linux 2011.09 has been tested on 2.7.x with facter version 1.6.2 * FreeBSD 9.0 + * Debian 6.0 Squeeze diff --git a/manifests/init.pp b/manifests/init.pp index 21f4296..b404071 100644 --- a/manifests/init.pp +++ b/manifests/init.pp @@ -105,8 +105,8 @@ class ntp($servers='UNSET', if ($supported == true) { package { 'ntp': - name => $pkg_name, ensure => $package_ensure, + name => $pkg_name, } file { $config: |