summaryrefslogtreecommitdiff
path: root/lib/puppet/parser
diff options
context:
space:
mode:
authorAdrien Thebo <git@somethingsinistral.net>2014-01-15 11:03:49 -0800
committerAdrien Thebo <git@somethingsinistral.net>2014-01-15 11:03:49 -0800
commit686a05aea20eb45560698caaad5c9f2f01821ae6 (patch)
treeefe17ed17deb0ff54151c1ce338f8efb30475757 /lib/puppet/parser
parent735db82bef56bf939c971ab76a66647269d6ae35 (diff)
downloadpuppet-stdlib-686a05aea20eb45560698caaad5c9f2f01821ae6.tar.gz
puppet-stdlib-686a05aea20eb45560698caaad5c9f2f01821ae6.tar.bz2
(maint) refactor ensure_packages for clarity
Diffstat (limited to 'lib/puppet/parser')
-rw-r--r--lib/puppet/parser/functions/ensure_packages.rb12
1 files changed, 7 insertions, 5 deletions
diff --git a/lib/puppet/parser/functions/ensure_packages.rb b/lib/puppet/parser/functions/ensure_packages.rb
index 371d63a..1e0f225 100644
--- a/lib/puppet/parser/functions/ensure_packages.rb
+++ b/lib/puppet/parser/functions/ensure_packages.rb
@@ -1,7 +1,6 @@
#
# ensure_packages.rb
#
-require 'puppet/parser/functions'
module Puppet::Parser::Functions
newfunction(:ensure_packages, :type => :statement, :doc => <<-EOS
@@ -9,12 +8,15 @@ Takes a list of packages and only installs them if they don't already exist.
EOS
) do |arguments|
- raise(Puppet::ParseError, "ensure_packages(): Wrong number of arguments " +
- "given (#{arguments.size} for 1)") if arguments.size != 1
- arguments[0] = [ arguments[0] ] unless arguments[0].kind_of?(Array)
+ if arguments.size != 1
+ raise(Puppet::ParseError, "ensure_packages(): Wrong number of arguments " +
+ "given (#{arguments.size} for 1)")
+ end
+
+ packages = Array(arguments[0])
Puppet::Parser::Functions.function(:ensure_resource)
- arguments[0].each { |package_name|
+ packages.each { |package_name|
function_ensure_resource(['package', package_name, {'ensure' => 'present' } ])
}
end