From c3565c8d4c7927970e172c8a8bd330915ead958d Mon Sep 17 00:00:00 2001 From: Charlie Sharpsteen Date: Tue, 13 Sep 2011 19:40:51 -0700 Subject: formula_installer.rb: Call link before check_PATH Otherwise, for formula that making the very first install into `sbin`, the following line will fail with a critical error: bin = (HOMEBREW_PREFIX/bin.basename).realpath.to_s Because `sbin` will not exist until after `link` runs. This isn't very clean as it moves `check_PATH` away from the other checks in `caveats`, but I couldn't think of a better hotfix that did not require a bit of code shuffling and re-writing. --- Library/Homebrew/formula_installer.rb | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) (limited to 'Library') diff --git a/Library/Homebrew/formula_installer.rb b/Library/Homebrew/formula_installer.rb index e44efa8dd..989bf565e 100644 --- a/Library/Homebrew/formula_installer.rb +++ b/Library/Homebrew/formula_installer.rb @@ -66,7 +66,6 @@ class FormulaInstaller ohai 'Caveats', f.keg_only_text @show_summary_heading = true else - check_PATH check_manpages check_infopages check_jars @@ -77,7 +76,10 @@ class FormulaInstaller def finish ohai 'Finishing up' if ARGV.verbose? - link unless f.keg_only? + unless f.keg_only? + link + check_PATH + end fix_install_names ohai "Summary" if ARGV.verbose? or show_summary_heading -- cgit v1.2.3