No need to set binary to undef for debian now we evaluate the provider variable instead.

diff --git a/manifests/params.pp b/manifests/params.pp
index b688dd536585a94699a9c0675ba38f72dc9c4c75..e5de774b996f60f0f01393808597f60e89d925e7 100644 (file)
--- a/manifests/params.pp
+++ b/manifests/params.pp
@@ -6,12 +6,12 @@ class tftp::params {
   $port       = '69'
   $options    = '--secure'
   $inetd_conf = '/etc/inetd.conf'
+  $binary     = '/usr/sbin/in.tftpd'
 
   case $::osfamily {
     'debian': {
       $package  = 'tftpd-hpa'
       $defaults = true
-      $binary   = undef
       $username = 'tftp'
       case $::operatingsystem {
         'debian': {
@@ -39,7 +39,6 @@ class tftp::params {
       $directory  = '/var/lib/tftpboot'
       $hasstatus  = false
       $provider   = 'base'
-      $binary     = '/usr/sbin/in.tftpd'
     }
     default: {
       $package    = 'tftpd'
@@ -47,7 +46,6 @@ class tftp::params {
       $defaults   = false
       $hasstatus  = false
       $provider   = undef
-      $binary     = '/usr/sbin/in.tftpd'
       warning("tftp:: $::operatingsystem may not be supported")
     }
   }