diff options
| author | Lamont Granquist <lamont@scriptkiddie.org> | 2017-02-14 14:33:51 -0800 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2017-02-14 14:33:51 -0800 |
| commit | 72591651e0c528cefe101fb69f04131450e20e2d (patch) | |
| tree | 4f687738fe30c36d44de7a29ea0875331a82abce /lib/chef | |
| parent | 50d0a6b6a0b55d038fceb3b4047b8b52e5fb110a (diff) | |
| parent | b10d17943f8b007f9adeebbd97c32fb606947607 (diff) | |
| download | chef-72591651e0c528cefe101fb69f04131450e20e2d.tar.gz | |
Merge pull request #5816 from chef/lcg/yum-coerce-array-attributes
coerce immutable arrays to normal arrays in the yum_package resource
Diffstat (limited to 'lib/chef')
| -rw-r--r-- | lib/chef/resource/package.rb | 2 | ||||
| -rw-r--r-- | lib/chef/resource/yum_package.rb | 15 |
2 files changed, 13 insertions, 4 deletions
diff --git a/lib/chef/resource/package.rb b/lib/chef/resource/package.rb index baaa5be2c8..a1f174a6f3 100644 --- a/lib/chef/resource/package.rb +++ b/lib/chef/resource/package.rb @@ -1,7 +1,7 @@ # # Author:: Adam Jacob (<adam@chef.io>) # Author:: Tyler Cloke (<tyler@chef.io>) -# Copyright:: Copyright 2008-2016, Chef Software, Inc. +# Copyright:: Copyright 2008-2017, Chef Software Inc. # License:: Apache License, Version 2.0 # # Licensed under the Apache License, Version 2.0 (the "License"); diff --git a/lib/chef/resource/yum_package.rb b/lib/chef/resource/yum_package.rb index 1e0ad197ba..2fc5db5cc3 100644 --- a/lib/chef/resource/yum_package.rb +++ b/lib/chef/resource/yum_package.rb @@ -1,6 +1,6 @@ # # Author:: AJ Christensen (<aj@chef.io>) -# Copyright:: Copyright 2008-2016, Chef Software, Inc. +# Copyright:: Copyright 2008-2017, Chef Software Inc. # License:: Apache License, Version 2.0 # # Licensed under the Apache License, Version 2.0 (the "License"); @@ -24,8 +24,17 @@ class Chef resource_name :yum_package provides :package, os: "linux", platform_family: %w{rhel fedora} - # Install a specific arch - property :arch, [ String, Array ] + # XXX: the coercions here are due to the provider promiscuously updating the properties on the + # new_resource which causes immutable modification exceptions when passed an immutable node array. + # + # <lecture> + # THIS is why updating the new_resource in a provider is so terrible, and is equivalent to methods scribbling over + # its own arguments as unintended side-effects (and why functional languages that don't allow modifcations + # of variables eliminate entire classes of bugs). + # </lecture> + property :package_name, [ String, Array ], identity: true, coerce: proc { |x| x.is_a?(Array) ? x.to_a : x } + property :version, [ String, Array ], coerce: proc { |x| x.is_a?(Array) ? x.to_a : x } + property :arch, [ String, Array ], coerce: proc { |x| x.is_a?(Array) ? x.to_a : x } property :flush_cache, Hash, |