Skip to content

Conversation

@nmattia
Copy link
Contributor

@nmattia nmattia commented Jul 18, 2025

this drops the nightly toolchain that was used in the fuzzers. Instead,
RUSTC_BOOTSTRAP=1 is set in the fuzzer targets, which has the same
effect without requiring the download of a second toolchain.

NOTE: This vendors the following upstream patch: bazelbuild/rules_rust#3510

@nmattia nmattia added the CI_ALL_BAZEL_TARGETS Runs all bazel targets and uploads them to S3 label Jul 18, 2025
@github-actions github-actions bot added the chore label Jul 18, 2025
@nmattia nmattia removed the CI_ALL_BAZEL_TARGETS Runs all bazel targets and uploads them to S3 label Jul 21, 2025
@nmattia nmattia force-pushed the nm-fuzz-env branch 2 times, most recently from 366583f to 96a545f Compare July 21, 2025 10:27
@nmattia nmattia marked this pull request as ready for review July 21, 2025 10:48
@nmattia nmattia requested review from a team as code owners July 21, 2025 10:48
Copy link
Collaborator

@basvandijk basvandijk left a comment

Choose a reason for hiding this comment

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

Nice simplification!

Copy link
Member

@venkkatesh-sekar venkkatesh-sekar left a comment

Choose a reason for hiding this comment

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

Awesome, thanks for making this change @nmattia

this drops the nightly toolchain that was used in the fuzzers. Instead,
`RUSTC_BOOTSTRAP=1` is set in the fuzzer targets, which has the same
effect without requiring the download of a second toolchain.

NOTE: This vendors the following upstream patch: bazelbuild/rules_rust#3510
@nmattia nmattia added this pull request to the merge queue Jul 21, 2025
Merged via the queue into master with commit 8293fce Jul 21, 2025
27 checks passed
@nmattia nmattia deleted the nm-fuzz-env branch July 21, 2025 16:56
Andrei1998 pushed a commit that referenced this pull request Jul 22, 2025
this drops the nightly toolchain that was used in the fuzzers. Instead,
`RUSTC_BOOTSTRAP=1` is set in the fuzzer targets, which has the same
effect without requiring the download of a second toolchain.

NOTE: This vendors the following upstream patch:
bazelbuild/rules_rust#3510
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants