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 features const_mut_refs and use_spin_nightly #77

Merged
merged 2 commits into from
Feb 26, 2023

Conversation

jannic
Copy link
Contributor

@jannic jannic commented Feb 24, 2023

Since rust 1.61, the underlying const functions are available in stable rust, and lock_api >= 0.4.7 automatically uses them. (Feature const_fn_trait_bound, rust-lang/rust#93827)

To avoid a breaking change, the features are still listed in Cargo.toml, but have no effect and are marked as deprecated. This bumps the minimum supported rust version to 1.61.

Since rust 1.61, the underlying const functions are available in
stable rust, and lock_api >= 0.4.7 automatically uses them. (Feature
const_fn_trait_bound, rust-lang/rust#93827)

To avoid a breaking change, the features are still listed in Cargo.toml,
but have no effect and are marked as deprecated.  This bumps the minimum
supported rust version to 1.61.
Copy link
Member

@phil-opp phil-opp left a comment

Choose a reason for hiding this comment

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

Thanks a lot!

Cargo.toml Outdated Show resolved Hide resolved
@phil-opp phil-opp enabled auto-merge February 26, 2023 11:35
@phil-opp phil-opp merged commit 07583b1 into rust-osdev:main Feb 26, 2023
@jannic jannic deleted the const-is-stable branch February 26, 2023 14:41
@jannic
Copy link
Contributor Author

jannic commented Mar 3, 2023

@phil-opp, would you consider doing a release of linked-list-allocator, so it can be used with stable rust?

@phil-opp
Copy link
Member

phil-opp commented Mar 4, 2023

Sure, published as v0.10.5.

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