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

crypto: Bump blst to 0.3.12 #69

Merged
merged 3 commits into from
Jun 24, 2024
Merged

crypto: Bump blst to 0.3.12 #69

merged 3 commits into from
Jun 24, 2024

Conversation

vapourismo
Copy link

Improves RISC-V compatibility.

@emturner
Copy link
Collaborator

looks good to me! Though might need to bump rust version and/or change to cargo check --locked in the ci job

@emturner emturner assigned vapourismo and unassigned emturner Jun 24, 2024
@vapourismo
Copy link
Author

vapourismo commented Jun 24, 2024

I had to bump curve25519-dalek to 4.1.3 but that one runs into unstable feature use errors in CI. The crate's MSRV is 1.60 and CI runs 1.64, I think 🤔

@vapourismo
Copy link
Author

I had to bump curve25519-dalek to 4.1.3 but that one runs into unstable feature use errors in CI. The crate's MSRV is 1.60 and CI runs 1.64, I think 🤔

The issue is actually that stdsimd no longer exists it appears. Maybe upgrading Rust version will magically fix it 🗡️

@vapourismo vapourismo merged commit 51f402b into master Jun 24, 2024
7 checks passed
@vapourismo vapourismo deleted the ole@blst-riscv branch June 24, 2024 18:59
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