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

hotfix Cython<3, else can't install #234

Merged
merged 1 commit into from
Jul 20, 2023
Merged

hotfix Cython<3, else can't install #234

merged 1 commit into from
Jul 20, 2023

Conversation

skjerns
Copy link
Collaborator

@skjerns skjerns commented Jul 19, 2023

With newest Cython version 3.0, something with finding the modules is broken. Until we have time to dive for a solution, I propose this hotfix to set Cython<3. As the Build is done in its own env, this should have no impact on the base env that we are installing to, ie. no side-effects

See #233

@OlivierKessler01
Copy link

OlivierKessler01 commented Jul 20, 2023

Please @skjerns merge this asap, using --no-build-isolation and installing cython in userland, while working on simple and well isolated projects, doesn't make for a proper fix since the cython version would conflict with other compiling needs.

@skjerns
Copy link
Collaborator Author

skjerns commented Jul 20, 2023

However, install should still be possible using the wheels on pypi, or do you require the dev version?

(I'm not sure if there is a way to install using pypi and having to use Cython?)

@skjerns skjerns merged commit 7919e51 into master Jul 20, 2023
@OlivierKessler01
Copy link

However, install should still be possible using the wheels on pypi, or do you require the dev version?

(I'm not sure if there is a way to install using pypi and having to use Cython?)

We are running 0.1.20 on musl based platforms, so no wheels available at the time, had I checked I would've seen that more recent versions of pyedflib provided wheels for us.

Cheers ! :)

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.

2 participants