From: Matthaus Litteken <matthaus@puppetlabs.com>
Date: Fri, 24 Feb 2012 05:01:20 +0000 (-0800)
Subject: Allow duplicate $required_packages
X-Git-Tag: 0.0.1~6^2~1
X-Git-Url: https://review.fuel-infra.org/gitweb?a=commitdiff_plain;h=f3735d2ba228fde8d2e119742e11bece4c2c78ac;p=puppet-modules%2Fpuppetlabs-apt.git

Allow duplicate $required_packages

Previously, if more than one apt::source required the same packages to be
installed it would fail with a duplicate exec resource. This adds the name of
the source resource to the exec and gives the exec a name, moving it to a
command parameter for the exec.
---

diff --git a/manifests/source.pp b/manifests/source.pp
index 0512fa9..9f31fe9 100644
--- a/manifests/source.pp
+++ b/manifests/source.pp
@@ -36,8 +36,9 @@ define apt::source(
   }
 
   if $required_packages != false {
-    exec { "${apt::params::provider} -y install ${required_packages}":
-      subscribe => File["${name}.list"],
+    exec { "Required packages: '${required_packages}' for ${name}":
+      command     => "${apt::params::provider} -y install ${required_packages}",
+      subscribe   => File["${name}.list"],
       refreshonly => true,
     }
   }