Conditionally adding the packages if they are not added previously

Replaced the not defined statements with a more practical ensure_packages and removed the not defined from passenger becase it should be defined only once
voxpupuli · Aug 11, 2016 · 8bd354a · 8bd354a
1 parent 127bdab
commit 8bd354a
Showing 1 changed file with 3 additions and 4 deletions.
diff --git a/manifests/package/debian.pp b/manifests/package/debian.pp
@@ -53,10 +53,9 @@
           key      => '16378A33A6EF16762922526E561F9B9CAC40B2F7',
         }
 
-        package { ['apt-transport-https', 'ca-certificates']:
-          ensure => 'present',
-          before => Apt::Source['nginx'],
-        }
+        ensure_packages([ 'apt-transport-https', 'ca-certificates' ])
+
+        Apt::Source['nginx'] -> Package['apt-transport-https','ca-certificates']
 
         package { 'passenger':
           ensure  => 'present',