Skip to content

Commit a9313e9

Browse files
authored
Rollup merge of #103648 - jyn514:no-preview, r=Mark-Simulacrum
Don't set `is_preview` for clippy and rustfmt These have been shipped on stable for many years now and it would be very disruptive to ever remove them. Remove the `-preview` suffix from their dist components. Based on #102565.
2 parents 41e0363 + fb3e724 commit a9313e9

File tree

4 files changed

+5
-7
lines changed

4 files changed

+5
-7
lines changed

src/bootstrap/dist.rs

-4
Original file line numberDiff line numberDiff line change
@@ -1191,7 +1191,6 @@ impl Step for Clippy {
11911191

11921192
let mut tarball = Tarball::new(builder, "clippy", &target.triple);
11931193
tarball.set_overlay(OverlayKind::Clippy);
1194-
tarball.is_preview(true);
11951194
tarball.add_file(clippy, "bin", 0o755);
11961195
tarball.add_file(cargoclippy, "bin", 0o755);
11971196
tarball.add_legal_and_readme_to("share/doc/clippy");
@@ -1289,7 +1288,6 @@ impl Step for Rustfmt {
12891288
.expect("cargo fmt expected to build - essential tool");
12901289
let mut tarball = Tarball::new(builder, "rustfmt", &target.triple);
12911290
tarball.set_overlay(OverlayKind::Rustfmt);
1292-
tarball.is_preview(true);
12931291
tarball.add_file(rustfmt, "bin", 0o755);
12941292
tarball.add_file(cargofmt, "bin", 0o755);
12951293
tarball.add_legal_and_readme_to("share/doc/rustfmt");
@@ -1550,8 +1548,6 @@ impl Step for Extended {
15501548
format!("{}-{}", name, target.triple)
15511549
} else if name == "rust-analyzer" {
15521550
"rust-analyzer-preview".to_string()
1553-
} else if name == "clippy" {
1554-
"clippy-preview".to_string()
15551551
} else if name == "rust-demangler" {
15561552
"rust-demangler-preview".to_string()
15571553
} else if name == "miri" {

src/bootstrap/download.rs

+2
Original file line numberDiff line numberDiff line change
@@ -326,6 +326,8 @@ impl Config {
326326
}
327327

328328
let filename = format!("rustfmt-{version}-{build}.tar.xz", build = host.triple);
329+
// cfg(bootstrap): will need to be changed from `rustfmt-preview` to `rustfmt` the next time you run `bump-stage0`.
330+
// See <https://github.com/rust-lang/rust/pull/103648>
329331
self.download_component(DownloadSource::Dist, filename, "rustfmt-preview", &date, "stage0");
330332

331333
self.fix_bin_or_dylib(&bin_root.join("bin").join("rustfmt"));

src/tools/build-manifest/src/versions.rs

+2-2
Original file line numberDiff line numberDiff line change
@@ -49,10 +49,10 @@ pkg_type! {
4949
Cargo = "cargo",
5050
HtmlDocs = "rust-docs",
5151
RustAnalysis = "rust-analysis",
52+
Clippy = "clippy",
53+
Rustfmt = "rustfmt",
5254
Rls = "rls"; preview = true,
5355
RustAnalyzer = "rust-analyzer"; preview = true,
54-
Clippy = "clippy"; preview = true,
55-
Rustfmt = "rustfmt"; preview = true,
5656
LlvmTools = "llvm-tools"; preview = true,
5757
Miri = "miri"; preview = true,
5858
JsonDocs = "rust-docs-json"; preview = true,

src/tools/bump-stage0/src/main.rs

+1-1
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ use std::convert::TryInto;
66

77
const PATH: &str = "src/stage0.json";
88
const COMPILER_COMPONENTS: &[&str] = &["rustc", "rust-std", "cargo"];
9-
const RUSTFMT_COMPONENTS: &[&str] = &["rustfmt-preview"];
9+
const RUSTFMT_COMPONENTS: &[&str] = &["rustfmt"];
1010

1111
struct Tool {
1212
config: Config,

0 commit comments

Comments
 (0)