aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMax Howell2009-07-31 00:55:00 +0100
committerMax Howell2009-07-31 00:55:00 +0100
commit71a78b0e561c8a34275f8db69fd3a9406b0f7bcc (patch)
treec523551cbfef6405486d82ef5d13c7746d4b072c
parentec65bb48190aca260280ef4ce90225af3e4d2d13 (diff)
downloadbrew-71a78b0e561c8a34275f8db69fd3a9406b0f7bcc.tar.bz2
FIX don't rmtree an already installed package
That's stupid. Also don't fail to rmtree because nothing was installed.
-rwxr-xr-xbin/brew11
1 files 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'