Don't install Thrift scripting bindings

You can install them with easy_install/gem. I confess I don't know if these are worse? But they install outside the Homebrew prefix (/Library/Ruby etc.) so it's against policy.

We could add a flag for it though if this method is better.

1 files changed, 10 insertions, 4 deletions

diff --git a/Library/Formula/thrift.rb b/Library/Formula/thrift.rb
index 5d351d3b2..c0cbe7476 100644
--- a/Library/Formula/thrift.rb
+++ b/Library/Formula/thrift.rb
@@ -14,22 +14,28 @@ class Thrift <Formula
   depends_on 'boost'
   
   def install
-    FileUtils.cp "/usr/X11/share/aclocal/pkg.m4", "aclocal"
+    cp "/usr/X11/share/aclocal/pkg.m4", "aclocal"
     system "./bootstrap.sh" if version == 'HEAD'
     system "./configure","--disable-debug","--without-java",
                          "--prefix=#{prefix}","--libdir=#{lib}",
                          # rationale: this can be installed with easy_install
                          # and when you do that, it installs properly, we
                          # can't install it properly without leaving Homebrew's prefix
-                         "--without-py"
+                         "--without-py",
+                         # again, use gem
+                         "--without-ruby",
+                         "--without-perl"
     system "make"
     system "make install"
   end
-  
+
   def caveats; <<-EOS.undent
-    We didn't install the python bindings, to do that:
+    Some bindings were not installed. You may like to do the following:
 
+        gem install thrift
         easy_install thrift
+
+    Perl bindings are a mystery someone should solve.
     EOS
   end
 end