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

chore: downgrade rust version to nightly-2023-09-05 #3536

Merged
merged 1 commit into from
Nov 29, 2023

Conversation

StackOverflowExcept1on
Copy link
Member

Resolves #3533 (see details about regression)
fixes cargo fuzz coverage

@StackOverflowExcept1on StackOverflowExcept1on added the A0-pleasereview PR is ready to be reviewed by the team label Nov 28, 2023
@shamilsan
Copy link
Contributor

It is better to upgrade it to 1.76: rust-lang/rust#117788 (comment)

We will be able to use new features such as faster compilation: https://blog.rust-lang.org/2023/11/09/parallel-rustc.html

@StackOverflowExcept1on
Copy link
Member Author

@shamilsan did you forget about rust-lang/rust#117211?

@shamilsan
Copy link
Contributor

@shamilsan did you forget about rust-lang/rust#117211?

This coverage causes too many problems. Do we really need it?

@StackOverflowExcept1on
Copy link
Member Author

Well, we can remove it from CI, but the fuzzer won't build.

@techraed techraed merged commit fca104c into master Nov 29, 2023
18 checks passed
@techraed techraed deleted the av/rust-2023-09-05-fuzzer branch November 29, 2023 08:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A0-pleasereview PR is ready to be reviewed by the team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fuzzer coverage doesn't work on current rust toolchain
4 participants