remove variable defaults, they are handled by the paramterization of the class
authorMicah Anderson <micah@riseup.net>
Wed, 3 Apr 2013 17:59:31 +0000 (13:59 -0400)
committerMicah Anderson <micah@riseup.net>
Wed, 3 Apr 2013 18:09:58 +0000 (14:09 -0400)
make sure template variables are properly looked up.

manifests/debian.pp
manifests/init.pp
manifests/linux.pp
templates/Debian/default

index ee2a2f1..eb4d57a 100644 (file)
@@ -9,20 +9,6 @@ class stunnel::debian inherits stunnel::linux {
     pattern => '/usr/bin/stunnel4',
   }
 
-  # make the /etc/default/stunnel ENABLED configurable with a variable
-  # and default to on
-  case $stunnel::startboot {
-    '': { $stunnel::startboot = '1' }
-    default: { $stunnel::startboot = '1' }
-  }
-
-  # make the /etc/default/stunnel extra configurable with a variable
-  # and default to adding nothing to the default file
-  case $stunnel::default_extra {
-    '': { $stunnel::default_extra = '' }
-    default: { $stunnel::default_extra = '' }
-  }
-
   file { '/etc/default/stunnel4':
     content => template('stunnel/Debian/default'),
     require => Package['stunnel4'],
index 1204909..f7f72b8 100644 (file)
@@ -19,6 +19,7 @@
 
 class stunnel ( $ensure_version = 'present', $startboot = '1', $default_extra )
 {
+
   case $::operatingsystem {
     debian: { class { 'stunnel::debian': } }
     centos: { class {  'stunnel::centos': } }
index 972f21c..a4a926e 100644 (file)
@@ -1,6 +1,5 @@
 class stunnel::linux inherits stunnel::base {
 
-  if $stunnel::ensure_version == '' { $stunnel::ensure_version = 'installed' }
   package { 'stunnel':
     ensure => $stunnel::ensure_version
   }
index ccfefa5..9e2f4d3 100644 (file)
@@ -3,11 +3,11 @@
 # September 2003
 
 # Change to one to enable stunnel automatic startup
-ENABLED=<%= startboot %>
+ENABLED=<%= scope.lookupvar('stunnel::startboot') %>
 FILES="/etc/stunnel/*.conf"
 OPTIONS=""
 
 # Change to one to enable ppp restart scripts
 PPP_RESTART=0
 
-<%= default_extra %>
+<%= scope.lookupvar('stunnel::default_extra') %>