Skip to content

Commit 6d8150f

Browse files
jieyouxujieyouxu
committed
Update run-make tests to use cargo wrapper cmd
1 parent f870a35 commit 6d8150f

File tree

2 files changed

+43
-63
lines changed

2 files changed

+43
-63
lines changed

‎tests/run-make/compiler-builtins/rmake.rs

+25-33
Original file line numberDiff line numberDiff line change
@@ -15,46 +15,38 @@
1515
#![deny(warnings)]
1616

1717
use std::collections::HashSet;
18-
use std::path::PathBuf;
1918

2019
use run_make_support::object::read::Object;
2120
use run_make_support::object::read::archive::ArchiveFile;
2221
use run_make_support::object::{ObjectSection, ObjectSymbol, RelocationTarget};
2322
use run_make_support::rfs::{read, read_dir};
24-
use run_make_support::{cmd, env_var, object};
23+
use run_make_support::{cargo, env_var, object, path, target};
2524

2625
fn main() {
27-
let target_dir = PathBuf::from("target");
28-
let target = env_var("TARGET");
29-
30-
println!("Testing compiler_builtins for {}", target);
31-
32-
let manifest_path = PathBuf::from("Cargo.toml");
33-
34-
let path = env_var("PATH");
35-
let rustc = env_var("RUSTC");
36-
let cargo = env_var("CARGO");
37-
let mut cmd = cmd(cargo);
38-
cmd.args(&[
39-
"build",
40-
"--manifest-path",
41-
manifest_path.to_str().unwrap(),
42-
"-Zbuild-std=core",
43-
"--target",
44-
&target,
45-
])
46-
.env("PATH", path)
47-
.env("RUSTC", rustc)
48-
.env("RUSTFLAGS", "-Copt-level=0 -Cdebug-assertions=yes")
49-
.env("CARGO_TARGET_DIR", &target_dir)
50-
.env("RUSTC_BOOTSTRAP", "1")
51-
// Visual Studio 2022 requires that the LIB env var be set so it can
52-
// find the Windows SDK.
53-
.env("LIB", std::env::var("LIB").unwrap_or_default());
54-
55-
cmd.run();
56-
57-
let rlibs_path = target_dir.join(target).join("debug").join("deps");
26+
let target_dir = path("target");
27+
28+
println!("Testing compiler_builtins for {}", target());
29+
30+
cargo()
31+
.args(&[
32+
"build",
33+
"--manifest-path",
34+
"Cargo.toml",
35+
"-Zbuild-std=core",
36+
"--target",
37+
&target(),
38+
])
39+
.env("PATH", env_var("PATH"))
40+
.env("RUSTC", env_var("RUSTC"))
41+
.env("RUSTFLAGS", "-Copt-level=0 -Cdebug-assertions=yes")
42+
.env("CARGO_TARGET_DIR", &target_dir)
43+
.env("RUSTC_BOOTSTRAP", "1")
44+
// Visual Studio 2022 requires that the LIB env var be set so it can
45+
// find the Windows SDK.
46+
.env("LIB", std::env::var("LIB").unwrap_or_default())
47+
.run();
48+
49+
let rlibs_path = target_dir.join(target()).join("debug").join("deps");
5850
let compiler_builtins_rlib = read_dir(rlibs_path)
5951
.find_map(|e| {
6052
let path = e.unwrap().path();

‎tests/run-make/thumb-none-cortex-m/rmake.rs

+18-30
Original file line numberDiff line numberDiff line change
@@ -14,10 +14,7 @@
1414
1515
//@ only-thumb
1616

17-
use std::path::PathBuf;
18-
19-
use run_make_support::rfs::create_dir;
20-
use run_make_support::{cmd, env_var, target};
17+
use run_make_support::{cargo, cmd, env, env_var, target};
2118

2219
const CRATE: &str = "cortex-m";
2320
const CRATE_URL: &str = "https://github.com/rust-embedded/cortex-m";
@@ -28,32 +25,23 @@ fn main() {
2825
// See below link for git usage:
2926
// https://stackoverflow.com/questions/3489173#14091182
3027
cmd("git").args(["clone", CRATE_URL, CRATE]).run();
31-
std::env::set_current_dir(CRATE).unwrap();
28+
env::set_current_dir(CRATE);
3229
cmd("git").args(["reset", "--hard", CRATE_SHA1]).run();
3330

34-
let target_dir = PathBuf::from("target");
35-
let manifest_path = PathBuf::from("Cargo.toml");
36-
37-
let path = env_var("PATH");
38-
let rustc = env_var("RUSTC");
39-
let cargo = env_var("CARGO");
40-
// FIXME: extract cargo invocations to a proper command
41-
// https://github.com/rust-lang/rust/issues/128734
42-
let mut cmd = cmd(cargo);
43-
cmd.args(&[
44-
"build",
45-
"--manifest-path",
46-
manifest_path.to_str().unwrap(),
47-
"-Zbuild-std=core",
48-
"--target",
49-
&target(),
50-
])
51-
.env("PATH", path)
52-
.env("RUSTC", rustc)
53-
.env("CARGO_TARGET_DIR", &target_dir)
54-
// Don't make lints fatal, but they need to at least warn
55-
// or they break Cargo's target info parsing.
56-
.env("RUSTFLAGS", "-Copt-level=0 -Cdebug-assertions=yes --cap-lints=warn");
57-
58-
cmd.run();
31+
cargo()
32+
.args(&[
33+
"build",
34+
"--manifest-path",
35+
"Cargo.toml",
36+
"-Zbuild-std=core",
37+
"--target",
38+
&target(),
39+
])
40+
.env("PATH", env_var("PATH"))
41+
.env("RUSTC", env_var("RUSTC"))
42+
.env("CARGO_TARGET_DIR", "target")
43+
// Don't make lints fatal, but they need to at least warn
44+
// or they break Cargo's target info parsing.
45+
.env("RUSTFLAGS", "-Copt-level=0 -Cdebug-assertions=yes --cap-lints=warn")
46+
.run();
5947
}

0 commit comments

Comments
 (0)