linting

diff --git a/manifests/base.pp b/manifests/base.pp
index db6852ac062c951e3910607092cabbd3a544c5bc..41348ef366a03be5320e6167d23644b75cf941f6 100644 (file)
--- a/manifests/base.pp
+++ b/manifests/base.pp
@@ -44,10 +44,10 @@ class shorewall::base {
     notify      => Service['shorewall'],
   }
   service{'shorewall':
-    ensure      => running,
-    enable      => true,
-    hasstatus   => true,
-    hasrestart  => true,
-    require     => Package['shorewall'],
+    ensure     => running,
+    enable     => true,
+    hasstatus  => true,
+    hasrestart => true,
+    require    => Package['shorewall'],
   }
 }

diff --git a/manifests/debian.pp b/manifests/debian.pp
index 326b42be40366d19fdeafa357b59ba3a4742cd05..07176a32a8ab52f27f12576ec90ff58d90d67fe0 100644 (file)
--- a/manifests/debian.pp
+++ b/manifests/debian.pp
@@ -1,11 +1,11 @@
+# debian specific things
 class shorewall::debian inherits shorewall::base {
   file{'/etc/default/shorewall':
-    content => template("shorewall/debian_default.erb"),
+    content => template('shorewall/debian_default.erb'),
     require => Package['shorewall'],
     notify  => Exec['shorewall_check'],
-    owner   => 'root', group => 'root', mode => '0644';
-  }
-  Service['shorewall']{
-    status => '/sbin/shorewall status'
+    owner   => 'root',
+    group   => 'root',
+    mode    => '0644';
   }
 }

diff --git a/manifests/extension_script.pp b/manifests/extension_script.pp
index 4abc6b18ed36a5f4632acf7a5b75fe08bb683b3b..80b83d3b26fcda934ecccca29737fa09e8529ce3 100644 (file)
--- a/manifests/extension_script.pp
+++ b/manifests/extension_script.pp
@@ -1,14 +1,16 @@
 # See http://shorewall.net/shorewall_extension_scripts.htm
-define shorewall::extension_script($script = '') {
-    case $name {
-        'init', 'initdone', 'start', 'started', 'stop', 'stopped', 'clear', 'refresh', 'continue', 'maclog': {
-          file { "/etc/shorewall/puppet/${name}":
-            content => "${script}\n",
-            notify  => Exec['shorewall_check'];
-          }
-        }
-        '', default: {
-          err("${name}: unknown shorewall extension script")
-        }
+define shorewall::extension_script(
+  $script
+) {
+  case $name {
+    'init', 'initdone', 'start', 'started', 'stop', 'stopped', 'clear', 'refresh', 'continue', 'maclog': {
+      file { "/etc/shorewall/puppet/${name}":
+        content => "${script}\n",
+        notify  => Exec['shorewall_check'];
+      }
     }
+    default: {
+      err("${name}: unknown shorewall extension script")
+    }
+  }
 }