HyperWinX/brew
1
0
Fork 0
mirror of https://github.com/Homebrew/brew.git synced 2026-08-12 22:29:27 +04:00
Code Issues Packages Projects Releases Wiki Activity

Remove dead requirements_lines methods from brew info

Browse Source 
`Info.requirements_lines` has no callers, and its only callee
`Info.cask_requirements_lines` is unused once it is gone. Remove both and
drop `cask_requirements_lines` from the `private_class_method` list.

Claude-Session: https://claude.ai/code/session_011BgWRBydtdkvt8ocqH4nj9
This commit is contained in:
Douglas Eichelberger
2026-07-26 14:25:15 -07:00
parent 33bf599d66
commit c791ca532e
1 changed files with 1 additions and 36 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Library/Homebrew/cmd/info.rb
+1 -36
View File
@@ -174,20 +174,6 @@ module Homebrew
end
end
sig { params(formula_or_cask: T.any(Formula, Cask::Cask)).returns(T::Array[String]) }
def self.requirements_lines(formula_or_cask)
return [] unless $stdout.tty?
case formula_or_cask
when Formula
[]
when Cask::Cask
cask_requirements_lines(formula_or_cask)
else
T.absurd(formula_or_cask)
end
end
sig { params(tab: T.any(Tab, Cask::Tab)).returns(String) }
def self.installation_status(tab)
# TODO: Deprecate reading `installed_as_dependency`; `installed_on_request`
@@ -267,27 +253,6 @@ module Homebrew
metadata
end
sig { params(cask: Cask::Cask).returns(T::Array[String]) }
def self.cask_requirements_lines(cask)
macos_requirements = [cask.depends_on.macos, cask.depends_on.maximum_macos].compact
requirement = if macos_requirements.present?
requirement = macos_requirements.filter_map do |macos_requirement|
macos_requirement.display_s.delete_suffix(" (or Linux)").delete_prefix("macOS").strip.presence
end
requirement = requirement.present? ? "macOS #{requirement.join(", ")}" : "macOS"
requirement += " or Linux" if cask.supports_linux?
requirement
elsif cask.supports_macos? && cask.supports_linux?
"macOS or Linux"
elsif cask.supports_macos?
"macOS"
elsif cask.supports_linux?
"Linux"
end
requirement ? [requirement] : []
end
sig { params(time: T.any(Integer, Time)).returns(String) }
def self.formatted_time(time)
time = Time.at(time) if time.is_a?(Integer)
@@ -347,7 +312,7 @@ module Homebrew
end
private_class_method :formula_metadata_lines, :formatted_time, :pin_path_mtime,
:formula_installs_from_source?, :cask_requirements_lines
:formula_installs_from_source?
private