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

Apply suggestions from code review

Browse Source 
Code review changes

Co-authored-by: Mike McQuaid <mike@mikemcquaid.com>
This commit is contained in:
Lionell Loh Jian An 2020-07-08 23:55:48 +08:00 committed by GitHub
parent 84d86e6dbc
commit 5e99ecfbdb
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 7 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
Library/Homebrew/env_config.rb
View File

@ -112,7 +112,7 @@ module Homebrew
default: 15,
},
HOMEBREW_FORBIDDEN_LICENSES: {
description: "Use this environment variable to define a blacklist of space separated licenses and Homebrew " \
description: "Use this environment variable to define a denylist of space separated licenses and Homebrew " \
"will avoid installing the packages with those licenses.",
},
HOMEBREW_FORCE_BREWED_CURL: {

18
Library/Homebrew/formula_installer.rb
View File

@ -1106,23 +1106,17 @@ class FormulaInstaller
$stderr.puts @requirement_messages
end
def env_forbidden_licenses
Homebrew::EnvConfig.forbidden_licenses.split(" ")
end
def forbidden_license_check
forbidden_licenses = Homebrew::EnvConfig.forbidden_licenses.split(" ")
return if forbidden_licenses.blank?
def forbidden_license_check(f)
return if ENV["HOMEBREW_FORBIDDEN_LICENSES"].blank?
forbidden_licenses = env_forbidden_licenses
if forbidden_licenses.include? f.license
if forbidden_licenses.include? formula.license
raise CannotInstallFormulaError, <<~EOS
#{f.name} has a forbidden license #{f.license}.
#{formula.name} has a forbidden license #{formula.license}.
EOS
end
fi = FormulaInstaller.new(f)
fi.compute_dependencies.each do |dep, _|
compute_dependencies.each do |dep, _|
dep_f = dep.to_formula
next unless forbidden_licenses.include? dep_f.license