diff options
author | dheerajd-msys <dheeraj.dubey@msystechnologies.com> | 2018-10-23 12:24:02 +0530 |
---|---|---|
committer | dheerajd-msys <dheeraj.dubey@msystechnologies.com> | 2018-10-23 17:49:15 +0530 |
commit | e8fa4f16e8a7a24b5d66e0beb5bb90dd74496190 (patch) | |
tree | 7bd3c17f93cb6f48cc5ecd83854c7adb981828d4 | |
parent | 68cada49a89f33b8d82e5ec080aa3fbd8b265831 (diff) | |
download | chef-e8fa4f16e8a7a24b5d66e0beb5bb90dd74496190.tar.gz |
fixed review comments
Signed-off-by: dheerajd-msys <dheeraj.dubey@msystechnologies.com>
-rw-r--r-- | lib/chef/application.rb | 2 | ||||
-rw-r--r-- | lib/chef/config.rb | 3 |
2 files changed, 1 insertions, 4 deletions
diff --git a/lib/chef/application.rb b/lib/chef/application.rb index ca653e3375..a63d8218f4 100644 --- a/lib/chef/application.rb +++ b/lib/chef/application.rb @@ -205,7 +205,7 @@ class Chef # Based on config and whether or not STDOUT is a tty, should we setup a # secondary logger for stdout? def want_additional_logger? - !(Chef::Config.is_default?(Chef::Config[:log_location]) || STDOUT.closed? || Chef::Config[:daemonize]) + ( Chef::Config[:log_location].class != IO ) && STDOUT.tty? && !Chef::Config[:daemonize] end def configure_stdout_logger diff --git a/lib/chef/config.rb b/lib/chef/config.rb index aaa7f9f15d..f5466ae145 100644 --- a/lib/chef/config.rb +++ b/lib/chef/config.rb @@ -80,8 +80,5 @@ class Chef end end - def self.is_default?(key) - key.is_a?(IO) - end end end |