From 9b4bb3d9e4b1ea9380595d9f673d86b529f6b297 Mon Sep 17 00:00:00 2001 From: Jack Nagel Date: Tue, 17 Jun 2014 00:04:21 -0500 Subject: [PATCH] Fix retry condition when auto-tapping deps Fixes Homebrew/homebrew#30212. --- Library/Homebrew/formula_installer.rb | 13 +++++++++---- 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/Library/Homebrew/formula_installer.rb b/Library/Homebrew/formula_installer.rb index ea853be29e..e96ac69e9f 100644 --- a/Library/Homebrew/formula_installer.rb +++ b/Library/Homebrew/formula_installer.rb @@ -91,10 +91,15 @@ class FormulaInstaller end def verify_deps_exist - f.recursive_dependencies.map(&:to_formula) - rescue TapFormulaUnavailableError => e - Homebrew.install_tap(e.user, e.repo) - retry + begin + f.recursive_dependencies.map(&:to_formula) + rescue TapFormulaUnavailableError => e + if Homebrew.install_tap(e.user, e.repo) + retry + else + raise + end + end rescue FormulaUnavailableError => e e.dependent = f.name raise