OpenChemistry / openchemistry

Supermodule containing submodules and external project to build all components
http://openchemistry.org/
BSD 3-Clause "New" or "Revised" License
80 stars 60 forks source link

Include MMTF-Cpp in the superbuild #47

Closed frodofine closed 6 years ago

frodofine commented 6 years ago

Subject explains it all! I'm including mmtf-cpp as a required dependency, but I'm willing to change this if needed.

Signed-off-by: Jonathan Fine finej@purdue.edu

cryos commented 6 years ago

Super minor questions on comments, this built for me, and seemed to work in my initial testing. Thanks for moving this to the superbuild!

frodofine commented 6 years ago

Yes, those are copy-paste mistakes! I've pushed a fix.