wbrawner/homebrew-core
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

open-babel: 2.3.2 patch to fix pybel's Molecule.draw + fix for Cairo

Browse source 
Added a patch that fixes Molecule.draw() in pybel.py (similar to upstream commit df59c4a630cf753723d1318c40479d48b7507e1c)
Also fixed the inclusion of the Cairo library and includes. Earlier the CMake directives were poiting to locations within the
open-babel installation. There also seemed to be a confusion between strings and arguments, giving cmake problems to understand
the directive.

Closes Homebrew/homebrew#20800.

Signed-off-by: Adam Vandenberg <flangy@gmail.com>
This commit is contained in:
Fredrik Wallner 2013-06-26 11:02:51 +02:00 committed by Adam Vandenberg
parent a8c48b9a9e
commit e741373b6c
1 changed files with 9 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
Formula/open-babel.rb
View file

@ -15,12 +15,19 @@ class OpenBabel < Formula
depends_on 'cairo' => :optional
depends_on 'eigen' if build.with?('python') || build.with?('java')
# Patch to fix Molecule.draw() in pybel in accordance with upstream commit df59c4a630cf753723d1318c40479d48b7507e1c
def patches
"https://gist.github.com/fredrikw/5858168/raw"
end
def install
args = %W[ -DCMAKE_INSTALL_PREFIX=#{prefix} ]
args << "-DJAVA_BINDINGS=ON" if build.with? 'java'
args << "-DBUILD_GUI=ON" if build.with? 'wxmac'
args << "-DCAIRO_INCLUDE_DIRS=#{include}/cairo "\
"-DCAIRO_LIBRARIES=#{lib}/libcairo.dylib" if build.with? 'cairo'
# Looking for Cairo in HOMEBREW_PREFIX
# setting the arguments separately, joining them in one string fails with the 'system "cmake", *args' command
args << "-DCAIRO_INCLUDE_DIRS='#{HOMEBREW_PREFIX}/include/cairo'" if build.with? 'cairo'
args << "-DCAIRO_LIBRARIES='#{HOMEBREW_PREFIX}/lib/libcairo.dylib'" if build.with? 'cairo'
python do
args << "-DPYTHON_BINDINGS=ON"