summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorLamont Granquist <lamont@scriptkiddie.org>2018-07-05 17:16:51 -0700
committerLamont Granquist <lamont@scriptkiddie.org>2018-07-05 17:16:51 -0700
commitd176c42c151fe37401bbe42a470ff90f6f4b20c2 (patch)
treea3845b9cb66a2a2db61b7d7650b17706fff0f230
parent1e241b3f4327dfad0c26d141fe64eacd1d89494b (diff)
downloadchef-nolan/double-stdout.tar.gz
fix logger logic with is_default? from new mixlib-configchef-nolan/double-stdout
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
-rw-r--r--Gemfile.lock10
-rw-r--r--chef-config/chef-config.gemspec2
-rw-r--r--lib/chef/application.rb6
3 files changed, 7 insertions, 11 deletions
diff --git a/Gemfile.lock b/Gemfile.lock
index 07ff8835fa..539825271c 100644
--- a/Gemfile.lock
+++ b/Gemfile.lock
@@ -1,6 +1,6 @@
GIT
remote: https://github.com/chef/chefstyle.git
- revision: fac2105b132fddc5f671411b2b5f1cba7a2307c3
+ revision: c16b9ceab413259bda43a2d683f1f918c1be3db5
branch: master
specs:
chefstyle (0.10.0)
@@ -89,7 +89,7 @@ PATH
chef-config (14.3.29)
addressable
fuzzyurl
- mixlib-config (>= 2.2.11, < 3.0)
+ mixlib-config (>= 2.2.12, < 3.0)
mixlib-shellout (~> 2.0)
tomlrb (~> 1.2)
@@ -149,7 +149,7 @@ GEM
highline (1.7.10)
htmlentities (4.3.4)
iniparse (1.4.4)
- inspec-core (2.2.27)
+ inspec-core (2.2.34)
addressable (~> 2.4)
faraday (>= 0.9.0)
hashie (~> 3.4)
@@ -179,7 +179,7 @@ GEM
mixlib-log
mixlib-authentication (2.1.1)
mixlib-cli (1.7.0)
- mixlib-config (2.2.11)
+ mixlib-config (2.2.12)
tomlrb
mixlib-log (2.0.4)
mixlib-shellout (2.3.2)
@@ -300,7 +300,7 @@ GEM
systemu (2.6.5)
thor (0.20.0)
tomlrb (1.2.7)
- train-core (1.4.19)
+ train-core (1.4.21)
json (>= 1.8, < 3.0)
mixlib-shellout (~> 2.0)
travis (1.8.8)
diff --git a/chef-config/chef-config.gemspec b/chef-config/chef-config.gemspec
index f372f944c6..a1afe21b18 100644
--- a/chef-config/chef-config.gemspec
+++ b/chef-config/chef-config.gemspec
@@ -16,7 +16,7 @@ Gem::Specification.new do |spec|
spec.require_paths = ["lib"]
spec.add_dependency "mixlib-shellout", "~> 2.0"
- spec.add_dependency "mixlib-config", ">= 2.2.11", "< 3.0"
+ spec.add_dependency "mixlib-config", ">= 2.2.12", "< 3.0"
spec.add_dependency "fuzzyurl"
spec.add_dependency "addressable"
spec.add_dependency "tomlrb", "~> 1.2"
diff --git a/lib/chef/application.rb b/lib/chef/application.rb
index 9df8de156a..81637eabb8 100644
--- a/lib/chef/application.rb
+++ b/lib/chef/application.rb
@@ -205,11 +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?
- ( !log_location_default? ) && STDOUT.tty? && !Chef::Config[:daemonize]
- end
-
- def log_location_default?
- Chef::Config.configurables[:log_location].instance_variable_get(:@default_value).inspect == Chef::Config[:log_location].inspect
+ Chef::Config.is_default?(:log_location) && Chef::Config[:log_location].tty? && !Chef::Config[:daemonize]
end
def configure_stdout_logger