Skip to content

Commit d3d7a6d

Browse files
committed
remove rls support from bootstrap
Signed-off-by: onur-ozkan <[email protected]>
1 parent d2b52c5 commit d3d7a6d

File tree

8 files changed

+2
-58
lines changed

8 files changed

+2
-58
lines changed

src/bootstrap/src/core/build_steps/check.rs

-1
Original file line numberDiff line numberDiff line change
@@ -454,7 +454,6 @@ tool_check_step!(Rustdoc { path: "src/tools/rustdoc", alt_path: "src/librustdoc"
454454
tool_check_step!(Clippy { path: "src/tools/clippy" });
455455
tool_check_step!(Miri { path: "src/tools/miri" });
456456
tool_check_step!(CargoMiri { path: "src/tools/miri/cargo-miri" });
457-
tool_check_step!(Rls { path: "src/tools/rls" });
458457
tool_check_step!(Rustfmt { path: "src/tools/rustfmt" });
459458
tool_check_step!(MiroptTestTools { path: "src/tools/miropt-test-tools" });
460459
tool_check_step!(TestFloatParse { path: "src/etc/test-float-parse" });

src/bootstrap/src/core/build_steps/clippy.rs

-1
Original file line numberDiff line numberDiff line change
@@ -346,7 +346,6 @@ lint_any!(
346346
OptDist, "src/tools/opt-dist", "opt-dist";
347347
RemoteTestClient, "src/tools/remote-test-client", "remote-test-client";
348348
RemoteTestServer, "src/tools/remote-test-server", "remote-test-server";
349-
Rls, "src/tools/rls", "rls";
350349
RustAnalyzer, "src/tools/rust-analyzer", "rust-analyzer";
351350
Rustdoc, "src/librustdoc", "clippy";
352351
Rustfmt, "src/tools/rustfmt", "rustfmt";

src/bootstrap/src/core/build_steps/dist.rs

-43
Original file line numberDiff line numberDiff line change
@@ -1157,48 +1157,6 @@ impl Step for Cargo {
11571157
}
11581158
}
11591159

1160-
#[derive(Debug, PartialOrd, Ord, Clone, Hash, PartialEq, Eq)]
1161-
pub struct Rls {
1162-
pub compiler: Compiler,
1163-
pub target: TargetSelection,
1164-
}
1165-
1166-
impl Step for Rls {
1167-
type Output = Option<GeneratedTarball>;
1168-
const ONLY_HOSTS: bool = true;
1169-
const DEFAULT: bool = true;
1170-
1171-
fn should_run(run: ShouldRun<'_>) -> ShouldRun<'_> {
1172-
let default = should_build_extended_tool(run.builder, "rls");
1173-
run.alias("rls").default_condition(default)
1174-
}
1175-
1176-
fn make_run(run: RunConfig<'_>) {
1177-
run.builder.ensure(Rls {
1178-
compiler: run.builder.compiler_for(
1179-
run.builder.top_stage,
1180-
run.builder.config.build,
1181-
run.target,
1182-
),
1183-
target: run.target,
1184-
});
1185-
}
1186-
1187-
fn run(self, builder: &Builder<'_>) -> Option<GeneratedTarball> {
1188-
let compiler = self.compiler;
1189-
let target = self.target;
1190-
1191-
let rls = builder.ensure(tool::Rls { compiler, target });
1192-
1193-
let mut tarball = Tarball::new(builder, "rls", &target.triple);
1194-
tarball.set_overlay(OverlayKind::Rls);
1195-
tarball.is_preview(true);
1196-
tarball.add_file(rls.tool_path, "bin", 0o755);
1197-
tarball.add_legal_and_readme_to("share/doc/rls");
1198-
Some(tarball.generate())
1199-
}
1200-
}
1201-
12021160
#[derive(Debug, PartialOrd, Ord, Clone, Hash, PartialEq, Eq)]
12031161
pub struct RustAnalyzer {
12041162
pub compiler: Compiler,
@@ -1528,7 +1486,6 @@ impl Step for Extended {
15281486
add_component!("rust-json-docs" => JsonDocs { host: target });
15291487
add_component!("cargo" => Cargo { compiler, target });
15301488
add_component!("rustfmt" => Rustfmt { compiler, target });
1531-
add_component!("rls" => Rls { compiler, target });
15321489
add_component!("rust-analyzer" => RustAnalyzer { compiler, target });
15331490
add_component!("llvm-components" => LlvmTools { target });
15341491
add_component!("clippy" => Clippy { compiler, target });

src/bootstrap/src/core/build_steps/setup.rs

+1-3
Original file line numberDiff line numberDiff line change
@@ -85,9 +85,7 @@ impl FromStr for Profile {
8585
"lib" | "library" => Ok(Profile::Library),
8686
"compiler" => Ok(Profile::Compiler),
8787
"maintainer" | "dist" | "user" => Ok(Profile::Dist),
88-
"tools" | "tool" | "rustdoc" | "clippy" | "miri" | "rustfmt" | "rls" => {
89-
Ok(Profile::Tools)
90-
}
88+
"tools" | "tool" | "rustdoc" | "clippy" | "miri" | "rustfmt" => Ok(Profile::Tools),
9189
"none" => Ok(Profile::None),
9290
"llvm" | "codegen" => Err("the \"llvm\" and \"codegen\" profiles have been removed,\
9391
use \"compiler\" instead which has the same functionality"

src/bootstrap/src/core/build_steps/tool.rs

-2
Original file line numberDiff line numberDiff line change
@@ -228,7 +228,6 @@ pub fn prepare_tool_cargo(
228228
let mut features = extra_features.to_vec();
229229
if builder.build.config.cargo_native_static {
230230
if path.ends_with("cargo")
231-
|| path.ends_with("rls")
232231
|| path.ends_with("clippy")
233232
|| path.ends_with("miri")
234233
|| path.ends_with("rustfmt")
@@ -1231,7 +1230,6 @@ tool_extended!(CargoMiri {
12311230
stable: false,
12321231
add_bins_to_sysroot: ["cargo-miri"]
12331232
});
1234-
tool_extended!(Rls { path: "src/tools/rls", tool_name: "rls", stable: true });
12351233
tool_extended!(Rustfmt {
12361234
path: "src/tools/rustfmt",
12371235
tool_name: "rustfmt",

src/bootstrap/src/core/builder/mod.rs

-4
Original file line numberDiff line numberDiff line change
@@ -895,7 +895,6 @@ impl<'a> Builder<'a> {
895895
tool::RemoteTestClient,
896896
tool::RustInstaller,
897897
tool::Cargo,
898-
tool::Rls,
899898
tool::RustAnalyzer,
900899
tool::RustAnalyzerProcMacroSrv,
901900
tool::Rustdoc,
@@ -938,7 +937,6 @@ impl<'a> Builder<'a> {
938937
clippy::OptDist,
939938
clippy::RemoteTestClient,
940939
clippy::RemoteTestServer,
941-
clippy::Rls,
942940
clippy::RustAnalyzer,
943941
clippy::Rustdoc,
944942
clippy::Rustfmt,
@@ -956,7 +954,6 @@ impl<'a> Builder<'a> {
956954
check::Miri,
957955
check::CargoMiri,
958956
check::MiroptTestTools,
959-
check::Rls,
960957
check::Rustfmt,
961958
check::RustAnalyzer,
962959
check::TestFloatParse,
@@ -1071,7 +1068,6 @@ impl<'a> Builder<'a> {
10711068
dist::Analysis,
10721069
dist::Src,
10731070
dist::Cargo,
1074-
dist::Rls,
10751071
dist::RustAnalyzer,
10761072
dist::Rustfmt,
10771073
dist::Clippy,

src/bootstrap/src/lib.rs

+1-1
Original file line numberDiff line numberDiff line change
@@ -253,7 +253,7 @@ pub enum Mode {
253253
/// Build a tool which uses the locally built rustc and the target std,
254254
/// placing the output in the "stageN-tools" directory. This is used for
255255
/// anything that needs a fully functional rustc, such as rustdoc, clippy,
256-
/// cargo, rls, rustfmt, miri, etc.
256+
/// cargo, rustfmt, miri, etc.
257257
ToolRustc,
258258
}
259259

src/bootstrap/src/utils/tarball.rs

-3
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,6 @@ pub(crate) enum OverlayKind {
2222
Clippy,
2323
Miri,
2424
Rustfmt,
25-
Rls,
2625
RustAnalyzer,
2726
RustcCodegenCranelift,
2827
LlvmBitcodeLinker,
@@ -56,7 +55,6 @@ impl OverlayKind {
5655
"src/tools/rustfmt/LICENSE-APACHE",
5756
"src/tools/rustfmt/LICENSE-MIT",
5857
],
59-
OverlayKind::Rls => &["src/tools/rls/README.md", "LICENSE-APACHE", "LICENSE-MIT"],
6058
OverlayKind::RustAnalyzer => &[
6159
"src/tools/rust-analyzer/README.md",
6260
"src/tools/rust-analyzer/LICENSE-APACHE",
@@ -90,7 +88,6 @@ impl OverlayKind {
9088
OverlayKind::Rustfmt => {
9189
builder.rustfmt_info.version(builder, &builder.release_num("rustfmt"))
9290
}
93-
OverlayKind::Rls => builder.release(&builder.release_num("rls")),
9491
OverlayKind::RustAnalyzer => builder
9592
.rust_analyzer_info
9693
.version(builder, &builder.release_num("rust-analyzer/crates/rust-analyzer")),

0 commit comments

Comments
 (0)