admin/brew
1
0
Fork 0
Code Issues Packages Projects Releases 6 Wiki Activity

utils: provide a better location in 'odeprecated'

Browse Source 
Try to find a formula in the backtrace to make the warning message more
helpful in identifying the culprit, as the formula is not always the
immediate caller of a deprecated method. Provide some sane fallbacks if
there's not formula in the call stack.
This commit is contained in:
Martin Afanasjew 2016-07-30 14:51:14 +02:00
parent 19a77c816e
commit 0a33cc591d
1 changed files with 13 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
Library/Homebrew/utils.rb
View File

@ -125,9 +125,18 @@ def odeprecated(method, replacement = nil, options = {})
"There is no replacement."
end
# Show the first location that's not in compat.
backtrace = options[:caller] || caller
caller_message = backtrace[1]
# Try to show the most relevant location in message, i.e. (if applicable):
# - Location in a formula.
# - Location outside of 'compat/'.
# - Location of caller of deprecated method (if all else fails).
backtrace = options.fetch(:caller, caller)
caller_message = backtrace.detect do |line|
line.start_with?("#{HOMEBREW_LIBRARY}/Taps/")
end
caller_message ||= backtrace.detect do |line|
!line.start_with?("#{HOMEBREW_LIBRARY_PATH}/compat/")
end
caller_message ||= backtrace[1]
message = <<-EOS.undent
Calling #{method} is #{verb}!
@ -136,7 +145,7 @@ def odeprecated(method, replacement = nil, options = {})
EOS
if ARGV.homebrew_developer? || options[:die]
raise FormulaMethodDeprecatedError.new message
raise FormulaMethodDeprecatedError, message
else
opoo "#{message}\n"
end