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

Merge pull request #20612 from Homebrew/dug/simplify-affix-checks

Browse Source 
Avoid repeated calls to affix checks
This commit is contained in:
Mike McQuaid 2025-09-01 13:13:37 +00:00 committed by GitHub
commit d9718cec92
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
5 changed files with 5 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
Library/Homebrew/dev-cmd/pr-pull.rb
View File

@ -384,7 +384,7 @@ module Homebrew
files_to_commits[file] ||= []
files_to_commits[file] << commit
tap_file = (tap.path/file).to_s
if (tap_file.start_with?("#{tap.formula_dir}/") || tap_file.start_with?("#{tap.cask_dir}/")) &&
if tap_file.start_with?("#{tap.formula_dir}/", "#{tap.cask_dir}/") &&
File.extname(file) == ".rb"
next
end

2
Library/Homebrew/extend/os/mac/diagnostic.rb
View File

@ -376,7 +376,7 @@ module OS
return if @found.all? do |path|
realpath = Pathname.new(path).realpath.to_s
allowlist.any? { |rack| realpath.start_with?(rack) }
realpath.start_with?(*allowlist)
end
end

2
Library/Homebrew/extend/os/mac/keg_relocate.rb
View File

@ -272,7 +272,7 @@ module OS
# https://gitlab.kitware.com/cmake/cmake/-/issues/23251
return true if HOMEBREW_TEMP.to_s == "/private/tmp" && filename.start_with?("/tmp/")
filename.start_with?(HOMEBREW_TEMP.to_s) || filename.start_with?(HOMEBREW_TEMP.realpath.to_s)
filename.start_with?(HOMEBREW_TEMP.to_s, HOMEBREW_TEMP.realpath.to_s)
end
end
end

2
Library/Homebrew/shims/super/cc
View File

@ -281,7 +281,7 @@ class Cmd
true
# first two paths: reject references to Cellar or opt paths
# for unspecified dependencies
elsif path.start_with?(cellar) || path.start_with?(opt)
elsif path.start_with?(cellar, opt)
dep = path[keg_regex, 2]
dep && @deps.include?(dep)
elsif path.start_with?(prefix, cachedir, tmpdir)

2
Library/Homebrew/unpack_strategy.rb
View File

@ -92,7 +92,7 @@ module UnpackStrategy
strategies&.sort_by { |s| s.extensions.map(&:length).max || 0 }
&.reverse
&.find { |s| s.extensions.any? { |ext| extension.end_with?(ext) } }
&.find { |s| extension.end_with?(*s.extensions) }
end
sig { params(path: Pathname).returns(T.nilable(UnpackStrategyType)) }