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

Reduce MSRV to 1.65 by polyfilling AtomicUsize::as_ptr(). #277

Merged
merged 1 commit into from
Mar 13, 2025

Conversation

briansmith
Copy link
Contributor

@briansmith briansmith commented Mar 13, 2025

Potentially the MSRV could be reduced further as cargo +1.63.0 check passes, but cargo +1.63.0 test fails:

$ cargo +1.63.0 test
error: package `regex-syntax v0.8.5` cannot be built because it requires
rustc 1.65 or newer, while the currently active rustc version is 1.63.0

@matklad
Copy link
Owner

matklad commented Mar 13, 2025

LGTM! Please also bump patch version in Cargo.toml and add a changelog entry!

@briansmith briansmith changed the title Restore MSRV to 1.63 by polyfilling AtomicUsize::as_ptr(). Reduce MSRV to 1.65 by polyfilling AtomicUsize::as_ptr(). Mar 13, 2025
@briansmith briansmith force-pushed the b/as-ptr branch 2 times, most recently from e4face9 to 9e01e87 Compare March 13, 2025 16:20
Potentially the MSRV could be reduced further as `cargo +1.63.0 check`
passes, but `cargo +1.63.0 test` fails:

```
$ cargo +1.63.0 test
error: package `regex-syntax v0.8.5` cannot be built because it requires
rustc 1.65 or newer, while the currently active rustc version is 1.63.0
```
@briansmith
Copy link
Contributor Author

I have updated the PR to set MSRV to 1.65 since that is the minimum for cargo test to work, because I think any MSRV <= 1.66 is good enough for almost anybody, and because I think we'll benefit from at least MSRV 1.64 by fixing issue #220.

@matklad matklad merged commit a70d907 into matklad:master Mar 13, 2025
1 check passed
@matklad
Copy link
Owner

matklad commented Mar 13, 2025

Thanks!

@briansmith briansmith deleted the b/as-ptr branch March 13, 2025 18:38
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.

None yet

2 participants