Remove run_delaying_failure · rust-lang/rust@0de7b92 (original) (raw)

`@@ -156,7 +156,10 @@ You can skip linkcheck with --skip src/tools/linkchecker"

`

156

156

`let _guard =

`

157

157

` builder.msg(Kind::Test, compiler.stage, "Linkcheck", bootstrap_host, bootstrap_host);

`

158

158

`let _time = helpers::timeit(builder);

`

159

``

`-

builder.run_delaying_failure(linkchecker.arg(builder.out.join(host.triple).join("doc")));

`

``

159

`+

builder.run_tracked(

`

``

160

`+

BootstrapCommand::from(linkchecker.arg(builder.out.join(host.triple).join("doc")))

`

``

161

`+

.delay_failure(),

`

``

162

`+

);

`

160

163

`}

`

161

164

``

162

165

`fn should_run(run: ShouldRun<'_>) -> ShouldRun<'_> {

`

`@@ -213,8 +216,11 @@ impl Step for HtmlCheck {

`

213

216

` builder,

`

214

217

`));

`

215

218

``

216

``

`-

builder.run_delaying_failure(

`

217

``

`-

builder.tool_cmd(Tool::HtmlChecker).arg(builder.doc_out(self.target)),

`

``

219

`+

builder.run_tracked(

`

``

220

`+

BootstrapCommand::from(

`

``

221

`+

builder.tool_cmd(Tool::HtmlChecker).arg(builder.doc_out(self.target)),

`

``

222

`+

)

`

``

223

`+

.delay_failure(),

`

218

224

`);

`

219

225

`}

`

220

226

`}

`

`@@ -261,7 +267,7 @@ impl Step for Cargotest {

`

261

267

`.env("RUSTC", builder.rustc(compiler))

`

262

268

`.env("RUSTDOC", builder.rustdoc(compiler));

`

263

269

`add_rustdoc_cargo_linker_args(cmd, builder, compiler.host, LldThreads::No);

`

264

``

`-

builder.run_delaying_failure(cmd);

`

``

270

`+

builder.run_tracked(BootstrapCommand::from(cmd).delay_failure());

`

265

271

`}

`

266

272

`}

`

267

273

``

`@@ -813,7 +819,7 @@ impl Step for RustdocTheme {

`

813

819

`.env("RUSTC_BOOTSTRAP", "1");

`

814

820

` cmd.args(linker_args(builder, self.compiler.host, LldThreads::No));

`

815

821

``

816

``

`-

builder.run_delaying_failure(&mut cmd);

`

``

822

`+

builder.run_tracked(BootstrapCommand::from(&mut cmd).delay_failure());

`

817

823

`}

`

818

824

`}

`

819

825

``

`` @@ -1093,7 +1099,7 @@ HELP: to skip test's attempt to check tidiness, pass --skip src/tools/tidy to

``

1093

1099

`}

`

1094

1100

``

1095

1101

` builder.info("tidy check");

`

1096

``

`-

builder.run_delaying_failure(&mut cmd);

`

``

1102

`+

builder.run_tracked(BootstrapCommand::from(&mut cmd).delay_failure());

`

1097

1103

``

1098

1104

` builder.info("x.py completions check");

`

1099

1105

`let [bash, zsh, fish, powershell] = ["x.py.sh", "x.py.zsh", "x.py.fish", "x.py.ps1"]

`

`@@ -2179,7 +2185,8 @@ impl BookTest {

`

2179

2185

` compiler.host,

`

2180

2186

`);

`

2181

2187

`let _time = helpers::timeit(builder);

`

2182

``

`-

let toolstate = if builder.run_delaying_failure(&mut rustbook_cmd) {

`

``

2188

`+

let cmd = BootstrapCommand::from(&mut rustbook_cmd).delay_failure();

`

``

2189

`+

let toolstate = if builder.run_tracked(cmd).is_success() {

`

2183

2190

`ToolState::TestPass

`

2184

2191

`} else {

`

2185

2192

`ToolState::TestFail

`

`@@ -2371,7 +2378,8 @@ impl Step for RustcGuide {

`

2371

2378

``

2372

2379

`let src = builder.src.join(relative_path);

`

2373

2380

`let mut rustbook_cmd = builder.tool_cmd(Tool::Rustbook);

`

2374

``

`-

let toolstate = if builder.run_delaying_failure(rustbook_cmd.arg("linkcheck").arg(&src)) {

`

``

2381

`+

let cmd = BootstrapCommand::from(rustbook_cmd.arg("linkcheck").arg(&src)).delay_failure();

`

``

2382

`+

let toolstate = if builder.run_tracked(cmd).is_success() {

`

2375

2383

`ToolState::TestPass

`

2376

2384

`} else {

`

2377

2385

`ToolState::TestFail

`

`@@ -2985,7 +2993,7 @@ impl Step for Bootstrap {

`

2985

2993

`.current_dir(builder.src.join("src/bootstrap/"));

`

2986

2994

`// NOTE: we intentionally don't pass test_args here because the args for unittest and cargo test are mutually incompatible.

`

2987

2995

`` // Use python -m unittest manually if you want to pass arguments.

``

2988

``

`-

builder.run_delaying_failure(&mut check_bootstrap);

`

``

2996

`+

builder.run_tracked(BootstrapCommand::from(&mut check_bootstrap).delay_failure());

`

2989

2997

``

2990

2998

`let mut cmd = Command::new(&builder.initial_cargo);

`

2991

2999

` cmd.arg("test")

`

`@@ -3062,7 +3070,7 @@ impl Step for TierCheck {

`

3062

3070

`self.compiler.host,

`

3063

3071

`self.compiler.host,

`

3064

3072

`);

`

3065

``

`-

builder.run_delaying_failure(&mut cargo.into());

`

``

3073

`+

builder.run_tracked(BootstrapCommand::from(&mut cargo.into()).delay_failure());

`

3066

3074

`}

`

3067

3075

`}

`

3068

3076

``

`@@ -3148,7 +3156,7 @@ impl Step for RustInstaller {

`

3148

3156

` cmd.env("CARGO", &builder.initial_cargo);

`

3149

3157

` cmd.env("RUSTC", &builder.initial_rustc);

`

3150

3158

` cmd.env("TMP_DIR", &tmpdir);

`

3151

``

`-

builder.run_delaying_failure(&mut cmd);

`

``

3159

`+

builder.run_tracked(BootstrapCommand::from(&mut cmd).delay_failure());

`

3152

3160

`}

`

3153

3161

``

3154

3162

`fn should_run(run: ShouldRun<'_>) -> ShouldRun<'_> {

`