do we really need to quote the value?

diff --git a/manifests/debian.pp b/manifests/debian.pp
index 4f9f33725039ec9c356b372a6a5721b6269db7bd..dfa890fbd3772668eab27cbc0ed8965d4b13e43a 100644 (file)
--- a/manifests/debian.pp
+++ b/manifests/debian.pp
@@ -1,10 +1,10 @@
 class shorewall::debian inherits shorewall::base {
     case $shorewall_startup {
-      '': { $shorewall_startup = 1 }
+      '': { $shorewall_startup = "1" }
     }
     file{'/etc/default/shorewall':
         #source => "puppet://$server/modules/shorewall/debian/default",
-       content => template("shorewall/debian_default.erb"),
+        content => template("shorewall/debian_default.erb"),
         require => Package['shorewall'],
         notify => Service['shorewall'],
         owner => root, group => 0, mode => 0644;

diff --git a/templates/debian_default.erb b/templates/debian_default.erb
index 1a76fd67cd2b19e9fd4efc229358a2c38c97b8c3..96621f5ba7bf9d8006682e9fcbf41ca1e0038058 100644 (file)
--- a/templates/debian_default.erb
+++ b/templates/debian_default.erb
@@ -3,7 +3,7 @@
 
 # This file is brought to you by puppet
 
-<% if (shorewall_startup == "0" ) -%>
+<% if shorewall_startup == "0" -%>
 startup=0
 <% else -%>
 startup=1