diff options
| author | Charlie Sharpsteen | 2011-09-13 19:40:51 -0700 |
|---|---|---|
| committer | Charlie Sharpsteen | 2011-09-13 19:40:51 -0700 |
| commit | c3565c8d4c7927970e172c8a8bd330915ead958d (patch) | |
| tree | daa31b75e7b34e2165df087f50a606f35bb3bbb8 /Library | |
| parent | e45362296e2263e679d3f50a6548db92cf981707 (diff) | |
| download | homebrew-c3565c8d4c7927970e172c8a8bd330915ead958d.tar.bz2 | |
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.
Diffstat (limited to 'Library')
| -rw-r--r-- | Library/Homebrew/formula_installer.rb | 6 |
1 files changed, 4 insertions, 2 deletions
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 |