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

Prepare for pip / conda builds #3

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

Prepare for pip / conda builds #3

wants to merge 3 commits into from

Conversation

wd15
Copy link
Collaborator

@wd15 wd15 commented Jul 8, 2021

Preparing repository for pip and conda builds. Generally make the setup.py a little cleaner.

  • Remove graspi/graspi.cpp as not really necessary and potential for confusion. Also, can't realistically use GraSPI without a working Cython implementation anyway. Updated .gitignore to reflect this.
  • Moved cythonizeGraspi/ to graspi/ to fit in with typical Python package structure. Also this makes it work without an installed version of GraSPI for development purposes.
  • Add __version__ attribute to mirror other python projects
  • Removed unnecessary distutils directives from cython files

wd15 added 3 commits July 8, 2021 11:54
 - Add an __init__.py
 - Add a __version__ attribute
 - Remove unnecessary distutils directives from cython files
 - Remove unnecessary setupGraspiCython.py file
Adding the cpp file is not necessary as Cython is widely available and
it is already a dependency for this package.
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.

1 participant