From f2f68c6beba1cf71026024870f01ed856c7ee5c6 Mon Sep 17 00:00:00 2001
From: Adam Vandenberg <flangy@gmail.com>
Date: Sun, 25 Jul 2010 12:07:41 -0700
Subject: [PATCH] brew doctor - add check for non-standard X11

XQuartz is not known to work with Homebrew, though if you have it working
on your machine let us know.
---
 Library/Homebrew/brew_doctor.rb | 19 +++++++++++++++++++
 1 file changed, 19 insertions(+)

diff --git a/Library/Homebrew/brew_doctor.rb b/Library/Homebrew/brew_doctor.rb
index 9af1f4381e..0022439386 100644
--- a/Library/Homebrew/brew_doctor.rb
+++ b/Library/Homebrew/brew_doctor.rb
@@ -63,6 +63,24 @@ def check_for_x11
   end
 end
 
+def check_for_nonstandard_x11
+  return unless File.exists? '/usr/X11'
+  x11 = Pathname.new('/usr/X11')
+  if x11.symlink?
+    puts <<-EOS.undent
+      "/usr/X11" was found, but it is a symlink to:
+        #{x11.resolved_path}
+
+      Homebrew's X11 support has only be tested with Apple's X11,
+      preferably any updates from the latest Xcode package.
+
+      In particular, "XQuartz" is not known to allow Homebrew
+      software require X11 to compile.
+
+    EOS
+  end
+end
+
 def check_for_other_package_managers
   if macports_or_fink_installed?
     puts <<-EOS.undent
@@ -407,6 +425,7 @@ def brew_doctor
     check_gcc_versions
     check_for_other_package_managers
     check_for_x11
+    check_for_nonstandard_x11
     check_access_share_locale
     check_user_path
     check_which_pkg_config