From 3473bbc010e9b169181dc180406c97cb2176ecb8 Mon Sep 17 00:00:00 2001 From: Misty De Meo Date: Sun, 16 Sep 2012 14:06:50 -0500 Subject: [PATCH] Make check for stray developer dir more specific It looks like there can be harmless stuff leftover in /Developer/Library, even after Xcode uninstalls older versions. The existence of the uninstall-developer-folder script should be a better indicator. Fixes Homebrew/homebrew#14970. --- Library/Homebrew/cmd/doctor.rb | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/Library/Homebrew/cmd/doctor.rb b/Library/Homebrew/cmd/doctor.rb index d101ab7f6e..f703dd1beb 100644 --- a/Library/Homebrew/cmd/doctor.rb +++ b/Library/Homebrew/cmd/doctor.rb @@ -232,7 +232,9 @@ def check_for_latest_xcode end def check_for_stray_developer_directory - if MacOS::Xcode.version >= "4.3" and File.exist? "/Developer/Library" + # if the uninstaller script isn't there, it's a good guess neither are + # any troublesome leftover Xcode files + if MacOS::Xcode.version >= "4.3" and File.exist? "/Developer/Library/uninstall-developer-folder" return <<-EOS.undent You have leftover files from an older version of Xcode. You should delete them using: