Skip to content
This repository has been archived by the owner on Jun 24, 2020. It is now read-only.

Switch to numpy.distutils #1

Open
astrojuanlu opened this issue Mar 6, 2016 · 7 comments
Open

Switch to numpy.distutils #1

astrojuanlu opened this issue Mar 6, 2016 · 7 comments

Comments

@astrojuanlu
Copy link
Member

The included makefile only works on Linux, so this change should allow the library to be compiled on Windows and OS X too.

astrojuanlu added a commit to astrojuanlu/poliastro that referenced this issue Mar 6, 2016
@astrojuanlu
Copy link
Member Author

The option of distributing all of CEPHES along with pycephes has the potential drawback of coupling the platform dependent and the platform independent parts.

  • pycephes may evolve, but is nonetheless cross platform.
  • cephes won't ever evolve, but is not cross platform.

So I can provide cross platform packages of CEPHES once, and then focus on developing pycephes around it.

@astrojuanlu
Copy link
Member Author

Building with distutils does not render CEPHES a Python package though. I will have to play with the corresponding recipes.

@astrojuanlu
Copy link
Member Author

@astrojuanlu
Copy link
Member Author

@astrojuanlu
Copy link
Member Author

On the other hand the version packaged in http://www.moshier.net/double.zip requires careful selection of the sources to compile.

@astrojuanlu
Copy link
Member Author

By the way, we'd better leave the conda recipe outside of this repository and put it directly on conda-forge (see #2).

@astrojuanlu
Copy link
Member Author

A version which installs correctly is already available at https://github.com/poliastro/cephes/. I will close the issue as soon as everything is tested.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant