diff options
| author | Jack Nagel | 2015-04-17 22:58:02 -0400 | 
|---|---|---|
| committer | Jack Nagel | 2015-04-17 22:58:02 -0400 | 
| commit | 6bca860118938fdf98e04e4004840ac2f9bc12d3 (patch) | |
| tree | 5bb0a38625cf351484380bfaed1fc1324c06c44b /Library/Homebrew/cmd/cleanup.rb | |
| parent | 0d8748c4600f495de4b32cb5f8e4607cf3d28497 (diff) | |
| download | homebrew-6bca860118938fdf98e04e4004840ac2f9bc12d3.tar.bz2 | |
Ask the formula if it can be cleaned up once
Diffstat (limited to 'Library/Homebrew/cmd/cleanup.rb')
| -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 | 
