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

deps: Downgrade spl crates for upcoming yank #11

Closed
wants to merge 1 commit into from
Closed

deps: Downgrade spl crates for upcoming yank #11

wants to merge 1 commit into from

Conversation

joncinque
Copy link

Problem

New projects using spl crates are having a hard time because of the flexible dependency on v2 or v1 of the Solana crates -- cargo picks up the highest version of the dependency if there's no lockfile, and downstream users are left with confusing error messages about multiple versions of Solana crates present in the project.

To make it easier for new projects, we are yanking SPL crates that have the flexible dependency, and spl-type-length-value 0.3.1 is one of those.

This crate is the only dependency that explicitly uses it.

Summary of Changes

Downgrade spl-token-2022 to v1 and spl-type-length-value to v0.3.0 to avoid using it.

More information at solana-labs/solana-program-library#6897

Problem

New projects using spl crates are having a hard time because of the flexible
dependency on v2 or v1 of the Solana crates -- cargo picks up the highest
version of the dependency if there's no lockfile, and downstream users are
left with confusing error messages about multiple versions of Solana crates
present in the project.

To make it easier for new projects, we are yanking a lot of SPL crates that
have the flexible dependency, and spl-type-length-value 0.3.1 is one of those.

This crate is the only dependency that explicitly uses it.

Summary of Changes

Downgrade spl-token-2022 to v1 and spl-type-length-value to v0.3.0 to avoid
using it.

More information at solana-labs/solana-program-library#6897
@cavemanloverboy
Copy link
Owner

Thanks for this. I'll find some time this week to make sure everything works and merge.

@joncinque joncinque closed this by deleting the head repository Jul 16, 2024
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