diff options
| author | Andrew Janke | 2013-04-24 17:38:13 -0400 |
|---|---|---|
| committer | Jack Nagel | 2013-04-24 18:43:32 -0500 |
| commit | df7f810a1625a10400e4082b527b9ae7ddf268e5 (patch) | |
| tree | 947833637752e62a1515a983fc7acc302b1ee9a0 /Library/Formula | |
| parent | 0789a28dadbdb6d62e1d36811266ba3eb88723e1 (diff) | |
| download | homebrew-df7f810a1625a10400e4082b527b9ae7ddf268e5.tar.bz2 | |
ecasound 2.8.1: Fix ruby support behavior
Change formula to always pass explicit --enable-rubyecasound=<yes|no>
so it works regardless of the ecasound configure's default behavior
for --enable-ruby. Formula was assuming default was no, but default was
actually yes, so the ruby stuff got built even when --with-ruby was
not supplied, which could break the build when attempting to install
in to /Library/Ruby.
Fixes #19357.
Closes #19420.
Signed-off-by: Jack Nagel <jacknagel@gmail.com>
Diffstat (limited to 'Library/Formula')
| -rw-r--r-- | Library/Formula/ecasound.rb | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/Library/Formula/ecasound.rb b/Library/Formula/ecasound.rb index 57156f9cd..13a8a1048 100644 --- a/Library/Formula/ecasound.rb +++ b/Library/Formula/ecasound.rb @@ -14,7 +14,8 @@ class Ecasound < Formula --disable-dependency-tracking --prefix=#{prefix} ] - args << "--enable-rubyecasound=yes" if build.include? 'with-ruby' + # Always explicitly control ruby, since there's some confusion about the default + args << ("--enable-rubyecasound=%s" % ((build.include? 'with-ruby') ? 'yes' : 'no')) system "./configure", *args system "make install" end |