From 54d3a967952bf96537167bd61d6a038c32940842 Mon Sep 17 00:00:00 2001 From: dheerajd-msys Date: Mon, 25 Jun 2018 14:53:25 +0530 Subject: [MSYS-847] fix bare rescue Signed-off-by: dheerajd-msys --- .rubocop.yml | 4 ---- lib/mixlib/shellout/windows.rb | 4 ++-- mixlib-shellout-windows.gemspec | 2 +- 3 files changed, 3 insertions(+), 7 deletions(-) diff --git a/.rubocop.yml b/.rubocop.yml index 02e7511..489eaaa 100644 --- a/.rubocop.yml +++ b/.rubocop.yml @@ -7,9 +7,5 @@ Lint/UselessAccessModifier: Exclude: - 'lib/mixlib/shellout/windows/core_ext.rb' -# Set for mixlib-shell-out-windows.gemspec -Security/Eval: - Enabled: false - Style/HashSyntax: Enabled: true diff --git a/lib/mixlib/shellout/windows.rb b/lib/mixlib/shellout/windows.rb index da49d4a..07d3fbc 100644 --- a/lib/mixlib/shellout/windows.rb +++ b/lib/mixlib/shellout/windows.rb @@ -122,7 +122,7 @@ module Mixlib wmi = WmiLite::Wmi.new kill_process_tree(process.process_id, wmi, logger) Process.kill(:KILL, process.process_id) - rescue + rescue SystemCallError logger.warn("Failed to kill timed out process #{process.process_id}") if logger end @@ -357,7 +357,7 @@ module Mixlib ].join) end Process.kill(:KILL, instance.wmi_ole_object.processid) - rescue + rescue SystemCallError if logger logger.debug([ "Failed to kill child process #{child_pid}::", diff --git a/mixlib-shellout-windows.gemspec b/mixlib-shellout-windows.gemspec index bd3c21d..3a3968d 100644 --- a/mixlib-shellout-windows.gemspec +++ b/mixlib-shellout-windows.gemspec @@ -1,4 +1,4 @@ -gemspec = eval(File.read(File.expand_path("../mixlib-shellout.gemspec", __FILE__))) +gemspec = instance_eval(File.read(File.expand_path("../mixlib-shellout.gemspec", __FILE__))) gemspec.platform = Gem::Platform.new(%w{universal mingw32}) -- cgit v1.2.1