search

pele-python / pele

Python energy landscape explorer
Other
95 stars 41 forks source link

add an alternate setup file which builds the cxx extensions entirely with cmake #83

Closed js850 closed 10 years ago

js850 commented 10 years ago

this is much faster and cleaner than letting numpy.distutils do it.

smcantab commented 10 years ago

hero! :+1

On Wed, Oct 8, 2014 at 3:21 PM, Jacob Stevenson notifications@github.com wrote:

this is much faster and cleaner than letting numpy.distutils do it.

You can merge this Pull Request by running

git pull https://github.com/js850/pele cmake_build

Or view, comment on, or merge it at:

https://github.com/pele-python/pele/pull/83 Commit Summary

  • start to add a CMakeLists.txt that can build the cython .so files
  • it works with a library now
  • add potentials/_pele.so to CMake build
  • add a cmake function to build the library automatically
  • add file to setup cython libraries with cmake

File Changes

Patch Links:

— Reply to this email directly or view it on GitHub https://github.com/pele-python/pele/pull/83.

js850 commented 10 years ago

the numpy include directories are specified by hand, these should be passes from setup.py