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

Update lock files to fix aarch64 compatibility #6

Merged
merged 2 commits into from
Jun 29, 2023

Conversation

Hooksie
Copy link
Collaborator

@Hooksie Hooksie commented Jun 26, 2023

Updates lock files and associated toml configs for latest maturin version, since current values are no longer compatible with the latest versions. Also bump the min python version

Since this changes compatibility for Python version, it's technically backwards incompatible. But given the project is <1.0, IIRC semver doesnt require a major version bump, so I'll leave it at 0.3.

Fixes #5

@aviramha
Copy link
Owner

Thanks! Can you signoff your commits please?

Signed-off-by: Matt Hooks <mhooks@rippling.com>
@Hooksie
Copy link
Collaborator Author

Hooksie commented Jun 27, 2023

@aviramha Certainly, all set

BTW, I dont have the manylinux env setup on this dev machine, and so I havent tested the release builds with these new lockfiles on anything aside from an M1 Macbook. I'm assuming the CI on this project will handle that?

@aviramha
Copy link
Owner

aviramha commented Jun 27, 2023

Crap, 3.6 reached EOL - can you remove it as part of this PR please?

Removes flags no longer supported by maturin (--sdist), in favor of the
new default behavior. Also extends the release_macos job to use matrix
for build target arch for future wheel cross-compilation.

Also swap the rust toolchain to stable, since nightly is not needed
here.

Signed-off-by: Matt Hooks <mhooks@rippling.com>
@Hooksie
Copy link
Collaborator Author

Hooksie commented Jun 28, 2023

@aviramha Went ahead and pulled those out, and extended out the github actions as well to include the latest Python versions.

I did poke around a bit at cross-compiling the M1 wheel, but ultimately chose not to, wanting to avoid the work of pulling in new openssl steps for alternative targets. I moved the target to a matrix in macos though, so it will be easy drop in for the future.

maturin removed --no-sdist, which is now the default, so I removed that as well. I swapped the toolchains to stable instead of nightly, but if you'd prefer they stay on nightly i can swap them back.

@aviramha
Copy link
Owner

aviramha commented Jun 29, 2023

Thanks! I appreciate the work you did. I'd like to offer you to join as a maintainer as it seems you have stake & interest (atleast more than I do at the moment).
I think there's some space to improve the crate - for example add upstream (or fork) a support for this without openssl (using ring probably)

@aviramha aviramha merged commit 9f9ba46 into aviramha:master Jun 29, 2023
13 checks passed
@Hooksie
Copy link
Collaborator Author

Hooksie commented Jul 6, 2023

@aviramha Thanks! (and apologies for delay, I've been on vacation 🙂 )

I'd be happy to continue helping, although notably I don't think I'll have a guaranteed time commitment. In the meantime, I'd be happy to drop a few more maintenance patches in, if it's not a burden

@aviramha
Copy link
Owner

aviramha commented Jul 7, 2023

Sure! There's no time commitment from my side as well - It seems ormsgpack is already being used by quite few companies, and I don't want to be single point of failure for changes :)
I'll add you permissions :)

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.

rfernet fails to install on aarch64 (Macbook M1)
2 participants