Skip to content

Commit

Permalink
Merge pull request #7359 from Bo98/shims-check
Browse files Browse the repository at this point in the history
formula_cellar_checks: check for shim dir references
  • Loading branch information
MikeMcQuaid committed Apr 17, 2020
2 parents 5f9fee4 + c85ff09 commit 04105ab
Showing 1 changed file with 24 additions and 0 deletions.
24 changes: 24 additions & 0 deletions Library/Homebrew/formula_cellar_checks.rb
Original file line number Diff line number Diff line change
Expand Up @@ -201,6 +201,29 @@ def check_python_packages(lib, deps)
EOS
end

def check_shim_references(prefix)
return unless prefix.directory?

keg = Keg.new(prefix)

matches = []
keg.each_unique_file_matching(HOMEBREW_SHIMS_PATH) do |f|
match = f.relative_path_from(keg.to_path)

next if match.to_s.match? %r{^share/doc/.+?/INFO_BIN$}

matches << match
end

return if matches.empty?

<<~EOS
Files were found with references to the Homebrew shims directory.
The offending files are:
#{matches * "\n "}
EOS
end

def audit_installed
@new_formula ||= false

Expand All @@ -216,6 +239,7 @@ def audit_installed
problem_if_output(check_elisp_dirname(formula.share, formula.name))
problem_if_output(check_elisp_root(formula.share, formula.name))
problem_if_output(check_python_packages(formula.lib, formula.deps))
problem_if_output(check_shim_references(formula.prefix))
end
alias generic_audit_installed audit_installed

Expand Down

0 comments on commit 04105ab

Please sign in to comment.