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

PEP 517 build support #1

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

Conversation

corbettht
Copy link

Hi,

First, thanks for the great package -- I have been experimenting with using it for fast catalog lookups from large binary tables, and am really impressed with the speeds.

I've made a few changes to the build process to support a PEP 517 style build, with the goal of getting it to work nicely with packaging tools like Poetry (https://python-poetry.org). As-is, the build fails when installing with Poetry due to the lack of numpy (and possibly with other tools that build in an isolated-environment, though I have not tested them).

Would you consider these changes for main?

@corbettht
Copy link
Author

Hi, Sorry to double post here, but I think this may be broken more widely as of pip 23.1 per the broad tree of discussions here: pypa/pip#8559.

Thanks again

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