diff options
| author | Joshua Nelson <jnelson@cloudflare.com> | 2022-04-10 16:37:24 -0500 |
|---|---|---|
| committer | Joshua Nelson <jnelson@cloudflare.com> | 2022-04-10 17:15:31 -0500 |
| commit | 986c1687f868e7839f4e967bba32aeec6b2df5da (patch) | |
| tree | 6232eac6d679ca644b217161707ff574ece45b04 /src | |
| parent | 027a232755fa9728e9699337267f6675dfd0a8ba (diff) | |
| download | rust-986c1687f868e7839f4e967bba32aeec6b2df5da.tar.gz rust-986c1687f868e7839f4e967bba32aeec6b2df5da.zip | |
Remove duplicate aliases for `codegen_{cranelift,gcc}`
Bootstrap already allows selecting these in `PathSet::has`, which allows any string that matches the end of a full path. I found these by adding `assert!(path.exists())` in `StepDescription::paths`. I think ideally we wouldn't have any aliases that aren't paths, but I've held off on enforcing that here since it may be controversial, I'll open a separate PR.
Diffstat (limited to 'src')
| -rw-r--r-- | src/bootstrap/check.rs | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/src/bootstrap/check.rs b/src/bootstrap/check.rs index 28e7f1fdca7..432a6c34ed5 100644 --- a/src/bootstrap/check.rs +++ b/src/bootstrap/check.rs @@ -243,12 +243,7 @@ impl Step for CodegenBackend { const DEFAULT: bool = true; fn should_run(run: ShouldRun<'_>) -> ShouldRun<'_> { - run.paths(&[ - "compiler/rustc_codegen_cranelift", - "rustc_codegen_cranelift", - "compiler/rustc_codegen_gcc", - "rustc_codegen_gcc", - ]) + run.paths(&["compiler/rustc_codegen_cranelift", "compiler/rustc_codegen_gcc"]) } fn make_run(run: RunConfig<'_>) { |