formula_installer: check for recommended/optional.

build.without? seemingly always returns true for requirements that are
neither recommended nor optional so check these before deciding not to
install a default formula.

Closes Homebrew/homebrew#31511.
This commit is contained in:
Mike McQuaid 2014-08-11 20:15:25 +01:00
parent 4790d4cba9
commit 1455d83dd8

View File

@ -241,7 +241,7 @@ class FormulaInstaller
def install_requirement_default_formula?(req, build) def install_requirement_default_formula?(req, build)
return false unless req.default_formula? return false unless req.default_formula?
return false if build.without?(req) return false if build.without?(req) && (req.recommended? || req.optional?)
return true unless req.satisfied? return true unless req.satisfied?
pour_bottle? || build_bottle? pour_bottle? || build_bottle?
end end