diff options
| author | Jack Nagel | 2012-02-10 12:56:22 -0600 |
|---|---|---|
| committer | Jack Nagel | 2012-02-10 12:58:24 -0600 |
| commit | a38ae664cc8d62c2d58c8e87dd6d9e1c895423b3 (patch) | |
| tree | 362f25848a21d816d683bc58e32318406bba5de8 | |
| parent | 2d2bc066a5b97b2f380e16839f11889d3654ce32 (diff) | |
| download | homebrew-a38ae664cc8d62c2d58c8e87dd6d9e1c895423b3.tar.bz2 | |
Revert "install: unlink old kegs"
This reverts commit 2eabe2cbc84649696aeb6fa842a70f3794955597.
When Keg#unlink looks for symlinks relative to the keg, it can hit false
positives that actually belong to a different keg and unlink them
anyway. This breaks our "force identical directory symlinks to be shared
real directory" case.
This may be a problem in general with the unlinking code and should be
investigated.
Signed-off-by: Jack Nagel <jacknagel@gmail.com>
| -rw-r--r-- | Library/Homebrew/cmd/install.rb | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/Library/Homebrew/cmd/install.rb b/Library/Homebrew/cmd/install.rb index 4260eaac9..5aa198f31 100644 --- a/Library/Homebrew/cmd/install.rb +++ b/Library/Homebrew/cmd/install.rb @@ -103,10 +103,6 @@ module Homebrew extend self begin fi = FormulaInstaller.new(f) fi.install - # Due to the nature of Keg#unlink, this will remove symlinks from an - # older keg, which may still be present if an uninstallation was done - # via `rm -rf <keg>`; this is desired. - Keg.new("#{f.rack}/#{f.version}").unlink fi.caveats fi.finish rescue FormulaAlreadyInstalledError => e |