Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for cython, depend on sagemath, instructions for PyPI #19

Open
wants to merge 15 commits into
base: master
Choose a base branch
from

Conversation

mmasdeu
Copy link
Sponsor

@mmasdeu mmasdeu commented Jul 20, 2017

After using the template to distribute my own project, I have added some functionality to the package. In particular:

  1. Added a cython file and the necessary changes to make it work.
  2. This being a Sage package, it depends on the python package sagemath. This is a tiny python package available at PyPI that allows to check which version of Sage is installed.
  3. Added some basic instructions to distribute the code to PyPI.

@videlec
Copy link
Collaborator

videlec commented Aug 3, 2017

You can get the list of tarballs without using Python in one line

wget $SAGE_SERVER --quiet --output-document=- | grep "sage-.[^\s]*bz2" --only-matching

bash is much better than Python for that kind of things

@videlec
Copy link
Collaborator

videlec commented Aug 3, 2017

Would be much cleaner to set a variable

SAGE="$HOME/SageMath/sage"

and use it everywhere sage is called

@seblabbe
Copy link
Collaborator

I am currently at Sage Days 117. There was a demo earlier this week on how to share code and create a package where we used sage_sample. I am now taking a look at all the PR that have been here for years now...

The current PR has conflicts. I let the author or someone else fix them.

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

Successfully merging this pull request may close these issues.

None yet

4 participants