Update formulae for XQuartz compatibility

Formula/aalib.rb

@@ -5,10 +5,7 @@ class Aalib < Formula
   url 'http://downloads.sourceforge.net/aa-project/aalib-1.4rc4.tar.gz'
   md5 'd5aa8e9eae07b7441298b5c30490f6a6'
 
-  # Fix malloc/stdlib issue on OS X
-  def patches
-    DATA
-  end
+  depends_on :x11
 
   if MacOS.xcode_version >= "4.3"
     # remove the autoreconf if possible, no comment provided about why it is there
@@ -17,17 +14,21 @@ class Aalib < Formula
     depends_on "libtool" => :build
   end
 
+  # Fix malloc/stdlib issue on OS X
+  def patches
+    DATA
+  end
+
   def install
-    # Build fails some of the time without `ENV.x11`!
-    # See: https://github.com/mxcl/homebrew/pull/10356
-    ENV.x11
     ENV.ncurses_define
     system 'autoreconf --force --install'
-    system "./configure", "--disable-debug", "--disable-dependency-tracking",
+    system "./configure", "--disable-debug",
+                          "--disable-dependency-tracking",
                           "--prefix=#{prefix}",
                           "--mandir=#{man}",
                           "--infodir=#{info}",
-                          "--enable-shared=yes", "--enable-static=yes"
+                          "--enable-shared=yes",
+                          "--enable-static=yes"
     system "make install"
   end
 end

Formula/abiword.rb

@@ -10,6 +10,7 @@ class Abiword < Formula
     sha1 '34a6e4e9c5619e8f2d619ac844519fc9378405b3'
   end
 
+  depends_on :libpng
   depends_on 'jpeg'
   depends_on 'fribidi'
   depends_on 'libgsf'
@@ -20,7 +21,6 @@ class Abiword < Formula
   depends_on 'imagemagick'
 
   def install
-    ENV.libpng
     system "./configure", "--disable-debug", "--disable-dependency-tracking",
                           "--prefix=#{prefix}"
     system "make install"

Formula/aview.rb

@@ -6,14 +6,15 @@ class Aview < Formula
   md5 '093f298e7787591e229b59d039c72f4d'
 
   depends_on 'aalib'
+  depends_on :x11
 
   def patches
     DATA
   end
 
   def install
-    ENV.x11
-    system "./configure", "--disable-debug", "--disable-dependency-tracking",
+    system "./configure", "--disable-debug",
+                          "--disable-dependency-tracking",
                           "--prefix=#{prefix}",
                           "--mandir=#{man}"
     system "make install"

Formula/bsdmake.rb

@@ -32,7 +32,7 @@ class Bsdmake < Formula
                  '@PREFIX@', prefix
 
     # X11 path shouldn't be munged
-    inreplace 'mk/sys.mk', '@PREFIX@', '/usr/X11'
+    inreplace 'mk/sys.mk', '@PREFIX@', MacOS.x11_prefix
 
     inreplace 'mk/bsd.own.mk' do |s|
       s.gsub! '@INSTALL_USER@', `id -un`.chomp

Formula/clusterit.rb

@@ -5,8 +5,9 @@ class Clusterit < Formula
   url 'http://downloads.sourceforge.net/project/clusterit/clusterit/clusterit-2.5/clusterit-2.5.tar.gz'
   md5 'f0e772e07122e388de629fb57f7237ab'
 
+  depends_on :x11
+
   def install
-    ENV.x11
     system "./configure", "--disable-debug", "--disable-dependency-tracking",
                           "--prefix=#{prefix}"
     system "make install"

Formula/clutter.rb

@@ -7,12 +7,11 @@ class Clutter < Formula
 
   depends_on 'pkg-config' => :build
   depends_on 'atk'
-  # Cairo is keg-only and usually only used for Leopard builds.
-  # But Clutter requires a newer version of Cairo that what comes with Snow Leopard.
-  depends_on 'cairo'
   depends_on 'intltool'
   depends_on 'json-glib'
   depends_on 'pango'
+  depends_on 'cairo' # for cairo-gobject
+  depends_on :x11
 
   def install
     system "./configure", "--prefix=#{prefix}",

Formula/dcmtk.rb

@@ -5,6 +5,7 @@ class Dcmtk < Formula
   url 'ftp://dicom.offis.de/pub/dicom/offis/software/dcmtk/dcmtk360/dcmtk-3.6.0.tar.gz'
   md5 '19409e039e29a330893caea98715390e'
 
+  depends_on :x11
   depends_on 'libtiff'
   depends_on 'doxygen' if ARGV.include? '--with-docs'
 
@@ -18,7 +19,6 @@ class Dcmtk < Formula
 
   def install
     ENV.m64 if MacOS.prefer_64_bit?
-    ENV.x11
 
     args = std_cmake_args
     args << '-DDCMTK_WITH_DOXYGEN=YES' if ARGV.include? '--with-docs'

Formula/devil.rb

@@ -5,6 +5,7 @@ class Devil < Formula
   homepage 'http://sourceforge.net/projects/openil/'
   md5 '7918f215524589435e5ec2e8736d5e1d'
 
+  depends_on :x11
   depends_on 'jpeg'
   depends_on 'libtiff'
   depends_on 'little-cms'
@@ -22,9 +23,6 @@ class Devil < Formula
   def patches; DATA; end
 
   def install
-    # devil won't find libpng without ENV.x11
-    ENV.x11
-
     system "./configure", "--disable-debug", "--disable-dependency-tracking",
                           "--prefix=#{prefix}", "--enable-ILU"
     system "make install"

Formula/dia.rb

@@ -12,6 +12,7 @@ class Dia < Formula
   depends_on 'pango'
   depends_on 'libtiff'
   depends_on 'gtk+'
+  depends_on :x11
 
   def patches
     # fixes compilation with glib 2.31+
@@ -21,7 +22,6 @@ class Dia < Formula
   end
 
   def install
-    ENV.x11
     # fix for Leopard, potentially others with isspecial defined elswhere
     inreplace 'objects/GRAFCET/boolequation.c', 'isspecial', 'char_isspecial'
     system "./configure", "--enable-debug=no",

Formula/dvdauthor.rb

@@ -10,6 +10,7 @@ class Dvdauthor < Formula
 
   depends_on 'pkg-config' => :build
   depends_on 'libdvdread'
+  depends_on :x11
 
   # Fix build with png-1.5. Patch has been applied upstream, but no tagged
   # release has been made since 2010. See:
@@ -20,7 +21,6 @@ class Dvdauthor < Formula
   end
 
   def install
-    ENV.x11 # For libpng, etc.
     system "./configure", "--disable-dependency-tracking",
                           "--prefix=#{prefix}",
                           "--mandir=#{man}"

Formula/dzen2.rb

@@ -5,8 +5,9 @@ class Dzen2 < Formula
   homepage 'http://sites.google.com/site/gotmor/dzen'
   md5 '5978620c2124c8a8ad52d7f17ce94fd7'
 
+  depends_on :x11
+
   def install
-    ENV.x11
     ENV.append 'LDFLAGS', '-lX11 -lXinerama -lXpm'
     ENV.append_to_cflags '-DVERSION=\"${VERSION}\" -DDZEN_XINERAMA -DDZEN_XPM'
 

Formula/emacs.rb

@@ -12,6 +12,8 @@ class Emacs < Formula
     head 'bzr://http://bzr.savannah.gnu.org/r/emacs/trunk'
   end
 
+  depends_on :x11 if ARGV.include? "--with-x"
+
   # Stripping on Xcode 4 causes malformed object errors.
   # Just skip everything.
   skip_clean :all
@@ -69,7 +71,6 @@ class Emacs < Formula
       prefix.install "nextstep/Emacs.app"
     else
       if ARGV.include? "--with-x"
-        ENV.x11
         # These libs are not specified in xft's .pc. See:
         # https://trac.macports.org/browser/trunk/dports/editors/emacs/Portfile#L74
         # https://github.com/mxcl/homebrew/issues/8156

Formula/feh.rb

@@ -6,10 +6,9 @@ class Feh < Formula
   md5 '7dee285e8dd34f69058b0977283b3a8a'
 
   depends_on 'giblib' => :build
+  depends_on :x11
 
   def install
-    ENV.x11
-
     system "make", "PREFIX=#{prefix}"
     system "make", "PREFIX=#{prefix}", "install"
   end

Formula/ffmbc.rb

@@ -6,6 +6,7 @@ class Ffmbc < Formula
   md5 '547bb7b7963224dd66dffa8b25e623b3'
 
   depends_on 'yasm' => :build
+  depends_on :x11
   depends_on 'x264' => :optional
   depends_on 'faac' => :optional
   depends_on 'lame' => :optional
@@ -16,7 +17,6 @@ class Ffmbc < Formula
   depends_on 'xvid' => :optional
 
   def install
-    ENV.x11
     args = ["--prefix=#{prefix}",
             "--disable-debug",
             "--disable-shared",

Formula/ffmpeg.rb

@@ -16,6 +16,7 @@ class Ffmpeg < Formula
   head 'git://git.videolan.org/ffmpeg.git'
 
   depends_on 'yasm' => :build
+  depends_on :x11
   depends_on 'x264' => :optional
   depends_on 'faac' => :optional
   depends_on 'lame' => :optional
@@ -41,7 +42,6 @@ class Ffmpeg < Formula
   end
 
   def install
-    ENV.x11
     args = ["--prefix=#{prefix}",
             "--enable-shared",
             "--enable-gpl",

Formula/fltk.rb

@@ -11,6 +11,7 @@ class Fltk < Formula
     version '1.3.x-r9327'
   end
 
+  depends_on :libpng
   depends_on 'jpeg'
 
   fails_with :clang do
@@ -19,7 +20,6 @@ class Fltk < Formula
   end
 
   def install
-    ENV.libpng
     system "./configure", "--prefix=#{prefix}", "--enable-threads"
     system "make install"
   end

Formula/fontconfig.rb

@@ -9,9 +9,9 @@ class Fontconfig < Formula
     "Leopard comes with version 2.4.x, which is too old for many packages."
 
   depends_on 'pkg-config' => :build
+  depends_on :x11
 
   def install
-    ENV.x11
     system "./configure", "--disable-dependency-tracking", "--prefix=#{prefix}"
     system "make install"
   end

Formula/fontforge.rb

@@ -11,6 +11,7 @@ class Fontforge < Formula
   depends_on 'gettext'
   depends_on 'pango'
   depends_on 'potrace'
+  depends_on :x11
 
   def options
     [['--without-python', 'Build without Python.']]
@@ -28,7 +29,6 @@ class Fontforge < Formula
 
     args << "--without-python" if ARGV.include? "--without-python"
 
-    ENV.x11
     # Fix linker error; see: http://trac.macports.org/ticket/25012
     ENV.append "LDFLAGS", "-lintl"
     system "./configure", *args

Formula/fox.rb

@@ -11,17 +11,17 @@ class Fox < Formula
     md5 '1cf2607d15ffad5b664cf65bfcd249bc'
   end
 
+  depends_on :x11
+
   fails_with :llvm do
     cause "Inline asm errors during build"
   end if ARGV.build_devel?
 
   def install
-    ENV.x11
-
     # Yep, won't find freetype unless this is all set.
-    ENV.append "CFLAGS", "-I/usr/X11/include/freetype2"
-    ENV.append "CPPFLAGS", "-I/usr/X11/include/freetype2"
-    ENV.append "CXXFLAGS", "-I/usr/X11/include/freetype2"
+    ENV.append "CFLAGS", "-I#{MacOS.x11_prefix}/include/freetype2"
+    ENV.append "CPPFLAGS", "-I#{MacOS.x11_prefix}/include/freetype2"
+    ENV.append "CXXFLAGS", "-I#{MacOS.x11_prefix}/include/freetype2"
 
     system "./configure", "--enable-release",
                           "--prefix=#{prefix}",

Formula/ftgl.rb

@@ -6,10 +6,9 @@ class Ftgl < Formula
   md5 'fcf4d0567b7de9875d4e99a9f7423633'
 
   depends_on 'pkg-config' => :build
+  depends_on :x11
 
   def install
-    ENV.x11 # Put freetype-config in path
-
     # If doxygen is installed, the docs may still fail to build.
     # So we disable building docs.
     inreplace "configure", "set dummy doxygen;", "set dummy no_doxygen;"

Formula/gd.rb

@@ -1,13 +1,14 @@
 require 'formula'
 
 class Gd < Formula
-  url "http://www.libgd.org/releases/gd-2.0.36RC1.tar.gz"
-  homepage "http://bitbucket.org/pierrejoye/gd-libgd"
-  mirror "http://download.osgeo.org/mapserver/libgd/gd-2.0.36RC1.tar.gz"
-  md5 "39ac48e6d5e0012a3bd2248a0102f209"
+  homepage 'http://bitbucket.org/pierrejoye/gd-libgd'
+  url 'http://www.libgd.org/releases/gd-2.0.36RC1.tar.gz'
+  mirror 'http://download.osgeo.org/mapserver/libgd/gd-2.0.36RC1.tar.gz'
+  md5 '39ac48e6d5e0012a3bd2248a0102f209'
 
-  head "http://bitbucket.org/pierrejoye/gd-libgd", :using => :hg
+  head 'http://bitbucket.org/pierrejoye/gd-libgd', :using => :hg
 
+  depends_on :x11
   depends_on 'jpeg' => :recommended
 
   fails_with :llvm do
@@ -16,8 +17,7 @@ class Gd < Formula
   end
 
   def install
-    ENV.x11
-    system "./configure", "--prefix=#{prefix}", "--with-freetype=/usr/X11"
+    system "./configure", "--prefix=#{prefix}", "--with-freetype=#{MacOS.x11_prefix}"
     system "make install"
     (lib+'pkgconfig/gdlib.pc').write pkg_file
   end

Formula/gdal.rb

@@ -39,6 +39,8 @@ class Gdal < Formula
   # For creating up to date man pages.
   depends_on 'doxygen' => :build if ARGV.build_head?
 
+  depends_on :x11
+
   depends_on 'jpeg'
   depends_on 'giflib'
   depends_on 'proj'
@@ -108,7 +110,7 @@ class Gdal < Formula
 
       # Backends supported by OS X.
       "--with-libz=/usr",
-      "--with-png=/usr/X11",
+      "--with-png=#{MacOS.x11_prefix}",
       "--with-expat=/usr",
 
       # Default Homebrew backends.

Formula/gdk-pixbuf.rb

@@ -10,6 +10,7 @@ class GdkPixbuf < Formula
   depends_on 'glib'
   depends_on 'jasper'
   depends_on 'libtiff'
+  depends_on :x11
 
   # 'loaders.cache' must be writable by other packages
   skip_clean 'lib/gdk-pixbuf-2.0'

Formula/geda-gaf.rb

@@ -15,6 +15,7 @@ class GedaGaf < Formula
   depends_on 'gtk+'
   depends_on 'guile'
   depends_on 'gawk'
+  depends_on :x11
 
   # MacPorts fix for glib 2.32 includes
   # Needed for 1.6.2 and 1.7.x
@@ -24,7 +25,6 @@ class GedaGaf < Formula
 
   def install
     # Help configure find libraries
-    ENV.x11
     gettext = Formula.factory('gettext')
 
     system "./configure", "--prefix=#{prefix}",

Formula/geomview.rb

@@ -5,14 +5,13 @@ class Geomview < Formula
   homepage 'http://www.geomview.org'
   md5 '29c7e6d678af7b9968980f92954419bb'
 
+  depends_on :x11
   depends_on 'lesstif'
 
   def install
-    ENV.x11
-
-    system "./configure", "--disable-debug", "--disable-dependency-tracking",
+    system "./configure", "--disable-debug",
+                          "--disable-dependency-tracking",
                           "--prefix=#{prefix}"
-
     system "make install"
   end
 end

Formula/gerbv.rb

@@ -8,15 +8,13 @@ class Gerbv < Formula
   depends_on 'pkg-config' => :build
   depends_on 'gtk+'
   depends_on 'cairo' if MacOS.leopard?
+  depends_on :x11
 
   def install
-    system "./configure", "--disable-debug", "--disable-dependency-tracking",
+    system "./configure", "--disable-debug",
+                          "--disable-dependency-tracking",
                           "--prefix=#{prefix}",
                           "--disable-update-desktop-database"
     system "make install"
   end
-
-  def caveats
-    "Note: gerbv is an X11 application."
-  end
 end

Formula/ggobi.rb

@@ -12,6 +12,7 @@ class Ggobi < Formula
   depends_on 'fontconfig'
   depends_on 'pango'
   depends_on 'gettext'
+  depends_on :x11
 
   def install
     # Necessary for ggobi to build - based on patch from MacPorts

Formula/ghostscript.rb

@@ -18,6 +18,7 @@ class Ghostscript < Formula
   depends_on 'libtiff'
   depends_on 'jbig2dec'
   depends_on 'little-cms2'
+  depends_on :libpng
 
   if ARGV.build_head? and MacOS.xcode_version >= "4.3"
     depends_on "automake" => :build
@@ -36,10 +37,9 @@ class Ghostscript < Formula
   end
 
   def install
-    ENV.libpng
     ENV.deparallelize
     # ghostscript configure ignores LDFLAGs apparently
-    ENV['LIBS'] = "-L/usr/X11/lib"
+    ENV['LIBS'] = "-L#{MacOS.x11_prefix}/lib"
 
     src_dir = ARGV.build_head? ? "gs" : "."
 

Formula/gifsicle.rb

@@ -5,8 +5,9 @@ class Gifsicle < Formula
   url 'http://www.lcdf.org/gifsicle/gifsicle-1.64.tar.gz'
   md5 'dabe9ee0d6d9cea099d9a7e6ecdcc443'
 
+  depends_on :x11
+
   def install
-    ENV.x11
     system "./configure", "--disable-dependency-tracking",
                           "--prefix=#{prefix}",
                           "--enable-all"

Formula/glade.rb

@@ -11,6 +11,7 @@ class Glade < Formula
   depends_on 'intltool'
   depends_on 'libglade'
   depends_on 'hicolor-icon-theme'
+  depends_on :x11
 
   def install
     pydir = 'python' + `python -c 'import sys;print(sys.version[:3])'`.strip

Formula/gle.rb

@@ -7,12 +7,11 @@ class Gle < Formula
   md5 '5eef0a63ee0077237b8a36fe1a24407f'
 
   depends_on 'pkg-config' => :build
+  depends_on :x11
   depends_on 'jpeg' => :optional
   depends_on 'libtiff' => :optional
 
   def install
-    ENV.x11
-
     arch = MacOS.prefer_64_bit? ? "x86_64" : "i386"
     system "./configure", "--prefix=#{prefix}",
                           "--with-arch=#{arch}",

Formula/gnu-shogi.rb

@@ -5,6 +5,8 @@ class GnuShogi < Formula
   homepage 'http://www.cs.caltech.edu/~mvanier/hacking/gnushogi/gnushogi.html'
   md5 'a18eae93afc89bfd368ed5f6768be791'
 
+  depends_on :x11
+
   def patches
       # gcc 4.2.1 flags the following error
       # In file included from attacks.c:32:
@@ -21,8 +23,8 @@ class GnuShogi < Formula
   def install
     system "./configure", "--disable-debug", "--disable-dependency-tracking",
                           "--prefix=#{prefix}",
-                          "--x-include=/usr/X11/include",
-                          "--x-lib=/usr/X11/lib"
+                          "--x-include=#{MacOS.x11_prefix}/include",
+                          "--x-lib=#{MacOS.x11_prefix}/lib"
     system "make"
     system "make", "install", "MANDIR=#{man6}", "INFODIR=#{info}"
   end

Formula/gnuplot.rb

@@ -9,6 +9,7 @@ class Gnuplot < Formula
   depends_on 'pkg-config' => :build
   depends_on 'readline'
   depends_on 'pango'
+  depends_on :x11
   depends_on 'pdflib-lite' if ARGV.include? "--pdf"
   depends_on 'lua' unless ARGV.include? '--nolua'
   depends_on 'gd' unless ARGV.include? "--nogd"
@@ -30,7 +31,6 @@ class Gnuplot < Formula
 
   def install
     # Help configure find libraries
-    ENV.x11
     readline = Formula.factory 'readline'
     pdflib = Formula.factory 'pdflib-lite'
     gd = Formula.factory 'gd'

Formula/gobby.rb

@@ -16,9 +16,9 @@ class Gobby < Formula
   depends_on 'gettext'
   depends_on 'hicolor-icon-theme'
   depends_on 'libinfinity'
+  depends_on :x11
 
   def install
-    ENV.x11
     system "./configure", "--disable-dependency-tracking",
                           "--prefix=#{prefix}"
     system "make install"

Formula/gource.rb

@@ -9,6 +9,7 @@ class Gource < Formula
   depends_on 'glm' => :build
   depends_on 'pkg-config' => :build
 
+  depends_on :x11 # for Freetype
   depends_on 'boost'
   depends_on 'glew'
   depends_on 'jpeg'
@@ -22,8 +23,6 @@ class Gource < Formula
   end
 
   def install
-    ENV.x11 # Put freetype-config in path
-
     # For non-/usr/local installs
     ENV.append "CXXFLAGS", "-I#{HOMEBREW_PREFIX}/include"
 

Formula/gpac.rb

@@ -17,6 +17,8 @@ class Gpac < Formula
 
   head 'https://gpac.svn.sourceforge.net/svnroot/gpac/trunk/gpac', :using => :svn
 
+  depends_on :x11
+
   depends_on 'a52dec' => :optional
   depends_on 'jpeg' => :optional
   depends_on 'faad2' => :optional
@@ -35,9 +37,9 @@ class Gpac < Formula
             "--prefix=#{prefix}",
             "--mandir=#{man}",
             # gpac build system is barely functional
-            "--extra-cflags=-I/usr/X11/include",
+            "--extra-cflags=-I#{MacOS.x11_prefix}/include",
             # Force detection of X libs on 64-bit kernel
-            "--extra-ldflags=-L/usr/X11/lib"]
+            "--extra-ldflags=-L#{MacOS.x11_prefix}/lib"]
 
     system "chmod +x configure"
     system "./configure", *args

Formula/grace.rb

@@ -5,6 +5,7 @@ class Grace < Formula
   url 'ftp://plasma-gate.weizmann.ac.il/pub/grace/src/grace5/grace-5.1.22.tar.gz'
   sha1 '3ce88c7c264d2de73b8935f835a1c1f8e368c78f'
 
+  depends_on :x11
   depends_on 'pdflib-lite'
   depends_on 'jpeg'
   depends_on 'fftw'
@@ -17,7 +18,6 @@ class Grace < Formula
   end
 
   def install
-    ENV.x11
     system "./configure", "--disable-debug",
                           "--disable-dependency-tracking",
                           "--prefix=#{prefix}",

Formula/graphicsmagick.rb

@@ -29,6 +29,7 @@ class Graphicsmagick < Formula
 
   head 'hg://http://graphicsmagick.hg.sourceforge.net:8000/hgroot/graphicsmagick/graphicsmagick'
 
+  depends_on :x11
   depends_on 'jpeg'
   depends_on 'libwmf' if use_wmf?
   depends_on 'libtiff' => :optional
@@ -55,8 +56,6 @@ class Graphicsmagick < Formula
   end
 
   def install
-    ENV.x11
-
     # versioned stuff in main tree is pointless for us
     inreplace 'configure', '${PACKAGE_NAME}-${PACKAGE_VERSION}', '${PACKAGE_NAME}'
 

Formula/graphviz.rb

@@ -28,7 +28,6 @@ class Graphviz < Formula
   end
 
   def install
-    ENV.x11
     args = ["--disable-debug",
             "--disable-dependency-tracking",
             "--prefix=#{prefix}",

Formula/grass.rb

@@ -31,8 +31,8 @@ class Grass < Formula
   depends_on "libtiff"
   depends_on "unixodbc"
   depends_on "fftw"
-
   depends_on "cairo" if MacOS.leopard?
+  depends_on :x11
 
   # Patches ensure 32 bit system python is used for wxWidgets and that files
   # are not installed outside of the prefix.
@@ -60,7 +60,7 @@ class Grass < Formula
 
     args = [
       "--disable-debug", "--disable-dependency-tracking",
-      "--with-libs=/usr/X11/lib #{HOMEBREW_PREFIX}/lib",
+      "--with-libs=#{MacOS.x11_prefix}/lib #{HOMEBREW_PREFIX}/lib",
       "--with-includes=#{HOMEBREW_PREFIX}/include",
       "--enable-largefile",
       "--enable-shared",
@@ -74,7 +74,7 @@ class Grass < Formula
       "--with-sqlite",
       "--with-odbc",
       "--with-geos=#{HOMEBREW_PREFIX}/bin/geos-config",
-      "--with-png-includes=/usr/X11/include",
+      "--with-png-includes=#{MacOS.x11_prefix}/include",
       "--with-png",
       "--with-readline-includes=#{readline.include}",
       "--with-readline-libs=#{readline.lib}",
@@ -82,7 +82,7 @@ class Grass < Formula
       "--with-nls-includes=#{gettext.include}",
       "--with-nls-libs=#{gettext.lib}",
       "--with-nls",
-      "--with-freetype-includes=/usr/X11/include /usr/X11/include/freetype2",
+      "--with-freetype-includes=#{MacOS.x11_prefix}/include #{MacOS.x11_prefix}/include/freetype2",
       "--with-freetype",
       "--without-tcltk" # Disabled due to compatibility issues with OS X Tcl/Tk
     ]
@@ -106,7 +106,7 @@ class Grass < Formula
       args << "--with-cairo-includes=#{cairo.include}/cairo"
       args << "--with-cairo-libs=#{cairo.lib}"
     else
-      args << "--with-cairo-includes=/usr/X11/include /usr/X11/include/cairo"
+      args << "--with-cairo-includes=#{MacOS.x11_prefix}/include #{MacOS.x11_prefix}/include/cairo"
     end
 
     args << "--with-cairo"

Formula/gtk+.rb

@@ -14,6 +14,7 @@ class Gtkx < Formula
   depends_on 'pango'
   depends_on 'jasper' => :optional
   depends_on 'atk' => :optional
+  depends_on :x11
 
   fails_with :llvm do
     build 2326

Formula/gtk-chtheme.rb

@@ -11,6 +11,7 @@ class GtkChtheme < Formula
   depends_on 'glib'
   depends_on 'gtk+'
   depends_on 'pango'
+  depends_on :x11
 
   def install
     # Unfortunately chtheme relies on some deprecated functionality

Formula/gtkglext.rb

@@ -8,6 +8,7 @@ class Gtkglext < Formula
   depends_on 'pkg-config' => :build
   depends_on 'glib'
   depends_on 'gtk+'
+  depends_on :x11
 
   def patches
       # fixes an incompatibility with recent GTK versions
@@ -16,7 +17,6 @@ class Gtkglext < Formula
   end
 
   def install
-    ENV.x11
     system "./configure", "--disable-debug", "--disable-dependency-tracking",
                           "--prefix=#{prefix}"
     system "make install"

Formula/gtkglextmm.rb

@@ -8,6 +8,7 @@ class Gtkglextmm < Formula
   depends_on 'pkg-config' => :build
   depends_on 'gtkglext'
   depends_on 'gtkmm'
+  depends_on :x11
 
   # Fixes header include order, using patch from macports: https://trac.macports.org/ticket/27059
   # Upstream bug report: https://bugzilla.gnome.org/show_bug.cgi?id=636831

Formula/gtkmm.rb

@@ -11,6 +11,7 @@ class Gtkmm < Formula
   depends_on 'libsigc++'
   depends_on 'pangomm'
   depends_on 'atkmm'
+  depends_on :x11
 
   def install
     system "./configure", "--disable-dependency-tracking", "--prefix=#{prefix}"

Formula/gtkwave.rb

@@ -7,6 +7,7 @@ class Gtkwave < Formula
 
   depends_on 'gtk+'
   depends_on 'xz'
+  depends_on :x11
 
   def install
     system './configure', "--prefix=#{prefix}", '--disable-dependency-tracking'

Formula/gv.rb

@@ -8,11 +8,11 @@ class Gv < Formula
 
   depends_on 'ghostscript'
   depends_on 'xaw3d'
+  depends_on :x11
 
   skip_clean 'share/gv/safe-gs-workdir'
 
   def install
-    ENV.x11
     system "./configure", "--disable-dependency-tracking",
                           "--prefix=#{prefix}", "--enable-SIGCHLD-fallback"
     system "make install"

Formula/icoutils.rb

@@ -5,8 +5,9 @@ class Icoutils < Formula
   homepage 'http://www.nongnu.org/icoutils/'
   md5 'b58f375e0f8731595e8d0ecdc3a0acb9'
 
+  depends_on :libpng
+
   def install
-    ENV.libpng
     system "./configure", "--disable-dependency-tracking",
                           "--disable-rpath",
                           "--prefix=#{prefix}"

Formula/imagemagick.rb

@@ -63,6 +63,8 @@ class Imagemagick < Formula
   depends_on 'pkg-config' => :build
   depends_on 'jpeg'
 
+  depends_on :x11
+
   depends_on 'ghostscript' => :recommended if ghostscript_srsly?
 
   depends_on 'libtiff' => :optional
@@ -103,8 +105,6 @@ class Imagemagick < Formula
   end
 
   def install
-    ENV.x11 # Add to PATH for freetype-config on Snow Leopard
-
     args = [ "--disable-osx-universal-binary",
              "--without-perl", # I couldn't make this compile
              "--prefix=#{prefix}",

Formula/imake.rb

@@ -11,10 +11,9 @@ class Imake < Formula
   sha1 '1fd3dca267d125ad86583d7f9663b6ff532cddd1'
 
   depends_on 'pkg-config' => :build
+  depends_on :x11
 
   def install
-    # So it can find xorg-macros.pc.  Using ENV.x11 didn't work.
-    ENV['PKG_CONFIG_PATH'] = '/usr/X11/share/pkgconfig'
     ENV.deparallelize
     system './configure', "--prefix=#{prefix}", '--disable-dependency-tracking'
     system "make install"

Formula/imlib2.rb

@@ -6,9 +6,9 @@ class Imlib2 < Formula
   md5 '859e5fede51ec819f4314eee11da3ea5'
 
   depends_on 'pkg-config' => :build
+  depends_on :x11 # for Freetype
 
   def install
-    ENV.x11 # For freetype
     system "./configure", "--disable-dependency-tracking",
                           "--prefix=#{prefix}",
                           "--enable-amd64=no"

Formula/ipe.rb

@@ -9,6 +9,7 @@ class Ipe < Formula
   depends_on 'makeicns' => :build
   depends_on 'lua'
   depends_on 'qt'
+  depends_on :x11
 
   # configure library paths using pkg-config
   # because ipe assumes that Qt and other libs are installed in

Formula/leptonica.rb

@@ -5,6 +5,7 @@ class Leptonica < Formula
   url 'http://www.leptonica.org/source/leptonica-1.68.tar.gz'
   md5 '5cd7092f9ff2ca7e3f3e73bfcd556403'
 
+  depends_on :x11
   depends_on 'jpeg'
   depends_on 'libtiff'
 
@@ -16,7 +17,6 @@ class Leptonica < Formula
   end
 
   def install
-    ENV.x11
     system "./configure", "--disable-dependency-tracking",
                           "--prefix=#{prefix}"
     system "make install"

Formula/lesstif.rb

@@ -1,13 +1,13 @@
 require 'formula'
 
 class Lesstif < Formula
+  homepage 'http://lesstif.sourceforge.net'
   url 'http://sourceforge.net/projects/lesstif/files/lesstif/0.95.2/lesstif-0.95.2.tar.bz2'
   md5 '754187dbac09fcf5d18296437e72a32f'
-  homepage 'http://lesstif.sourceforge.net'
+
+  depends_on :x11
 
   def install
-    ENV.x11
-
     # LessTif does naughty, naughty, things by assuming we want autoconf macros
     # to live in wherever `aclocal --print-ac-dir` says they should.
 

Formula/libagg.rb

@@ -8,6 +8,7 @@ class Libagg < Formula
   depends_on "automake" => :build if MacOS.xcode_version >= "4.3"
   depends_on 'pkg-config' => :build
   depends_on 'sdl'
+  depends_on :x11 # for Freetype
 
   fails_with :clang do
     build 318
@@ -18,8 +19,6 @@ class Libagg < Formula
   end
 
   def install
-    ENV.x11 # For freetype
-
     # AM_C_PROTOTYPES was removed in automake 1.12, as it's only needed for
     # pre-ANSI compilers
     inreplace 'configure.in', 'AM_C_PROTOTYPES', ''

Formula/libass.rb

@@ -7,6 +7,7 @@ class Libass < Formula
 
   depends_on 'pkg-config' => :build
   depends_on 'fribidi'
+  depends_on :x11
 
   def install
     system "./configure", "--disable-dependency-tracking", "--prefix=#{prefix}"

Formula/libcaca.rb

@@ -8,6 +8,7 @@ class Libcaca < Formula
 
   depends_on 'pkg-config' => :build
   depends_on 'imlib2'
+  depends_on :x11
 
   fails_with :llvm do
     cause "Unsupported inline asm: input constraint with a matching output constraint of incompatible type"

Formula/libgaiagraphics.rb

@@ -7,13 +7,11 @@ class Libgaiagraphics < Formula
 
   depends_on 'libgeotiff'
   depends_on 'jpeg'
-
-  # Leopard's Cairo is too old.
   depends_on 'cairo' if MacOS.leopard?
+  depends_on :x11
 
   def install
-    system "./configure", "--disable-dependency-tracking",
-                          "--prefix=#{prefix}"
+    system "./configure", "--disable-dependency-tracking", "--prefix=#{prefix}"
     system "make install"
   end
 end

Formula/libglade.rb

@@ -7,6 +7,7 @@ class Libglade < Formula
 
   depends_on 'libxml2'
   depends_on 'gtk+'
+  depends_on :x11
 
   def install
     ENV.append 'LDFLAGS', '-lgmodule-2.0'

Formula/libharu.rb

@@ -5,6 +5,8 @@ class Libharu < Formula
   url 'http://libharu.org/files/libharu-2.2.1.tar.bz2'
   md5 '4febd7e677b1c5d54db59a608b84e79f'
 
+  depends_on :libpng
+
   def patches
     # Fixes compilation against LibPNG 1.5. Can be removed on next release.
     # Based on a commit in the LibHaru repository which does not apply cleanly
@@ -15,10 +17,11 @@ class Libharu < Formula
   end
 
   def install
-    system "./configure", "--disable-debug", "--disable-dependency-tracking",
+    system "./configure", "--disable-debug",
+                          "--disable-dependency-tracking",
                           "--prefix=#{prefix}",
-                          # ENV.x11 doesn't get picked up
-                          "--with-png=/usr/X11"
+                          # libpng doesn't get picked up
+                          "--with-png=#{MacOS.x11_prefix}"
     system "make install"
   end
 end

Formula/libicns.rb

@@ -6,12 +6,12 @@ class Libicns < Formula
   sha256 '335f10782fc79855cf02beac4926c4bf9f800a742445afbbf7729dab384555c2'
 
   depends_on 'jasper'
+  depends_on :libpng
 
   def install
     # Fix for libpng 1.5 on Lion, may not be needed in head version of libicns
     inreplace 'icnsutils/png2icns.c', 'png_set_gray_1_2_4_to_8', 'png_set_expand_gray_1_2_4_to_8'
 
-    ENV.libpng
     ENV.universal_binary # Also build 32-bit so Wine can use it
 
     system "./configure", "--disable-debug", "--disable-dependency-tracking",

Formula/libkate.rb

@@ -5,6 +5,7 @@ class Libkate < Formula
   url 'http://libkate.googlecode.com/files/libkate-0.4.1.tar.gz'
   sha1 '87fd8baaddb7120fb4d20b0a0437e44ea8b6c9d8'
 
+  depends_on :libpng
   depends_on 'libogg' => :recommended
 
   fails_with :gcc do
@@ -13,7 +14,6 @@ class Libkate < Formula
   end
 
   def install
-    ENV.libpng
     system "./configure", "--disable-debug",
                           "--disable-dependency-tracking",
                           "--prefix=#{prefix}"

Formula/libmikmod.rb

@@ -5,12 +5,13 @@ class Libmikmod < Formula
   url 'http://mikmod.shlomifish.org/files/libmikmod-3.2.0b4.tar.gz'
   sha256 'da0c4fcfc5ca15c653baeb62b8cf91b35cfa11c1081a1aacc1e443a7d35db870'
 
+  depends_on :x11
+
   def options
     [[ '--with-debug', 'Enable debugging symbols']]
   end
 
   def install
-    ENV.x11
     if ARGV.include? '--with-debug' then
       (ENV.compiler == :clang) ? ENV.Og : ENV.O2
     end

Formula/libming.rb

@@ -5,9 +5,9 @@ class Libming < Formula
   md5 'db6bae65d000e2f2ac78583fd453f99a'
   homepage 'http://www.libming.org'
 
-  def install
-    ENV.x11 # For PNG support.
+  depends_on :libpng
 
+  def install
     # TODO: Libming also includes scripting front-ends for Perl, Python, TCL
     # and PHP.  These are disabled by default.  Figure out what it would take to
     # enable them.

Formula/libpano.rb

@@ -6,11 +6,11 @@ class Libpano < Formula
   homepage 'http://panotools.sourceforge.net/'
   md5 '9c3a4fce8b6f1d79e395896ce5d8776e'
 
+  depends_on :libpng
   depends_on 'jpeg'
   depends_on 'libtiff'
 
   def install
-    ENV.x11 # for libpng
     system "./configure", "--disable-debug", "--disable-dependency-tracking",
                           "--prefix=#{prefix}",
                           "--mandir=#{man}"

Formula/librasterlite.rb

@@ -5,11 +5,11 @@ class Librasterlite < Formula
   url 'http://www.gaia-gis.it/gaia-sins/librasterlite-sources/librasterlite-1.1c.tar.gz'
   sha1 'c54f38ef2974bc92410e2c2196088af14bd9c21a'
 
+  depends_on :x11
   depends_on "libgeotiff"
   depends_on "libspatialite"
 
   def install
-    ENV.x11 # For image format libraries
     # Ensure Homebrew SQLite libraries are found before the system SQLite
     ENV.append 'LDFLAGS', "-L#{HOMEBREW_PREFIX}/lib"
     system "./configure", "--disable-dependency-tracking",

Formula/librsvg.rb

@@ -8,6 +8,7 @@ class Librsvg < Formula
   depends_on 'pkg-config' => :build
   depends_on 'gtk+'
   depends_on 'libcroco'
+  depends_on :x11
 
   def install
     system "./configure", "--disable-dependency-tracking",

Formula/libsvg-cairo.rb

@@ -8,6 +8,7 @@ class LibsvgCairo < Formula
   depends_on 'pkg-config' => :build
   depends_on 'libsvg'
   depends_on 'cairo' if MacOS.leopard?
+  depends_on :x11
 
   def install
     system "./configure", "--disable-dependency-tracking", "--disable-debug", "--prefix=#{prefix}"

Formula/libsvg.rb

@@ -5,10 +5,10 @@ class Libsvg < Formula
   homepage 'http://cairographics.org/'
   md5 'ce0715e3013f78506795fba16e8455d3'
 
+  depends_on :libpng
   depends_on 'jpeg'
 
   def install
-    ENV.x11 # for libpng
     system "./configure", "--prefix=#{prefix}"
     system "make install"
   end

Formula/libtiff.rb

@@ -5,12 +5,13 @@ class Libtiff < Formula
   url 'http://download.osgeo.org/libtiff/tiff-4.0.1.tar.gz'
   sha256 '9a7a039e516c37478038740f1642818250bfb1414cf404cc8b569e5f9d4bf2f0'
 
+  depends_on :x11
+
   def options
     [["--universal", "Builds a universal binary"]]
   end
 
   def install
-    ENV.x11 # Needed to pick up GL/gL.h, otherwise compile error.
     ENV.universal_binary if ARGV.build_universal?
     system "./configure", "--disable-dependency-tracking",
                           "--prefix=#{prefix}"

Formula/libwmf.rb

@@ -7,12 +7,12 @@ class Libwmf < Formula
 
   depends_on 'pkg-config' => :build
   depends_on 'gd'
+  depends_on :libpng
 
   def install
-    ENV.libpng
     system "./configure", "--disable-debug", "--disable-dependency-tracking",
                           "--prefix=#{prefix}",
-                          "--with-freetype=/usr/X11"
+                          "--with-freetype=#{MacOS.x11_prefix}"
     system "make"
     ENV.j1 # yet another rubbish Makefile
     system "make install"

Formula/logstalgia.rb

@@ -12,6 +12,7 @@ class Logstalgia < Formula
   depends_on 'ftgl'
   depends_on 'jpeg'
   depends_on 'pcre'
+  depends_on :x11
 
   if ARGV.build_head? and MacOS.xcode_version >= "4.3"
     depends_on "automake" => :build
@@ -20,8 +21,6 @@ class Logstalgia < Formula
 
 
   def install
-    ENV.x11 # Put freetype-config in path
-
     # For non-/usr/local installs
     ENV.append "CXXFLAGS", "-I#{HOMEBREW_PREFIX}/include"
 

Formula/mapnik.rb

@@ -13,6 +13,7 @@ class Mapnik < Formula
   depends_on 'icu4c'
   depends_on 'boost'
   depends_on 'cairomm' => :optional
+  depends_on :x11
 
   # Reported upstream: https://github.com/mapnik/mapnik/issues/1171
   # Fix is in head.  Remove at 2.0.2.
@@ -21,8 +22,6 @@ class Mapnik < Formula
   end
 
   def install
-    ENV.x11 # for freetype-config
-
     icu = Formula.factory("icu4c")
     # mapnik compiles can take ~1.5 GB per job for some .cpp files
     # so lets be cautious by limiting to CPUS/2

Formula/mapserver.rb

@@ -5,6 +5,7 @@ class Mapserver < Formula
   url 'http://download.osgeo.org/mapserver/mapserver-6.0.2.tar.gz'
   md5 'd831c905b1b0df7ac09a80c3f9387374'
 
+  depends_on :x11
   depends_on 'gd'
   depends_on 'proj'
   depends_on 'gdal'
@@ -28,7 +29,7 @@ class Mapserver < Formula
       "--with-proj",
       "--with-gdal",
       "--with-ogr",
-      "--with-png=/usr/X11"
+      "--with-png=#{MacOS.x11_prefix}"
     ]
 
     args.push "--with-geos" if ARGV.include? '--with-geos'
@@ -56,7 +57,6 @@ class Mapserver < Formula
   end
 
   def install
-    ENV.x11
     system "./configure", *configure_args
     system "make"
     bin.install %w(mapserv shp2img legend shptree shptreevis

Formula/mathgl.rb

@@ -6,9 +6,9 @@ class Mathgl < Formula
   md5 'acd33e68911d9506f60d769dce23f95e'
 
   depends_on 'gsl'
+  depends_on :x11
 
   def install
-    ENV.x11
     ENV['LIBS'] = '-lz'
 
     # Fixed upstream: this line can be removed when v2 is released

Formula/meld.rb

@@ -7,6 +7,7 @@ class Meld < Formula
 
   depends_on 'intltool' => :build
   depends_on 'xz' => :build
+  depends_on :x11
   depends_on 'pygtk'
   depends_on 'pygobject'
   depends_on 'rarian'

Formula/mesalib-glw.rb

@@ -5,6 +5,8 @@ class MesalibGlw < Formula
   homepage 'http://www.mesa3d.org'
   md5 '81a2a4b7cbfce7553f7ad8d924edbe2f'
 
+  depends_on :x11
+
   def options
     [
       ['--enable-static', "build static library"]
@@ -28,7 +30,7 @@ class MesalibGlw < Formula
     inreplace 'configs/autoconf' do |s|
       s.gsub! /.so/, '.dylib'
       s.gsub! /SRC_DIRS = mesa glw/, 'SRC_DIRS = glw'
-      s.gsub! /-L\$\(TOP\)\/\$\(LIB_DIR\)/, '-L/usr/X11/lib'
+      s.gsub! /-L\$\(TOP\)\/\$\(LIB_DIR\)/, "-L#{MacOS.x11_prefix}/lib"
     end
 
     inreplace 'src/glw/Makefile' do |s|

Formula/mjpegtools.rb

@@ -7,6 +7,7 @@ class Mjpegtools < Formula
 
   depends_on 'pkg-config' => :build
   depends_on 'jpeg'
+  depends_on :x11
   depends_on 'libquicktime' => :optional
   depends_on 'libdv' => :optional
   depends_on 'gtk+' => :optional
@@ -28,7 +29,6 @@ class Mjpegtools < Formula
   end
 
   def install
-    ENV.x11
     args = ["--disable-dependency-tracking",
             "--enable-simd-accel",
             "--prefix=#{prefix}"]

Formula/mldonkey.rb

@@ -6,6 +6,7 @@ class Mldonkey < Formula
   sha1 'be8c261e2fbd365dd3866af15a854b074fbc59c1'
 
   depends_on 'objective-caml'
+  depends_on :libpng
 
   if ARGV.include? "--with-x"
     depends_on 'librsvg'
@@ -26,8 +27,6 @@ class Mldonkey < Formula
   end
 
   def install
-    ENV.libpng
-
     # Fix compiler selection
     ENV['OCAMLC'] = "#{HOMEBREW_PREFIX}/bin/ocamlc.opt -cc #{ENV.cc}"
 

Formula/mupdf.rb

@@ -29,6 +29,7 @@ class Mupdf < Formula
   depends_on 'jpeg'
   depends_on 'openjpeg'
   depends_on 'jbig2dec'
+  depends_on :x11 # libpng, freetype
 
   def patches
     # Fix up the Makefile so it doesn't mess with our CFLAGS.
@@ -36,7 +37,6 @@ class Mupdf < Formula
   end
 
   def install
-    ENV.x11 # For LibPNG and Freetype
     system "make", "install", "prefix=#{prefix}"
   end
 end

Formula/nagios.rb

@@ -7,6 +7,7 @@ class Nagios < Formula
 
   depends_on 'gd'
   depends_on 'nagios-plugins'
+  depends_on :x11 # Required to compile some CGI's against the build-in libpng.
 
   def nagios_sbin;  prefix+'cgi-bin';       end
   def nagios_etc;   etc+'nagios';           end
@@ -16,7 +17,6 @@ class Nagios < Formula
   def group;        `id -gn`.chomp;         end
 
   def install
-    ENV.x11 # Required to compile some CGI's against the build-in libpng.
     system "./configure", "--disable-debug",
                           "--disable-dependency-tracking",
                           "--prefix=#{prefix}",

Formula/ncview.rb

@@ -5,6 +5,7 @@ class Ncview < Formula
   homepage 'http://meteora.ucsd.edu/~pierce/ncview_home_page.html'
   md5 '34e25f5949af342a1783542799f51bed'
 
+  depends_on :x11
   depends_on "netcdf"
 
   # Disable a block in configure that tries to pass an RPATH to the compiler.
@@ -14,8 +15,6 @@ class Ncview < Formula
   end
 
   def install
-    ENV.x11 # For GUI and libpng.
-
     system "./configure", "--prefix=#{prefix}",
                           "--disable-dependency-tracking"
     system "make install"

Formula/netpbm.rb

@@ -9,10 +9,9 @@ class Netpbm < Formula
 
   depends_on "libtiff"
   depends_on "jasper"
+  depends_on :libpng
 
   def install
-    ENV.x11 # For PNG
-
     if ARGV.build_head?
       system "cp", "config.mk.in", "config.mk"
       config = "config.mk"

Formula/octave.rb

@@ -27,6 +27,7 @@ class Octave < Formula
   depends_on 'gnu-sed' => :build
   depends_on 'texinfo' => :build     # OS X's makeinfo won't work for this
 
+  depends_on :x11
   depends_on 'fftw'
   # When building 64-bit binaries on Snow Leopard, there are naming issues with
   # the dot product functions in the BLAS library provided by Apple's
@@ -69,7 +70,6 @@ class Octave < Formula
     # build time with -O3: user 58m58.054s  sys 7m52.221s
     ENV.m64 if MacOS.prefer_64_bit?
     ENV.append_to_cflags "-D_REENTRANT"
-    ENV.x11
 
     args = [
       "--disable-dependency-tracking",

Formula/openjpeg.rb

@@ -9,9 +9,9 @@ class Openjpeg < Formula
 
   depends_on 'little-cms2'
   depends_on 'libtiff'
+  depends_on :libpng
 
   def install
-    ENV.x11  # So it can find libpng
     system "./configure", "--disable-dependency-tracking", "--prefix=#{prefix}"
     system "make install"
   end

Formula/pango.rb

@@ -8,6 +8,7 @@ class Pango < Formula
   depends_on 'pkg-config' => :build
   depends_on 'xz' => :build
   depends_on 'glib'
+  depends_on :x11
 
   if MacOS.leopard?
     depends_on 'fontconfig' # Leopard's fontconfig is too old.
@@ -26,7 +27,6 @@ class Pango < Formula
   end
 
   def install
-    ENV.x11
     system "./configure", "--disable-dependency-tracking",
                           "--disable-debug",
                           "--prefix=#{prefix}",

Formula/pangomm.rb

@@ -9,6 +9,7 @@ class Pangomm < Formula
   depends_on 'pango'
   depends_on 'glibmm'
   depends_on 'cairomm'
+  depends_on :x11
 
   def install
     system "./configure", "--disable-dependency-tracking", "--prefix=#{prefix}"

Formula/pcb.rb

@@ -16,6 +16,7 @@ class Pcb < Formula
   depends_on 'gd'
   depends_on 'glib'
   depends_on 'gtkglext'
+  depends_on :x11
 
   # See comments in intltool formula
   depends_on 'XML::Parser' => :perl
@@ -25,8 +26,6 @@ class Pcb < Formula
   end
 
   def install
-    ENV.x11
-
     system "./autogen.sh" if ARGV.build_head?
 
     system "./configure", "--disable-debug", "--disable-dependency-tracking",

Formula/pdf2image.rb

@@ -5,6 +5,8 @@ class Pdf2image < Formula
   url 'http://pdf2image.googlecode.com/files/pdf2image-0.53-source.tar.gz'
   md5 'df9614aa45284d2aa4c6e2596062eeb7'
 
+  depends_on :x11
+
   def install
     system "./configure", "--prefix=#{prefix}"
 
@@ -12,9 +14,8 @@ class Pdf2image < Formula
     # http://code.google.com/p/pdf2json/issues/detail?id=2
     inreplace "Makefile", "/man/", "/share/man/"
 
-    # Add X11 libs manually; the Makefiles don't use LDFLAGS properly, so ENV.x11 doesn't work
-    # - @adamv
-    inreplace ["src/Makefile", "xpdf/Makefile"], "LDFLAGS =", "LDFLAGS=-L/usr/X11/lib"
+    # Add X11 libs manually; the Makefiles don't use LDFLAGS properly
+    inreplace ["src/Makefile", "xpdf/Makefile"], "LDFLAGS =", "LDFLAGS=-L#{MacOS.x11_prefix}/lib"
 
     system "make"
     system "make install"

Formula/pdf2svg.rb

@@ -25,20 +25,18 @@ class Pdf2svg < Formula
 
   depends_on "pkg-config" => :build
 
+  depends_on :x11
   depends_on "poppler"
   depends_on PopplerGlib.new
   depends_on "gtk+"
   depends_on "cairo" # Poppler-glib needs a newer cairo than provided by OS X 10.6.x
                      # and pdf2svg needs it to be on PKG_CONFIG_PATH during the build
-
   def patches
     # fix call to poppler to render normal thickness lines in firefox
     DATA
   end
 
   def install
-    ENV.x11
-
     system "./configure", "--prefix=#{prefix}", "--disable-debug", "--disable-dependency-tracking"
     system "make install"
   end

Formula/pil.rb

@@ -5,6 +5,7 @@ class Pil < Formula
   homepage 'http://www.pythonware.com/products/pil/'
   md5 'fc14a54e1ce02a0225be8854bfba478e'
 
+  depends_on :x11
   depends_on 'jpeg' => :recommended
   depends_on 'little-cms' => :optional
 
@@ -21,10 +22,10 @@ class Pil < Formula
     inreplace "setup.py" do |s|
       # Tell setup where Freetype2 is on 10.5/10.6
       s.gsub! 'add_directory(include_dirs, "/sw/include/freetype2")',
-              'add_directory(include_dirs, "/usr/X11/include")'
+              'add_directory(include_dirs, "#{MacOS.x11_prefix}/include")'
 
       s.gsub! 'add_directory(include_dirs, "/sw/lib/freetype2/include")',
-              'add_directory(library_dirs, "/usr/X11/lib")'
+              'add_directory(library_dirs, "#{MacOS.x11_prefix}/lib")'
 
       # Tell setup where our stuff is
       s.gsub! 'add_directory(library_dirs, "/sw/lib")',

Formula/pixie.rb

@@ -8,12 +8,12 @@ class Pixie < Formula
   depends_on 'libtiff'
   depends_on 'fltk'
   depends_on 'openexr'
+  depends_on :libpng
 
   def install
     openexr = Formula.factory('openexr')
     ilmbase = Formula.factory('ilmbase')
 
-    ENV.x11 # For libpng
     ENV.append "CPPFLAGS", "-I#{openexr.include}/OpenEXR -I#{ilmbase.include}/OpenEXR"
     ENV.append "LDFLAGS",  "-L#{openexr.lib} -L#{ilmbase.lib}"
     system "./configure", "--disable-dependency-tracking",

Formula/pixman.rb

@@ -6,6 +6,7 @@ class Pixman < Formula
   sha1 'efaa09789128ebc42d17a11d2e350b7217a7cd05'
 
   depends_on 'pkg-config' => :build
+  depends_on :x11
 
   fails_with :llvm do
     build 2336
@@ -21,7 +22,6 @@ class Pixman < Formula
   end
 
   def install
-    ENV.x11
     ENV.universal_binary if ARGV.build_universal?
 
     # Disable gtk as it is only used to build tests

Formula/plotutils.rb

@@ -6,9 +6,9 @@ class Plotutils < Formula
   homepage 'http://www.gnu.org/software/plotutils/'
   md5 'c08a424bd2438c80a786a7f4b5bb6a40'
 
-  def install
-    ENV.x11 # enable libpng support
+  depends_on :libpng
 
+  def install
     # Fix usage of libpng to be 1.5 compatible
     inreplace 'libplot/z_write.c', 'png_ptr->jmpbuf', 'png_jmpbuf (png_ptr)'
 

Formula/png2ico.rb

@@ -5,8 +5,9 @@ class Png2ico < Formula
   url 'http://www.winterdrache.de/freeware/png2ico/data/png2ico-src-2002-12-08.tar.gz'
   md5 '9b663df81c826cd564638cba2e6bc75b'
 
+  depends_on :libpng
+
   def install
-    ENV.libpng
     inreplace 'Makefile', 'g++', '$(CXX)'
     system "make", "CPPFLAGS=#{ENV.cxxflags} #{ENV.cppflags} #{ENV.ldflags}"
     bin.install 'png2ico'

Formula/pngnq.rb

@@ -6,6 +6,7 @@ class Pngnq < Formula
   md5 'fdbb94d504931b50c54202b62f98aa44'
 
   depends_on 'pkg-config' => :build
+  depends_on :libpng
 
   # Fixes compilation on OSX Lion
   # png.h on Lion does not, in fact, include zlib.h
@@ -16,7 +17,6 @@ class Pngnq < Formula
   end
 
   def install
-    ENV.x11 # for libpng
     system "./configure", "--disable-dependency-tracking",
                           "--prefix=#{prefix}"
     system "make install"

Formula/pngrewrite.rb

@@ -5,8 +5,9 @@ class Pngrewrite < Formula
   homepage 'http://entropymine.com/jason/pngrewrite/'
   md5 '37216932d12bf9b47dca1f45724080d6'
 
+  depends_on :libpng
+
   def install
-    ENV.libpng
     system "#{ENV.cc} #{ENV.cflags} #{ENV.cppflags} -o pngrewrite pngrewrite.c #{ENV.ldflags} -lpng"
     bin.install 'pngrewrite'
   end

Formula/podofo.rb

@@ -8,13 +8,12 @@ class Podofo < Formula
   depends_on 'pkg-config' => :build
   depends_on 'cmake' => :build
 
+  depends_on :x11
   depends_on 'jpeg'
   depends_on 'libtiff'
   depends_on 'lua'
 
   def install
-    ENV.x11 # For Freetype and Fontconfig
-
     mkdir 'build' do
       # Build shared to simplify linking for other programs.
       system "cmake", "..",

Formula/poppler.rb

@@ -17,6 +17,7 @@ class Poppler < Formula
   depends_on 'qt' if qt?
   depends_on 'glib' if glib?
   depends_on 'cairo' if glib? # Needs a newer Cairo build than OS X 10.6.7 provides
+  depends_on :x11 # Fontconfig headers
 
   def options
     [
@@ -26,8 +27,6 @@ class Poppler < Formula
   end
 
   def install
-    ENV.x11 # For Fontconfig headers
-
     if qt?
       ENV['POPPLER_QT4_CFLAGS'] = `#{HOMEBREW_PREFIX}/bin/pkg-config QtCore QtGui --libs`.chomp
       ENV.append 'LDFLAGS', "-Wl,-F#{HOMEBREW_PREFIX}/lib"

Formula/py2cairo.rb

@@ -6,6 +6,7 @@ class Py2cairo < Formula
   md5 '20337132c4ab06c1146ad384d55372c5'
 
   depends_on 'cairo'
+  depends_on :x11
 
   def options
     [['--universal', 'Build universal binaries']]