py2cairo: don't link libpython

Hacks around waf's python extension mode, which unnecessarily forces an
explicit link to the python framework. This allows us to avoid a
dependency on :python so that bottles don't install Homebrew python.

Closes #36546.

Signed-off-by: Tim D. Smith <git@tim-smith.us>

1 files changed, 15 insertions, 15 deletions

diff --git a/Library/Formula/py2cairo.rb b/Library/Formula/py2cairo.rb
index 888ba530a..857812735 100644
--- a/Library/Formula/py2cairo.rb
+++ b/Library/Formula/py2cairo.rb
@@ -1,9 +1,7 @@
-require 'formula'
-
 class Py2cairo < Formula
-  homepage 'http://cairographics.org/pycairo/'
-  url 'http://cairographics.org/releases/py2cairo-1.10.0.tar.bz2'
-  sha1 '2efa8dfafbd6b8e492adaab07231556fec52d6eb'
+  homepage "http://cairographics.org/pycairo/"
+  url "http://cairographics.org/releases/py2cairo-1.10.0.tar.bz2"
+  sha1 "2efa8dfafbd6b8e492adaab07231556fec52d6eb"
 
   bottle do
     cellar :any
@@ -12,10 +10,9 @@ class Py2cairo < Formula
     sha1 "bc34e6cf22e942d57a9618821e024c65c6a07fa3" => :mountain_lion
   end
 
-  depends_on 'pkg-config' => :build
-  depends_on 'cairo'
-  depends_on :x11
-  depends_on :python
+  depends_on "pkg-config" => :build
+  depends_on "cairo"
+  depends_on :python if MacOS.version <= :snow_leopard
 
   option :universal
 
@@ -27,16 +24,19 @@ class Py2cairo < Formula
   def install
     ENV.refurbish_args
 
+    # disable waf's python extension mode because it explicitly links libpython
+    # https://code.google.com/p/waf/issues/detail?id=1531
+    inreplace "src/wscript", "pyext", ""
+    ENV["LINKFLAGS"] = "-undefined dynamic_lookup"
+    ENV.append_to_cflags `python-config --includes`
+
     # Python extensions default to universal but cairo may not be universal
     ENV['ARCHFLAGS'] = "-arch #{MacOS.preferred_arch}" unless build.universal?
 
-    # waf miscompiles py2cairo on >= lion with HB python, linking the wrong
-    # Python Library.  So add a LINKFLAG that sets the path.
-    # https://github.com/Homebrew/homebrew/issues/12893
-    # https://github.com/Homebrew/homebrew/issues/14781
-    # https://bugs.freedesktop.org/show_bug.cgi?id=51544
-    ENV['LINKFLAGS'] = "-L#{%x(python-config --prefix).chomp}/lib/python2.7/config"
     system "./waf", "configure", "--prefix=#{prefix}", "--nopyc", "--nopyo"
     system "./waf", "install"
+
+    module_dir = lib/"python2.7/site-packages/cairo"
+    mv module_dir/"lib_cairo.dylib", module_dir/"_cairo.so"
   end
 end