diff options
| author | Charlie Sharpsteen | 2011-09-13 19:40:51 -0700 |
|---|---|---|
| committer | Charlie Sharpsteen | 2011-09-13 19:40:51 -0700 |
| commit | 555b9d149f862bf687481816349e2c9739c1220b (patch) | |
| tree | f2a8ec2620e008ebc801f285037dd0d1bd790f2b /Library | |
| parent | 5790935b1267bbd8dac3fd2564b3dd9c46b5dae9 (diff) | |
| download | brew-555b9d149f862bf687481816349e2c9739c1220b.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 |