Skip to content

Commit 162a9fb

Browse files
committed
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.
1 parent da12148 commit 162a9fb

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

README.markdown

+2-1
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,8 @@ This module has been built and tested using Puppet 2.6.x
66

77
# Platforms #
88

9-
* Enterprise Linux 5
9+
* Enterprise Linux 5 (and CentOS 5.4)
1010
* Ubuntu 10.04 Lucid
1111
* Amazon Linux 2011.09 has been tested on 2.7.x with facter version 1.6.2
1212
* FreeBSD 9.0
13+
* Debian 6.0 Squeeze

manifests/init.pp

+1-1
Original file line numberDiff line numberDiff line change
@@ -105,8 +105,8 @@
105105
if ($supported == true) {
106106

107107
package { 'ntp':
108-
name => $pkg_name,
109108
ensure => $package_ensure,
109+
name => $pkg_name,
110110
}
111111

112112
file { $config:

0 commit comments

Comments
 (0)