From 1815e6caa1bed5dfbadc8609d1b62921484f5d98 Mon Sep 17 00:00:00 2001 From: Mike Lapinsky Date: Mon, 7 Jan 2013 13:39:02 -0800 Subject: [PATCH] Don't remove linked formulas when doing a 'cleanup' Closes Homebrew/homebrew#16947. Signed-off-by: Adam Vandenberg --- Library/Homebrew/cmd/cleanup.rb | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/Library/Homebrew/cmd/cleanup.rb b/Library/Homebrew/cmd/cleanup.rb index 59f47493f4..6f078f720c 100644 --- a/Library/Homebrew/cmd/cleanup.rb +++ b/Library/Homebrew/cmd/cleanup.rb @@ -37,11 +37,15 @@ module Homebrew extend self f.rack.children.each do |keg| if File.directory? keg and f.version > Keg.new(keg).version if f.can_cleanup? - if ARGV.dry_run? - puts "Would remove: #{keg}" + if !Keg.new(keg).linked? + if ARGV.dry_run? + puts "Would remove: #{keg}" + else + puts "Removing: #{keg}..." + rm_rf keg + end else - puts "Removing: #{keg}..." - rm_rf keg + opoo "Skipping (old) #{keg} due to it being linked" end else opoo "Skipping (old) keg-only: #{keg}"