search

rabrahm / ceres

A set of pipelines and routines for echelle spectrographs
MIT License
39 stars 27 forks source link

Updated SSEphem Makefile for OSX venv #4

Closed jmccormac01 closed 8 years ago

jmccormac01 commented 8 years ago

The _jplephem.so file was not being linked correctly to the right version of Python. This meant import jplephem would crash and burn with a Fatal Python error: PyThreadState_Get: no current thread error, see #3.

The change to the Makefile alters the _jplephem.so file to point to the right python installation. The change affects OSX only.

The change to install.py explicitly sets the python version for python setup.py build to be the same as that used for python install.py

Thanks to @mindriot101 and @pchote for their help in fixing this.

jmccormac01 commented 8 years ago

I've tested this pull request using some sample CAFE data on OSX and it runs through without any problems. I cannot test on Linux directly, hopefully there are no issues.