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

Fix Cython.Build module import #47

Merged
merged 3 commits into from
Feb 15, 2022
Merged

Conversation

saemy
Copy link
Contributor

@saemy saemy commented Jan 14, 2022

The module name starts with an uppercase "C" or else it won't be
found (on case-sensitive file systems).

Also, rebuilt the C files with Cython 0.9.26 to fix any issues that
were present for builds against Python 3.10 or when using newer
versions of gcc.

Fixes: #45, #46

saemy added 2 commits January 14, 2022 16:34
The module name starts with an uppercase "C" or else it won't be
found (on case-sensitive file systems).

Also, rebuilt the C files with Cython 0.9.26 to fix any issues that
were present for builds against Python 3.10 or when using newer
versions of gcc.

Fixes: pyranges#45, pyranges#46
kcleal
kcleal previously approved these changes Feb 8, 2022
@endrebak
Copy link
Collaborator

endrebak commented Feb 9, 2022

Looking at it now. Thanks.

Will need to test locally. Dunno' if I've ever seen uppercase imports before.

@endrebak
Copy link
Collaborator

endrebak commented Feb 9, 2022

Looks good. Would you mind updating the ncls/version.py and CHANGELOG too?

@endrebak endrebak merged commit 526de2a into pyranges:master Feb 15, 2022
@endrebak
Copy link
Collaborator

Sorry for the late merge. Will upload to pypi now :D

@endrebak
Copy link
Collaborator

Should be out now 👍

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.

Does not compile with gcc 11.1.0
3 participants