aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMike McQuaid2016-12-10 19:21:17 +0000
committerMike McQuaid2016-12-10 19:21:17 +0000
commitbf9da2532ebf813f447249d621ec6daa51f8f76a (patch)
tree75eb04e7b41d6e1296f8cce88517a0ccb644be93
parent7258557aea4b6a9c6833efe0fc09c6a2145f54f7 (diff)
downloadbrew-bf9da2532ebf813f447249d621ec6daa51f8f76a.tar.bz2
formula_installer: remove dependency_installer.
This is no longer needed as the reason it was added (argument sanitisation) is now done properly. Also, remove `skip_deps_check?` in favour of `ignore_deps?` as it too is no longer needed.
-rw-r--r--Library/Homebrew/formula_installer.rb20
1 files changed, 5 insertions, 15 deletions
diff --git a/Library/Homebrew/formula_installer.rb b/Library/Homebrew/formula_installer.rb
index 65607d88a..bd1cc9379 100644
--- a/Library/Homebrew/formula_installer.rb
+++ b/Library/Homebrew/formula_installer.rb
@@ -60,10 +60,6 @@ class FormulaInstaller
@pour_failed = false
end
- def skip_deps_check?
- ignore_deps?
- end
-
# When no build tools are available and build flags are passed through ARGV,
# it's necessary to interrupt the user before any sort of installation
# can proceed. Only invoked when the user has no developer tools.
@@ -125,7 +121,7 @@ class FormulaInstaller
def prelude
Tab.clear_cache
- verify_deps_exist unless skip_deps_check?
+ verify_deps_exist unless ignore_deps?
lock
check_install_sanity
end
@@ -147,7 +143,7 @@ class FormulaInstaller
def check_install_sanity
raise FormulaInstallationAlreadyAttemptedError, formula if @@attempted.include?(formula)
- return if skip_deps_check?
+ return if ignore_deps?
recursive_deps = formula.recursive_dependencies
unlinked_deps = recursive_deps.map(&:to_formula).select do |dep|
@@ -197,7 +193,7 @@ class FormulaInstaller
raise BuildToolsError, [formula]
end
- unless skip_deps_check?
+ unless ignore_deps?
deps = compute_dependencies
check_dependencies_bottled(deps) if pour_bottle? && !DevelopmentTools.installed?
install_dependencies(deps)
@@ -250,7 +246,7 @@ class FormulaInstaller
onoe e.message
opoo "Bottle installation failed: building from source."
raise BuildToolsError, [formula] unless DevelopmentTools.installed?
- compute_and_install_dependencies unless skip_deps_check?
+ compute_and_install_dependencies unless ignore_deps?
else
@poured_bottle = true
end
@@ -442,12 +438,6 @@ class FormulaInstaller
@show_header = true unless deps.empty?
end
- class DependencyInstaller < FormulaInstaller
- def skip_deps_check?
- true
- end
- end
-
def install_dependency(dep, inherited_options)
df = dep.to_formula
tab = Tab.for_formula(df)
@@ -463,7 +453,7 @@ class FormulaInstaller
installed_keg.rename(tmp_keg)
end
- fi = DependencyInstaller.new(df)
+ fi = FormulaInstaller.new(df)
fi.options |= tab.used_options
fi.options |= Tab.remap_deprecated_options(df.deprecated_options, dep.options)
fi.options |= inherited_options