diff options
author | Stuart Preston <stuart@chef.io> | 2018-04-21 04:41:40 +0200 |
---|---|---|
committer | Stuart Preston <stuart@chef.io> | 2018-04-21 04:41:40 +0200 |
commit | d6b8aa701aea18db31b4d9ec2943aa6bfb0efc4e (patch) | |
tree | 870ac0f8344f3c0a565c9d9db3f7f1a958d79ef3 | |
parent | f3698119a48d4706bccd4767f3c50f0820f9ae5c (diff) | |
download | chef-d6b8aa701aea18db31b4d9ec2943aa6bfb0efc4e.tar.gz |
Avoid conflict with build_powershell_command from powershell_out mixin
Signed-off-by: Stuart Preston <stuart@chef.io>
-rw-r--r-- | lib/chef/provider/package/powershell.rb | 18 |
1 files changed, 9 insertions, 9 deletions
diff --git a/lib/chef/provider/package/powershell.rb b/lib/chef/provider/package/powershell.rb index 87af7f61d8..a7aa6cb223 100644 --- a/lib/chef/provider/package/powershell.rb +++ b/lib/chef/provider/package/powershell.rb @@ -36,7 +36,7 @@ class Chef def define_resource_requirements super - if powershell_out("$PSVersionTable.PSVersion.Major").stdout.strip.to_i < 5 + if node['languages']['powershell']['version'].to_i < 5 raise "Minimum installed Powershell Version required is 5" end requirements.assert(:install) do |a| @@ -53,7 +53,7 @@ class Chef # Installs the package specified with the version passed else latest version will be installed def install_package(names, versions) names.each_with_index do |name, index| - powershell_out(build_powershell_command("Install-Package '#{name}'", versions[index]), timeout: new_resource.timeout) + powershell_out(build_powershell_package_command("Install-Package '#{name}'", versions[index]), timeout: new_resource.timeout) end end @@ -61,11 +61,11 @@ class Chef def remove_package(names, versions) names.each_with_index do |name, index| if versions && !versions[index].nil? - powershell_out(build_powershell_command("Uninstall-Package '#{name}'", versions[index]), timeout: new_resource.timeout) + powershell_out(build_powershell_package_command("Uninstall-Package '#{name}'", versions[index]), timeout: new_resource.timeout) else version = "0" until version.empty? - version = powershell_out(build_powershell_command("Uninstall-Package '#{name}'"), timeout: new_resource.timeout).stdout.strip + version = powershell_out(build_powershell_package_command("Uninstall-Package '#{name}'"), timeout: new_resource.timeout).stdout.strip unless version.empty? logger.info("Removed package '#{name}' with version #{version}") end @@ -79,9 +79,9 @@ class Chef versions = [] new_resource.package_name.each_with_index do |name, index| version = if new_resource.version && !new_resource.version[index].nil? - powershell_out(build_powershell_command("Find-Package '#{name}'", new_resource.version[index]), timeout: new_resource.timeout).stdout.strip + powershell_out(build_powershell_package_command("Find-Package '#{name}'", new_resource.version[index]), timeout: new_resource.timeout).stdout.strip else - powershell_out(build_powershell_command("Find-Package '#{name}'"), timeout: new_resource.timeout).stdout.strip + powershell_out(build_powershell_package_command("Find-Package '#{name}'"), timeout: new_resource.timeout).stdout.strip end if version.empty? version = nil @@ -96,9 +96,9 @@ class Chef version_list = [] new_resource.package_name.each_with_index do |name, index| version = if new_resource.version && !new_resource.version[index].nil? - powershell_out(build_powershell_command("Get-Package '#{name}'", new_resource.version[index]), timeout: new_resource.timeout).stdout.strip + powershell_out(build_powershell_package_command("Get-Package '#{name}'", new_resource.version[index]), timeout: new_resource.timeout).stdout.strip else - powershell_out(build_powershell_command("Get-Package '#{name}'"), timeout: new_resource.timeout).stdout.strip + powershell_out(build_powershell_package_command("Get-Package '#{name}'"), timeout: new_resource.timeout).stdout.strip end if version.empty? version = nil @@ -108,7 +108,7 @@ class Chef version_list end - def build_powershell_command(command, version = nil) + def build_powershell_package_command(command, version = nil) command = [command] unless command.is_a?(Array) command.unshift("(") %w{-Force -ForceBootstrap}.each do |arg| |