From 384556510c16b14f5faeb2c5581bde55a602e1f2 Mon Sep 17 00:00:00 2001 From: Ximin Luo Date: Tue, 8 Sep 2020 17:38:19 +0100 Subject: [PATCH] Ignore submodules that we're not building, or that are unused like Forwarded: not-needed mdbook-linkcheck that pull in a ton of dependencies that are unneeded. Gbp-Pq: Name d-0000-ignore-removed-submodules.patch --- Cargo.toml | 26 -------------------------- src/bootstrap/bootstrap.py | 4 ---- src/bootstrap/builder.rs | 7 +------ src/bootstrap/doc.rs | 2 +- src/bootstrap/test.rs | 16 ++++++++-------- src/bootstrap/tool.rs | 4 ++-- src/doc/index.md | 2 +- src/tools/rustbook/Cargo.toml | 8 +------- src/tools/tidy/src/deps.rs | 4 ++-- 9 files changed, 16 insertions(+), 57 deletions(-) diff --git a/Cargo.toml b/Cargo.toml index a242f090fb..3cfadf9428 100644 --- a/Cargo.toml +++ b/Cargo.toml @@ -6,7 +6,6 @@ members = [ "src/libtest", "src/librustc_codegen_llvm", "src/tools/cargotest", - "src/tools/clippy", "src/tools/compiletest", "src/tools/error_index_generator", "src/tools/linkchecker", @@ -14,14 +13,8 @@ members = [ "src/tools/unstable-book-gen", "src/tools/tidy", "src/tools/build-manifest", - "src/tools/remote-test-client", - "src/tools/remote-test-server", "src/tools/rust-installer", - "src/tools/cargo", "src/tools/rustdoc", - "src/tools/rls", - "src/tools/rustfmt", - "src/tools/miri", "src/tools/rustdoc-themes", ] exclude = [ @@ -46,29 +39,10 @@ debug-assertions = false debug = false debug-assertions = false -# We want the RLS to use the version of Cargo that we've got vendored in this -# repository to ensure that the same exact version of Cargo is used by both the -# RLS and the Cargo binary itself. The RLS depends on Cargo as a git repository -# so we use a `[patch]` here to override the github repository with our local -# vendored copy. -[patch."https://github.com/rust-lang/cargo"] -cargo = { path = "src/tools/cargo" } - [patch.crates-io] -# Similar to Cargo above we want the RLS to use a vendored version of `rustfmt` -# that we're shipping as well (to ensure that the rustfmt in RLS and the -# `rustfmt` executable are the same exact version). -rustfmt-nightly = { path = "src/tools/rustfmt" } - -# See comments in `src/tools/rustc-workspace-hack/README.md` for what's going on -# here -rustc-workspace-hack = { path = 'src/tools/rustc-workspace-hack' } # See comments in `tools/rustc-std-workspace-core/README.md` for what's going on # here rustc-std-workspace-core = { path = 'src/tools/rustc-std-workspace-core' } rustc-std-workspace-alloc = { path = 'src/tools/rustc-std-workspace-alloc' } rustc-std-workspace-std = { path = 'src/tools/rustc-std-workspace-std' } - -[patch."https://github.com/rust-lang/rust-clippy"] -clippy_lints = { path = "src/tools/clippy/clippy_lints" } diff --git a/src/bootstrap/bootstrap.py b/src/bootstrap/bootstrap.py index 63a484505b..e2f7fd95cf 100644 --- a/src/bootstrap/bootstrap.py +++ b/src/bootstrap/bootstrap.py @@ -671,10 +671,6 @@ class RustBuild(object): os.path.join(self.rust_root, "src/bootstrap/Cargo.toml")] for _ in range(1, self.verbose): args.append("--verbose") - if self.use_locked_deps: - args.append("--locked") - if self.use_vendored_sources: - args.append("--frozen") run(args, env=env, verbose=self.verbose) def build_triple(self): diff --git a/src/bootstrap/builder.rs b/src/bootstrap/builder.rs index 8b0ad169cf..522adeb90b 100644 --- a/src/bootstrap/builder.rs +++ b/src/bootstrap/builder.rs @@ -453,8 +453,6 @@ impl<'a> Builder<'a> { dist::Miri, dist::LlvmTools, dist::Lldb, - dist::Extended, - dist::HashSign ), Kind::Install => describe!( install::Docs, @@ -1211,10 +1209,7 @@ impl<'a> Builder<'a> { } } - if self.config.locked_deps { - cargo.arg("--locked"); - } - if self.config.vendor || self.is_sudo { + if self.is_sudo { cargo.arg("--frozen"); } diff --git a/src/bootstrap/doc.rs b/src/bootstrap/doc.rs index 57f6dd9603..0a9b04dc2e 100644 --- a/src/bootstrap/doc.rs +++ b/src/bootstrap/doc.rs @@ -113,7 +113,7 @@ pub struct CargoBook { impl Step for CargoBook { type Output = (); - const DEFAULT: bool = true; + const DEFAULT: bool = false; fn should_run(run: ShouldRun<'_>) -> ShouldRun<'_> { let builder = run.builder; diff --git a/src/bootstrap/test.rs b/src/bootstrap/test.rs index f3b2a73d3c..59760888c8 100644 --- a/src/bootstrap/test.rs +++ b/src/bootstrap/test.rs @@ -1590,14 +1590,14 @@ impl Step for RustcGuide { } fn run(self, builder: &Builder<'_>) { - let src = builder.src.join("src/doc/rustc-guide"); - let mut rustbook_cmd = builder.tool_cmd(Tool::Rustbook); - let toolstate = if try_run(builder, rustbook_cmd.arg("linkcheck").arg(&src)) { - ToolState::TestPass - } else { - ToolState::TestFail - }; - builder.save_toolstate("rustc-guide", toolstate); + //let src = builder.src.join("src/doc/rustc-guide"); + //let mut rustbook_cmd = builder.tool_cmd(Tool::Rustbook); + //let toolstate = if try_run(builder, rustbook_cmd.arg("linkcheck").arg(&src)) { + // ToolState::TestPass + //} else { + // ToolState::TestFail + //}; + builder.save_toolstate("rustc-guide", ToolState::TestPass); } } diff --git a/src/bootstrap/tool.rs b/src/bootstrap/tool.rs index 73a4dda74e..a45eabde3c 100644 --- a/src/bootstrap/tool.rs +++ b/src/bootstrap/tool.rs @@ -277,14 +277,14 @@ pub fn prepare_tool_cargo( } fn rustbook_features() -> Vec { - let mut features = Vec::new(); + let features = Vec::new(); // Due to CI budged and risk of spurious failures we want to limit jobs running this check. // At same time local builds should run it regardless of the platform. // `CiEnv::None` means it's local build and `CHECK_LINKS` is defined in x86_64-gnu-tools to // explicitly enable it on single job if CiEnv::current() == CiEnv::None || env::var("CHECK_LINKS").is_ok() { - features.push("linkcheck".to_string()); + // features.push("linkcheck".to_string()); } features diff --git a/src/doc/index.md b/src/doc/index.md index 0a2a80e8fd..5861a0d186 100644 --- a/src/doc/index.md +++ b/src/doc/index.md @@ -87,7 +87,7 @@ accomplishing various tasks. ## The Cargo Book -[The Cargo Book](cargo/index.html) is a guide to Cargo, Rust's build tool and dependency manager. +[The Cargo Book](../../cargo-doc/doc/index.html) is a guide to Cargo, Rust's build tool and dependency manager. ## The Rustdoc Book diff --git a/src/tools/rustbook/Cargo.toml b/src/tools/rustbook/Cargo.toml index 327de29037..bc9ce5f92c 100644 --- a/src/tools/rustbook/Cargo.toml +++ b/src/tools/rustbook/Cargo.toml @@ -6,21 +6,15 @@ license = "MIT OR Apache-2.0" edition = "2018" [features] -linkcheck = ["mdbook-linkcheck", "codespan-reporting"] +linkcheck = [] [dependencies] clap = "2.25.0" failure = "0.1" -mdbook-linkcheck = { version = "0.5.0", optional = true } # Keep in sync with mdbook-linkcheck. codespan-reporting = { version = "0.5", optional = true } -# A noop dependency that changes in the Rust repository, it's a bit of a hack. -# See the `src/tools/rustc-workspace-hack/README.md` file in `rust-lang/rust` -# for more information. -rustc-workspace-hack = "1.0.0" - [dependencies.mdbook] version = "0.3.0" default-features = false diff --git a/src/tools/tidy/src/deps.rs b/src/tools/tidy/src/deps.rs index 36e412975b..cd185a62e8 100644 --- a/src/tools/tidy/src/deps.rs +++ b/src/tools/tidy/src/deps.rs @@ -409,8 +409,8 @@ fn check_crate_duplicate(resolve: &Resolve, bad: &mut bool) { // to accidentally sneak into our dependency graph, in order to ensure we keep our CI times // under control. - "cargo", - "rustc-ap-syntax", + //"cargo", + //"rustc-ap-syntax", ]; let mut name_to_id: HashMap<_, Vec<_>> = HashMap::new(); for node in resolve.nodes.iter() { -- 2.30.2