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

remove the dependency on the blake3 "digest" feature #35701

Closed
wants to merge 1 commit into from

Conversation

oconnor663
Copy link

Solana already depends on the "traits-preview" feature, which is the intended way to activate the optional dependency on digest. Version 1.5.2 of the blake3 crate started using the new-ish dep: notation in Cargo.toml, which removes the unintended "digest" feature. Solana currently pins v1.5.0, but this will be a build break when that pin is updated. I think a user has already run into this here: https://solana.stackexchange.com/q/17787/29050.

Solana already depends on the "traits-preview" feature, which is [the intended way](https://github.com/BLAKE3-team/BLAKE3/blob/1170f023bf459ea3d0bedd285d6ae32309ee348c/Cargo.toml#L52) to activate the optional dependency on `digest`. Version 1.5.2 of the `blake3` crate started using the new-ish `dep:` notation in `Cargo.toml`, which removes the unintended "digest" feature. Solana currently pins v1.5.0, but this will be a build break when that pin is updated. I think a user has already run into this here: https://solana.stackexchange.com/q/17787/29050.
Copy link
Collaborator

@anza-team anza-team left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the contribution! This repository is no longer in use. Please re-open this pull request in the agave repo: https://github.com/anza-xyz/agave

@anza-team anza-team closed this Nov 27, 2024
@mergify mergify bot added community Community contribution need:merge-assist labels Nov 27, 2024
@oconnor663
Copy link
Author

(I have release blake3 v1.5.5 with a back-compat fix instead.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community Community contribution need:merge-assist
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants