Include --check-cfg=cfg(kani)
in the rust flags to avoid a warning about an unknown cfg
.
#3187
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Starting with the 2024-05-05 toolchain (and the upcoming Rust 1.80 release), the
unexpected_cfgs
lint has been turned on by default. As a result, runningcargo kani
on a crate that has a#[cfg(kani)]
results in a warning (see #3186). To avoid this warning, this PR adds--check-cfg=cfg(kani)
toRUSTFLAGS
when Kani invokescargo
.Call-outs: On such packages, doing a
cargo build
will also result in this warning, unless:is added to the package's
build.rs
file. However, this warning would only occur withcargo build
if the package uses the 2024-05-05 toolchain (or newer), or the Rust version used in the package is upgraded to 1.80 (when it's released at the end of July 2024). Since we're likely to release a new version of Kani sooner than the 1.80 release, this PR mitigates the issue that is more likely to impact users (a warning fromcargo kani
).Resolves #3186
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 and MIT licenses.