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

formula_installer: remove feature flags.

Browse Source 
We've been testing the recursive dependency check and allowing unlinked
dependencies in CI for a while with no adverse consequences so enable
them globally now for all users.
This commit is contained in:
Mike McQuaid 2017-04-21 12:32:43 +01:00
parent 6e1faf5b88
commit e41f0bf8c8
1 changed files with 15 additions and 27 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
Library/Homebrew/formula_installer.rb
View File

@ -173,7 +173,6 @@ class FormulaInstaller
EOS EOS
end end
if ENV["HOMEBREW_CHECK_RECURSIVE_VERSION_DEPENDENCIES"]
version_hash = {} version_hash = {}
version_conflicts = Set.new version_conflicts = Set.new
recursive_formulae.each do |f| recursive_formulae.each do |f|
@ -191,17 +190,6 @@ class FormulaInstaller
View these with `brew deps --tree #{formula.full_name}`. View these with `brew deps --tree #{formula.full_name}`.
EOS EOS
end end
end
unless ENV["HOMEBREW_NO_CHECK_UNLINKED_DEPENDENCIES"]
unlinked_deps = recursive_formulae.select do |dep|
dep.installed? && !dep.keg_only? && !dep.linked_keg.directory?
end
unless unlinked_deps.empty?
raise CannotInstallFormulaError, "You must `brew link #{unlinked_deps*" "}` before #{formula.full_name} can be installed"
end
end
pinned_unsatisfied_deps = recursive_deps.select do |dep| pinned_unsatisfied_deps = recursive_deps.select do |dep|
dep.to_formula.pinned? && !dep.satisfied?(inherited_options_for(dep)) dep.to_formula.pinned? && !dep.satisfied?(inherited_options_for(dep))