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.
This commit is contained in:
parent
980f2f62b3
commit
00e01f3d73
1 changed files with 11 additions and 5 deletions
|
@ -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
|
||||
|
|
Loading…
Reference in a new issue