Use ensure_packages, to avoid duplicate definition in case...

Use ensure_packages, to avoid duplicate definition in case Package['shorewall'] is defined elsewhere already.

manifests/base.pp

 # base things for shorewall
 class shorewall::base {
 
-  package { 'shorewall':
-    ensure => $shorewall::ensure_version,
-  }
+  ensure_packages({ 'shorewall' => { ensure => $shorewall::ensure_version }})
 
   # This file has to be managed in place, so shorewall can find it
   file {