Moving out require parameter from module definition

diff --git a/manifests/init.pp b/manifests/init.pp
index 1c7872321c8efe3187c25af01c3eec65a960f7bb..cd220da7811cc07e933e9483eb735356c940d773 100644 (file)
--- a/manifests/init.pp
+++ b/manifests/init.pp
@@ -50,7 +50,7 @@ class apache {
 
   module { "macro":
     ensure  => present,
-    require => "mod_macro",
+    require => Package["mod_macro"],
   }
 
   # apache mod_macro configuration
@@ -127,14 +127,13 @@ class apache {
   # You can add a custom require (string) if the module depends on 
   # packages that aren't part of the default apache2 package. Because of 
   # the package dependencies, apache2 will automagically be included.
-  define module($ensure = 'present', $require = 'apache2') {
+  define module($ensure = 'present') {
     case $ensure {
       'present': {
         exec { "/usr/sbin/a2enmod $name":
           unless  => "/bin/sh -c '[ -L ${apache2_mods}-enabled/${name}.load ] \
                   && [ ${apache2_mods}-enabled/${name}.load -ef ${apache2_mods}-available/${name}.load ]'",
           notify  => Exec["force-reload-apache2"],
-          require => Package[$require],
         }
       }
       'absent': {
@@ -142,7 +141,6 @@ class apache {
           onlyif  => "/bin/sh -c '[ -L ${apache2_mods}-enabled/${name}.load ] \
                   && [ ${apache2_mods}-enabled/${name}.load -ef ${apache2_mods}-available/${name}.load ]'",
           notify  => Exec["force-reload-apache2"],
-          require => Package["apache2"],
         }
       }
       default: { err ("Unknown ensure value: '$ensure'") }