From 6c23f9b324be74fa5bc35317c5919010b31fb2a4 Mon Sep 17 00:00:00 2001
From: Adam Vandenberg <flangy@gmail.com>
Date: Tue, 29 Jun 2010 10:21:40 -0700
Subject: [PATCH] brew doctor - X11 check

Tweak wording in X11 check and use the same X11 detection function
as ENV.x11; also move that function into utils.
---
 Library/Homebrew/brew.h.rb      | 4 ----
 Library/Homebrew/brew_doctor.rb | 5 +++--
 Library/Homebrew/utils.rb       | 4 ++++
 3 files changed, 7 insertions(+), 6 deletions(-)

diff --git a/Library/Homebrew/brew.h.rb b/Library/Homebrew/brew.h.rb
index ba3c15f05d..8b091995d5 100644
--- a/Library/Homebrew/brew.h.rb
+++ b/Library/Homebrew/brew.h.rb
@@ -619,7 +619,3 @@ def llvm_build
     $1.to_i
   end
 end
-
-def x11_installed?
-  Pathname.new('/usr/X11/lib/libpng.dylib').exist?
-end
diff --git a/Library/Homebrew/brew_doctor.rb b/Library/Homebrew/brew_doctor.rb
index 76177d0683..fddd33a443 100644
--- a/Library/Homebrew/brew_doctor.rb
+++ b/Library/Homebrew/brew_doctor.rb
@@ -54,10 +54,11 @@ def check_for_stray_dylibs
 end
 
 def check_for_x11
-  unless File.exists? '/usr/X11/lib/libpng.dylib'
+  unless x11_installed?
+    opoo "X11 not installed."
     puts <<-EOS.undent
       You don't have X11 installed as part of your Xcode installation.
-      This isn't required for all formula, but is expected by some.
+      This isn't required for all formulae, but is expected by some.
 
     EOS
   end
diff --git a/Library/Homebrew/utils.rb b/Library/Homebrew/utils.rb
index a34e108de2..52b85bd191 100644
--- a/Library/Homebrew/utils.rb
+++ b/Library/Homebrew/utils.rb
@@ -276,4 +276,8 @@ def dump_build_env env
     value = env[k]
     puts "#{k}: #{value}" if value
   end
+end
+
+def x11_installed?
+  Pathname.new('/usr/X11/lib/libpng.dylib').exist?
 end
\ No newline at end of file