Skip to content

Commit

Permalink
test(build-std): dont require rustup (#14933)
Browse files Browse the repository at this point in the history
### What does this PR try to resolve?

It checks rustup existence even when test is skipped for other reasons.

### How should we test and review this PR?

See
<rust-lang/rust#134278 (comment)>.

### Additional information
  • Loading branch information
ehuss authored Dec 14, 2024
2 parents 7847c03 + 80409f7 commit 769f622
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion tests/testsuite/standard_lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -389,7 +389,7 @@ fn check_core() {
.run();
}

#[cargo_test(build_std_mock, requires = "rustup")]
#[cargo_test(build_std_mock)]
fn build_std_with_no_arg_for_core_only_target() {
let has_rustup_aarch64_unknown_none = std::process::Command::new("rustup")
.args(["target", "list", "--installed"])
Expand Down

0 comments on commit 769f622

Please sign in to comment.