From de24d731725a0a6be1339fbdd879f12a136c037d Mon Sep 17 00:00:00 2001 From: Jack Nagel Date: Mon, 23 Jun 2014 22:00:33 -0500 Subject: [PATCH] Use Keg#uninstall to uninstall kegs --- Library/Homebrew/cmd/uninstall.rb | 7 +++++-- Library/Homebrew/test/test_keg.rb | 20 ++++++++++---------- 2 files changed, 15 insertions(+), 12 deletions(-) diff --git a/Library/Homebrew/cmd/uninstall.rb b/Library/Homebrew/cmd/uninstall.rb index bfc521ff0b..a7b426c0c4 100644 --- a/Library/Homebrew/cmd/uninstall.rb +++ b/Library/Homebrew/cmd/uninstall.rb @@ -22,8 +22,11 @@ module Homebrew if rack.directory? puts "Uninstalling #{name}..." - rack.subdirs.each { |d| Keg.new(d).unlink } - rack.rmtree + rack.subdirs.each do |d| + keg = Keg.new(d) + keg.unlink + keg.uninstall + end end rm_opt_link name diff --git a/Library/Homebrew/test/test_keg.rb b/Library/Homebrew/test/test_keg.rb index e73b3efe64..7fd2cef6f7 100644 --- a/Library/Homebrew/test/test_keg.rb +++ b/Library/Homebrew/test/test_keg.rb @@ -25,6 +25,16 @@ class LinkTests < Homebrew::TestCase mkpath HOMEBREW_PREFIX/"lib" end + def teardown + @keg.unlink + @keg.uninstall + + $stdout = @old_stdout + + rmtree HOMEBREW_PREFIX/"bin" + rmtree HOMEBREW_PREFIX/"lib" + end + def test_linking_keg assert_equal 3, @keg.link (HOMEBREW_PREFIX/"bin").children.each { |c| assert_predicate c.readlink, :relative? } @@ -115,14 +125,4 @@ class LinkTests < Homebrew::TestCase refute_predicate HOMEBREW_PREFIX/"lib/foo", :directory? refute_predicate HOMEBREW_PREFIX/"lib/foo/.DS_Store", :exist? end - - def teardown - @keg.unlink - @keg.rmtree - - $stdout = @old_stdout - - rmtree HOMEBREW_PREFIX/"bin" - rmtree HOMEBREW_PREFIX/"lib" - end end