Skip to content

Bump rand_core to v0.10; MSRV 1.85#149

Merged
str4d merged 1 commit intozkcrypto:release-0.14.0from
tarcieri:rand_core/v0.10
Feb 26, 2026
Merged

Bump rand_core to v0.10; MSRV 1.85#149
str4d merged 1 commit intozkcrypto:release-0.14.0from
tarcieri:rand_core/v0.10

Conversation

@tarcieri
Copy link
Contributor

@tarcieri tarcieri commented Feb 1, 2026

Cargo.toml Outdated
[dev-dependencies]
blake2b_simd = "1"
rand = "0.9"
getrandom = { version = "0.4.0-rc.1", features = ["sys_rng"] }
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Note: rand hasn't been updated yet, however it seems like you don't need the full rand anyway

Copy link
Contributor Author

Choose a reason for hiding this comment

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

rand v0.10 is out now if you'd prefer that, although this seems like fewer dependencies.

]
edition = "2021"
rust-version = "1.63"
rust-version = "1.85"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Bumped this because rand_core is a hard dependency and v0.10 carries an MSRV of 1.85 (and has also been bumped to 2024 edition, which could be done in a followup here if desired)

tarcieri added a commit to tarcieri/group that referenced this pull request Feb 1, 2026
NOTE: depends on zkcrypto/ff#149 and forthcoming releases of `rand` and
`rand_xorshift`

Release notes:

https://github.com/rust-random/rand_core/releases/tag/v0.10.0
tarcieri added a commit to tarcieri/group that referenced this pull request Feb 3, 2026
NOTE: depends on zkcrypto/ff#149 and forthcoming releases of `rand` and
`rand_xorshift`

Release notes:

https://github.com/rust-random/rand_core/releases/tag/v0.10.0
tarcieri added a commit to tarcieri/group that referenced this pull request Feb 3, 2026
NOTE: depends on zkcrypto/ff#149 and forthcoming releases of `rand` and
`rand_xorshift`

Release notes:

https://github.com/rust-random/rand_core/releases/tag/v0.10.0
tarcieri added a commit to tarcieri/group that referenced this pull request Feb 5, 2026
tarcieri added a commit to tarcieri/group that referenced this pull request Feb 5, 2026
tarcieri added a commit to tarcieri/group that referenced this pull request Feb 5, 2026
@ebfull
Copy link
Collaborator

ebfull commented Feb 6, 2026

@str4d How do you feel about such a huge MSRV bump for this? I think we really need to get the mess involving rand addressed throughout our stack, and it looks like we'll need to do this...

(oh, 1.85 is already MSRV in all the librustzcash crates, so this is basically a no-op for our stack.)

@tarcieri
Copy link
Contributor Author

tarcieri commented Feb 6, 2026

@ebfull FWIW all of the RustCrypto crates will be bumping MSRV to 1.85 in the forthcoming release

Comment on lines -94 to -95
# Requires #![deny(rustdoc::broken_intra_doc_links)] in crates.
- run: sudo apt-get -y install libfontconfig1-dev
Copy link
Contributor Author

@tarcieri tarcieri Feb 23, 2026

Choose a reason for hiding this comment

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

This is no longer installable for whatever reason and breaks the build:

https://github.com/zkcrypto/ff/actions/runs/22320260133/job/64576314383?pr=149

@ebfull
Copy link
Collaborator

ebfull commented Feb 25, 2026

I updated the RFC with the rename for try_from_rng -> try_random.

Copy link
Member

@str4d str4d left a comment

Choose a reason for hiding this comment

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

utACK dbff4c2. Merging this now so I can cut a preview build with it; I'll do the rename in that next PR.

@str4d str4d merged commit bcd2cfa into zkcrypto:release-0.14.0 Feb 26, 2026
11 checks passed
tarcieri added a commit to tarcieri/ff that referenced this pull request Feb 26, 2026
Followup to zkcrypto#149, since the name has been changed in zkcrypto/rfcs#1
tarcieri added a commit to tarcieri/ff that referenced this pull request Feb 26, 2026
Followup to zkcrypto#149, since the name has been changed in zkcrypto/rfcs#1
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.

3 participants