From 71a78b0e561c8a34275f8db69fd3a9406b0f7bcc Mon Sep 17 00:00:00 2001 From: Max Howell Date: Fri, 31 Jul 2009 00:55:00 +0100 Subject: FIX don't rmtree an already installed package That's stupid. Also don't fail to rmtree because nothing was installed. --- bin/brew | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/bin/brew b/bin/brew index 2d3dc4da9..b8ab547d8 100755 --- a/bin/brew +++ b/bin/brew @@ -70,8 +70,6 @@ end def install formula require 'keg' - - raise "#{formula.name} already installed!\n\t#{formula.prefix}" if formula.installed? beginning = Time.now @@ -98,6 +96,9 @@ def install formula end end end + + raise "Nothing installed" unless formula.installed? + ohai 'Finishing up' keg=Keg.new formula keg.clean @@ -109,7 +110,7 @@ def install formula end puts "#{keg.path}: "+abv(keg)+", built in #{pretty_duration Time.now-beginning}" rescue Exception - formula.prefix.rmtree + formula.prefix.rmtree if formula.prefix.directory? raise end @@ -211,7 +212,9 @@ begin when 'install' require 'formula' extract_named_args.each do |name| - install Formula.create(name) + f=Formula.create(name) + raise "#{f.name} already installed!\n\t#{f.prefix}" if f.installed? + install f end when 'ln', 'link' -- cgit v1.2.3