From 7c9c93f2af8a970729bc9904faddbe2ef73d97b7 Mon Sep 17 00:00:00 2001
From: stephen <stephen@puppetlabs.com>
Date: Tue, 16 Oct 2012 12:48:29 +0100
Subject: have the module fail on unsupported os

---
 manifests/init.pp | 11 +----------
 1 file changed, 1 insertion(+), 10 deletions(-)

(limited to 'manifests')

diff --git a/manifests/init.pp b/manifests/init.pp
index b404071..0d19747 100644
--- a/manifests/init.pp
+++ b/manifests/init.pp
@@ -51,7 +51,6 @@ class ntp($servers='UNSET',
 
   case $::operatingsystem {
     debian, ubuntu: {
-      $supported  = true
       $pkg_name   = [ 'ntp' ]
       $svc_name   = 'ntp'
       $config     = '/etc/ntp.conf'
@@ -66,7 +65,6 @@ class ntp($servers='UNSET',
       }
     }
     centos, redhat, oel, linux, fedora, Amazon: {
-      $supported  = true
       $pkg_name   = [ 'ntp' ]
       $svc_name   = 'ntpd'
       $config     = '/etc/ntp.conf'
@@ -80,7 +78,6 @@ class ntp($servers='UNSET',
       }
     }
     freebsd: {
-      $supported  = true
       $pkg_name   = ['.*/net/ntp']
       $svc_name   = 'ntpd'
       $config     = '/etc/ntp.conf'
@@ -95,15 +92,10 @@ class ntp($servers='UNSET',
       }
     }
     default: {
-      $supported = false
-      notify { "${module_name}_unsupported":
-        message => "The ${module_name} module is not supported on ${::operatingsystem}",
-      }
+       fail("The ${module_name} module is not supported on ${::operatingsystem}")
     }
   }
 
-  if ($supported == true) {
-
     package { 'ntp':
       ensure => $package_ensure,
       name   => $pkg_name,
@@ -125,5 +117,4 @@ class ntp($servers='UNSET',
       hasrestart => true,
       subscribe  => [ Package[$pkg_name], File[$config] ],
     }
-  }
 }
-- 
cgit v1.2.3