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

Fixed release CI to support python 3.10 building #16

Closed

Conversation

joeyballentine
Copy link

@joeyballentine joeyballentine commented Oct 4, 2022

Apologies for the extra formatting, I can undo that if necessary.

I tested this and it successfully built the 3.10 wheels. All I did was I updated the setup-python action to v4 (v1 was having issues finding the right versions) and used strings for the Python versions (3.10 as a number was trying to install 3.1 lol).

I'm still working on trying to get arm64 building -- you are correct that it is not simple to get working but I think with enough tinkering I'll be able to get that to work. For now though, this at least fixes #14.

By the way, I did get it to build for arm64. Had to convert the project to use cmake + cibuildwheel, and this is only for 3.9 and 3.10, but at least it built. If you want me to PR everything I added, I can try to either make a hybrid release workflow that only uses cmake and cibuildwheel for the arm version, or I could just use that setup for both. This of course would only be after the wheels are confirmed working, because I have no idea if they even work.

@joeyballentine
Copy link
Author

Hm that's weird, it passed when I ran it

@joeyballentine
Copy link
Author

Looks like this is failing due to a package called traitlets. I'm guessing it posted an invalid update since the version it is trying to install is 5.2.2.post0. Not sure what to do about that

@tobywf
Copy link
Owner

tobywf commented Oct 17, 2022

Fair enough, thanks for having a go. The 3.10 fix was roughly known to me, but it's tough - I don't really do any Python development myself any more, so keeping track of new Python versions is not something that happens naturally. I think at this point, it's probably time to drop Python 3.6 (and maybe 3.7 already).

I was most interested in the ARM64 build. It's really unfortunate; setuptools is very opaque. Edit: The fusion with lipo is the hardest part, which is the route I'd ideally like to go down. Alternative is building two wheels which should be do-able.

The traitlets thing is weird; I think the solution is to check in poetry.lock? Not sure why I excluded that file from git.

@joeyballentine
Copy link
Author

Closing this PR as it is no longer needed by me.

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.

Can't install in Python 3.10
2 participants