From bbf48f635d80d882d44cbb409d5e3b9a7fc14922 Mon Sep 17 00:00:00 2001 From: Jack Nagel Date: Tue, 16 Sep 2014 22:32:18 -0500 Subject: Fix variable shadowing bug in requirement expansion Long term we should probably rename the installer's "f" attribute to "formula". --- Library/Homebrew/formula_installer.rb | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'Library') diff --git a/Library/Homebrew/formula_installer.rb b/Library/Homebrew/formula_installer.rb index 8178f9582..bb76a69cd 100644 --- a/Library/Homebrew/formula_installer.rb +++ b/Library/Homebrew/formula_installer.rb @@ -254,9 +254,9 @@ class FormulaInstaller if (req.optional? || req.recommended?) && build.without?(req) Requirement.prune - elsif req.build? && dependent == f && pour_bottle? + elsif req.build? && dependent == self.f && pour_bottle? Requirement.prune - elsif req.build? && dependent != f && install_bottle_for_dep?(dependent, build) + elsif req.build? && dependent != self.f && install_bottle_for_dep?(dependent, build) Requirement.prune elsif install_requirement_default_formula?(req, build) dep = req.to_dependency -- cgit v1.2.3