move package declarations in base resources and wrap with ensure_resource

1 files changed, 5 insertions, 3 deletions

diff --git a/manifests/maildir.pp b/manifests/maildir.pp
index ab7ff55..c394a5c 100644
--- a/manifests/maildir.pp
+++ b/manifests/maildir.pp
@@ -17,13 +17,15 @@
 #
 # 
 define backupninja::maildir(
-  $order = 99, $ensure = present, $when = 'everyday at 21:00', $srcdir = false,
+  $order = 99, $ensure = present, $ensure_rsync_version = 'installed',
+  $when = 'everyday at 21:00', $srcdir = false,
   $destdir = false, $desthost = false, $destuser = false, $destid_file = false,
   $remove = false, $multiconnection = yes, $keepdaily='4', $keepweekly='2',
   $keepmonthly='2')
 {
-  include backupninja::client::maildir
-                          
+  # install client dependencies
+  ensure_resource('package', 'rsync', {'ensure' => $ensure_rsync_version})
+
   case $srcdir { false: { err("need to define a source directory to backup!") } }
   case $destdir { false: { err("need to define a destination directory to backup!") } }
   case $desthost { false: { err("need to define a destination host for backups!") } }