diff options
| author | Jack Nagel | 2015-04-17 22:58:02 -0400 |
|---|---|---|
| committer | Jack Nagel | 2015-04-17 22:58:02 -0400 |
| commit | 338a08d8d6f10596593bfdf5ebc80e83c390069f (patch) | |
| tree | e9fdd64ba332e3085ef3ea7409a1d447e97ad738 | |
| parent | 505d06c1761dfb89961b4f6b8093c5d62248e5fb (diff) | |
| download | brew-338a08d8d6f10596593bfdf5ebc80e83c390069f.tar.bz2 | |
Ask the formula if it can be cleaned up once
| -rw-r--r-- | Library/Homebrew/cmd/cleanup.rb | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/Library/Homebrew/cmd/cleanup.rb b/Library/Homebrew/cmd/cleanup.rb index 3c133cf13..850cd45e5 100644 --- a/Library/Homebrew/cmd/cleanup.rb +++ b/Library/Homebrew/cmd/cleanup.rb @@ -42,12 +42,10 @@ module Homebrew def cleanup_formula f if f.installed? eligible_kegs = f.rack.subdirs.map { |d| Keg.new(d) }.select { |k| f.pkg_version > k.version } - eligible_kegs.each do |keg| - if f.can_cleanup? - cleanup_keg(keg) - else - opoo "Skipping (old) keg-only: #{keg}" - end + if eligible_kegs.any? && f.can_cleanup? + eligible_kegs.each { |keg| cleanup_keg(keg) } + else + eligible_kegs.each { |keg| opoo "Skipping (old) keg-only: #{keg}" } end elsif f.rack.subdirs.length > 1 # If the cellar only has one version installed, don't complain |