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

Bump to minimal MSRV required. #113

Closed
wants to merge 7 commits into from

Conversation

CPerezz
Copy link

@CPerezz CPerezz commented Sep 11, 2023

Resolves #112

CPerezz and others added 7 commits September 8, 2023 10:39
In order to solve:
`error: package `rayon-core v1.11.0` cannot be built because it requires rustc 1.59 or newer, while the currently active rustc version is 1.56.0`

This is due to the fact that:
```
❯ cargo tree -i rayon-core
rayon-core v1.11.0
└── rayon v1.7.0
    └── criterion v0.3.6
        [dev-dependencies]
        └── bls12_381 v0.8.0
```

As you can see here, `criterion` depends on `rayon_core` with `^0.3`. After `rayon` moved up MSRV without bumping minor version, everything broke.

This bumps the MSRV to 1.59 fixing this issue. See: zkcrypto#112
After bumping to `1.59`, I saw the following:
`error: package `regex-syntax v0.7.2` cannot be built because it requires rustc 1.60.0 or newer, while the currently active rustc version is 1.59.0`

```
cargo tree -i regex-syntax
regex-syntax v0.7.2
└── regex v1.8.4
    └── criterion v0.3.6
        [dev-dependencies]
        └── bls12_381 v0.8.0
```

It seems same thing happened as with `rayon_core`. Bumping to `1.60.0` solves all the issues.
It turns out that when `zeroize` and other features are set we get:
`package `memchr v2.6.3` cannot be built because it requires rustc 1.61
or newer`.

Therefore, this should be the last bump needed.
@str4d
Copy link
Member

str4d commented Jul 21, 2024

I checked in an MSRV-compatible Cargo.lock instead of bumping MSRV.

@str4d str4d closed this Jul 21, 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.

Tests can't compile due to rayon MSRV update without new minor bump
2 participants