From 37e2005e792a9f6d3e7597e06eb4dc9bb822b5b9 Mon Sep 17 00:00:00 2001 From: Mike McQuaid <mike@mikemcquaid.com> Date: Thu, 31 Oct 2013 16:58:55 -0700 Subject: [PATCH] Use HOMEBREW_LOGS more consistently. --- Library/Homebrew/cmd/doctor.rb | 3 +-- Library/Homebrew/exceptions.rb | 2 +- 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/Library/Homebrew/cmd/doctor.rb b/Library/Homebrew/cmd/doctor.rb index b306b5850d..877b7880eb 100644 --- a/Library/Homebrew/cmd/doctor.rb +++ b/Library/Homebrew/cmd/doctor.rb @@ -381,8 +381,7 @@ end end def check_access_logs - folder = Pathname.new('~/Library/Logs/Homebrew') - if folder.exist? and not folder.writable_real? + if HOMEBREW_LOGS.exist? and not HOMEBREW_LOGS.writable_real? <<-EOS.undent #{folder} isn't writable. This can happen if you "sudo make install" software that isn't managed diff --git a/Library/Homebrew/exceptions.rb b/Library/Homebrew/exceptions.rb index f57235cca1..a0c0bf0048 100644 --- a/Library/Homebrew/exceptions.rb +++ b/Library/Homebrew/exceptions.rb @@ -200,7 +200,7 @@ class BuildError < Homebrew::InstallationError Homebrew.dump_build_env(env) puts onoe "#{formula.name} did not build" - unless (logs = Dir["#{ENV['HOME']}/Library/Logs/Homebrew/#{formula}/*"]).empty? + unless (logs = Dir["#{HOMEBREW_LOGS}/#{formula}/*"]).empty? puts "Logs:" puts logs.map{|fn| " #{fn}"}.join("\n") end -- GitLab