[INFO] fetching crate kumitateru 0.4.0... [INFO] testing kumitateru-0.4.0 against try#b83b707f97d809763b7861afa7638871f3339a33 for pr-145838-1 [INFO] extracting crate kumitateru 0.4.0 into /workspace/builds/worker-2-tc2/source [INFO] started tweaking crates.io crate kumitateru 0.4.0 [INFO] finished tweaking crates.io crate kumitateru 0.4.0 [INFO] tweaked toml for crates.io crate kumitateru 0.4.0 written to /workspace/builds/worker-2-tc2/source/Cargo.toml [INFO] validating manifest of crates.io crate kumitateru 0.4.0 on toolchain b83b707f97d809763b7861afa7638871f3339a33 [INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+b83b707f97d809763b7861afa7638871f3339a33" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }` [INFO] crate crates.io crate kumitateru 0.4.0 already has a lockfile, it will not be regenerated [INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+b83b707f97d809763b7861afa7638871f3339a33" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }` [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-2-tc2/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-2-tc2/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/cargo-home:/opt/rustwide/cargo-home:ro,Z" "-v" "/var/lib/crater-agent-workspace/rustup-home:/opt/rustwide/rustup-home:ro,Z" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-w" "/opt/rustwide/workdir" "-m" "1610612736" "--user" "0:0" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:7ad1b28ee6f5f7f699f6cf7015098d6ccdd96d6f2d78dd06228f5b4c9faf309c" "/opt/rustwide/cargo-home/bin/cargo" "+b83b707f97d809763b7861afa7638871f3339a33" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }` [INFO] [stdout] 41d68ab6a27331c7947e638bf0a6560ef5eb9f4f3ed65bcb590c6fb4b480972f [INFO] running `Command { std: "docker" "start" "-a" "41d68ab6a27331c7947e638bf0a6560ef5eb9f4f3ed65bcb590c6fb4b480972f", kill_on_drop: false }` [INFO] running `Command { std: "docker" "inspect" "41d68ab6a27331c7947e638bf0a6560ef5eb9f4f3ed65bcb590c6fb4b480972f", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "41d68ab6a27331c7947e638bf0a6560ef5eb9f4f3ed65bcb590c6fb4b480972f", kill_on_drop: false }` [INFO] [stdout] 41d68ab6a27331c7947e638bf0a6560ef5eb9f4f3ed65bcb590c6fb4b480972f [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-2-tc2/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-2-tc2/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/cargo-home:/opt/rustwide/cargo-home:ro,Z" "-v" "/var/lib/crater-agent-workspace/rustup-home:/opt/rustwide/rustup-home:ro,Z" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-e" "CARGO_INCREMENTAL=0" "-e" "RUST_BACKTRACE=full" "-e" "RUSTFLAGS=--cap-lints=forbid" "-e" "RUSTDOCFLAGS=--cap-lints=forbid" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-w" "/opt/rustwide/workdir" "-m" "1610612736" "--user" "0:0" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:7ad1b28ee6f5f7f699f6cf7015098d6ccdd96d6f2d78dd06228f5b4c9faf309c" "/opt/rustwide/cargo-home/bin/cargo" "+b83b707f97d809763b7861afa7638871f3339a33" "build" "--frozen" "--message-format=json", kill_on_drop: false }` [INFO] [stdout] c0f5d6389ac1c2f8d20a2c84572e43723f26df4221f3ea46ee4e77237d5fc996 [INFO] running `Command { std: "docker" "start" "-a" "c0f5d6389ac1c2f8d20a2c84572e43723f26df4221f3ea46ee4e77237d5fc996", kill_on_drop: false }` [INFO] [stderr] Compiling libc v0.2.107 [INFO] [stderr] Compiling proc-macro2 v1.0.32 [INFO] [stderr] Compiling syn v1.0.81 [INFO] [stderr] Compiling log v0.4.14 [INFO] [stderr] Compiling parking_lot_core v0.8.5 [INFO] [stderr] Compiling serde_derive v1.0.130 [INFO] [stderr] Compiling signal-hook v0.3.10 [INFO] [stderr] Compiling smallvec v1.7.0 [INFO] [stderr] Compiling serde v1.0.130 [INFO] [stderr] Compiling anyhow v1.0.45 [INFO] [stderr] Compiling instant v0.1.12 [INFO] [stderr] Compiling lock_api v0.4.5 [INFO] [stderr] Compiling unicode-segmentation v1.8.0 [INFO] [stderr] Compiling regex-syntax v0.6.25 [INFO] [stderr] Compiling xml-rs v0.8.4 [INFO] [stderr] Compiling quick-xml v0.20.0 [INFO] [stderr] Compiling aho-corasick v0.7.18 [INFO] [stderr] Compiling home v0.5.3 [INFO] [stderr] Compiling heck v0.3.3 [INFO] [stderr] Compiling minidom v0.13.0 [INFO] [stderr] Compiling quote v1.0.10 [INFO] [stderr] Compiling signal-hook-registry v1.4.0 [INFO] [stderr] Compiling mio v0.7.14 [INFO] [stderr] Compiling atty v0.2.14 [INFO] [stderr] Compiling getrandom v0.2.3 [INFO] [stderr] Compiling colored v2.0.0 [INFO] [stderr] Compiling clap v2.33.3 [INFO] [stderr] Compiling yaserde v0.6.0 [INFO] [stderr] Compiling uuid v0.8.2 [INFO] [stderr] Compiling parking_lot v0.11.2 [INFO] [stderr] Compiling signal-hook-mio v0.2.1 [INFO] [stderr] Compiling crossterm v0.20.0 [INFO] [stderr] Compiling regex v1.5.4 [INFO] [stderr] Compiling yaserde_derive v0.5.1 [INFO] [stderr] Compiling toml v0.5.8 [INFO] [stderr] Compiling kumitateru v0.4.0 (/opt/rustwide/workdir) [INFO] [stdout] warning: function `construct_selection` is never used [INFO] [stdout] --> src/utils/tui/multiple_selection.rs:8:4 [INFO] [stdout] | [INFO] [stdout] 8 | fn construct_selection(header: &str, items: Vec<&str>, highlighted: i64, selected: bool) -> String { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function `display_cli_multiple_selection` is never used [INFO] [stdout] --> src/utils/tui/multiple_selection.rs:41:8 [INFO] [stdout] | [INFO] [stdout] 41 | pub fn display_cli_multiple_selection(header: &str, items: Vec<&str>) -> Result { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function `reset_selection` is never used [INFO] [stdout] --> src/utils/tui/multiple_selection.rs:105:4 [INFO] [stdout] | [INFO] [stdout] 105 | fn reset_selection(len: i64) { [INFO] [stdout] | ^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: struct `BarrelConfig` is never constructed [INFO] [stdout] --> src/ser_de/config/barrel_config.rs:5:12 [INFO] [stdout] | [INFO] [stdout] 5 | pub struct BarrelConfig { [INFO] [stdout] | ^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: struct `BarrelConfigPackage` is never constructed [INFO] [stdout] --> src/ser_de/config/barrel_config.rs:13:12 [INFO] [stdout] | [INFO] [stdout] 13 | pub struct BarrelConfigPackage { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: struct `BarrelConfigPackageMeta` is never constructed [INFO] [stdout] --> src/ser_de/config/barrel_config.rs:23:12 [INFO] [stdout] | [INFO] [stdout] 23 | pub struct BarrelConfigPackageMeta { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: struct `BarrelConfigBuild` is never constructed [INFO] [stdout] --> src/ser_de/config/barrel_config.rs:35:12 [INFO] [stdout] | [INFO] [stdout] 35 | pub struct BarrelConfigBuild { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/main.rs:161:21 [INFO] [stdout] | [INFO] [stdout] 161 | io::stdin().read_line(&mut proj_name); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] = note: `#[warn(unused_must_use)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 161 | let _ = io::stdin().read_line(&mut proj_name); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/main.rs:260:5 [INFO] [stdout] | [INFO] [stdout] 260 | io::stdin().read_line(&mut version); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 260 | let _ = io::stdin().read_line(&mut version); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/prepare_build.rs:16:5 [INFO] [stdout] | [INFO] [stdout] 16 | fs::create_dir("build"); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 16 | let _ = fs::create_dir("build"); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/prepare_build.rs:17:5 [INFO] [stdout] | [INFO] [stdout] 17 | fs::create_dir("build/tmp"); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 17 | let _ = fs::create_dir("build/tmp"); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/prepare_build.rs:18:5 [INFO] [stdout] | [INFO] [stdout] 18 | fs::create_dir("build/tmp/source"); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 18 | let _ = fs::create_dir("build/tmp/source"); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/prepare_build.rs:19:5 [INFO] [stdout] | [INFO] [stdout] 19 | fs::create_dir("build/tmp/resources"); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 19 | let _ = fs::create_dir("build/tmp/resources"); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: for loop over a `Result`. This is more readably written as an `if let` statement [INFO] [stdout] --> src/prepare_build.rs:32:22 [INFO] [stdout] | [INFO] [stdout] 32 | for entry in fs::read_dir(PathBuf::from(resource)) { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(for_loops_over_fallibles)]` on by default [INFO] [stdout] help: to check pattern in a loop use `while let` [INFO] [stdout] | [INFO] [stdout] 32 - for entry in fs::read_dir(PathBuf::from(resource)) { [INFO] [stdout] 32 + while let Ok(entry) = fs::read_dir(PathBuf::from(resource)) { [INFO] [stdout] | [INFO] [stdout] help: consider unwrapping the `Result` with `?` to iterate over its contents [INFO] [stdout] | [INFO] [stdout] 32 | for entry in fs::read_dir(PathBuf::from(resource))? { [INFO] [stdout] | + [INFO] [stdout] help: consider using `if let` to clear intent [INFO] [stdout] | [INFO] [stdout] 32 - for entry in fs::read_dir(PathBuf::from(resource)) { [INFO] [stdout] 32 + if let Ok(entry) = fs::read_dir(PathBuf::from(resource)) { [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/prepare_build.rs:119:13 [INFO] [stdout] | [INFO] [stdout] 119 | fs::copy(entry.path(), end_dir.clone()).with_context(|| format!("Failed to copy {:?} to {:?}", entry.path(), end_dir)); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 119 | let _ = fs::copy(entry.path(), end_dir.clone()).with_context(|| format!("Failed to copy {:?} to {:?}", entry.path(), end_dir)); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: call to `.clone()` on a reference in this situation does nothing [INFO] [stdout] --> src/utils/tui/item_selection.rs:20:78 [INFO] [stdout] | [INFO] [stdout] 20 | result.push_str(&*format!("{}", format!("{}) {}", i + 1, items[i].clone()).bold())); [INFO] [stdout] | ^^^^^^^^ help: remove this redundant call [INFO] [stdout] | [INFO] [stdout] = note: the type `str` does not implement `Clone`, so calling `clone` on `&str` copies the reference, which does not do anything and can be removed [INFO] [stdout] = note: `#[warn(noop_method_call)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: call to `.clone()` on a reference in this situation does nothing [INFO] [stdout] --> src/utils/tui/item_selection.rs:22:64 [INFO] [stdout] | [INFO] [stdout] 22 | result.push_str(&*format!("{}) {}", i + 1, items[i].clone())); [INFO] [stdout] | ^^^^^^^^ help: remove this redundant call [INFO] [stdout] | [INFO] [stdout] = note: the type `str` does not implement `Clone`, so calling `clone` on `&str` copies the reference, which does not do anything and can be removed [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/utils/tui/item_selection.rs:107:9 [INFO] [stdout] | [INFO] [stdout] 107 | io::stdout().execute(terminal::Clear(terminal::ClearType::CurrentLine)); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 107 | let _ = io::stdout().execute(terminal::Clear(terminal::ClearType::CurrentLine)); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/utils/tui/item_selection.rs:108:9 [INFO] [stdout] | [INFO] [stdout] 108 | io::stdout().execute(cursor::MoveUp(1)); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 108 | let _ = io::stdout().execute(cursor::MoveUp(1)); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: call to `.clone()` on a reference in this situation does nothing [INFO] [stdout] --> src/utils/tui/multiple_selection.rs:20:78 [INFO] [stdout] | [INFO] [stdout] 20 | result.push_str(&*format!("{}", format!("{}) {}", i + 1, items[i].clone()).bold())); [INFO] [stdout] | ^^^^^^^^ help: remove this redundant call [INFO] [stdout] | [INFO] [stdout] = note: the type `str` does not implement `Clone`, so calling `clone` on `&str` copies the reference, which does not do anything and can be removed [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: call to `.clone()` on a reference in this situation does nothing [INFO] [stdout] --> src/utils/tui/multiple_selection.rs:22:64 [INFO] [stdout] | [INFO] [stdout] 22 | result.push_str(&*format!("{}) {}", i + 1, items[i].clone())); [INFO] [stdout] | ^^^^^^^^ help: remove this redundant call [INFO] [stdout] | [INFO] [stdout] = note: the type `str` does not implement `Clone`, so calling `clone` on `&str` copies the reference, which does not do anything and can be removed [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/utils/tui/multiple_selection.rs:107:9 [INFO] [stdout] | [INFO] [stdout] 107 | io::stdout().execute(terminal::Clear(terminal::ClearType::CurrentLine)); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 107 | let _ = io::stdout().execute(terminal::Clear(terminal::ClearType::CurrentLine)); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/utils/tui/multiple_selection.rs:108:9 [INFO] [stdout] | [INFO] [stdout] 108 | io::stdout().execute(cursor::MoveUp(1)); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 108 | let _ = io::stdout().execute(cursor::MoveUp(1)); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: for loop over a `Result`. This is more readably written as an `if let` statement [INFO] [stdout] --> src/verify_project.rs:90:18 [INFO] [stdout] | [INFO] [stdout] 90 | for entry in fs::read_dir("resources") { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] help: to check pattern in a loop use `while let` [INFO] [stdout] | [INFO] [stdout] 90 - for entry in fs::read_dir("resources") { [INFO] [stdout] 90 + while let Ok(entry) = fs::read_dir("resources") { [INFO] [stdout] | [INFO] [stdout] help: consider unwrapping the `Result` with `?` to iterate over its contents [INFO] [stdout] | [INFO] [stdout] 90 | for entry in fs::read_dir("resources")? { [INFO] [stdout] | + [INFO] [stdout] help: consider using `if let` to clear intent [INFO] [stdout] | [INFO] [stdout] 90 - for entry in fs::read_dir("resources") { [INFO] [stdout] 90 + if let Ok(entry) = fs::read_dir("resources") { [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: for loop over a `Result`. This is more readably written as an `if let` statement [INFO] [stdout] --> src/verify_project.rs:100:26 [INFO] [stdout] | [INFO] [stdout] 100 | for entry in fs::read_dir(entry.path()) { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] help: to check pattern in a loop use `while let` [INFO] [stdout] | [INFO] [stdout] 100 - for entry in fs::read_dir(entry.path()) { [INFO] [stdout] 100 + while let Ok(entry) = fs::read_dir(entry.path()) { [INFO] [stdout] | [INFO] [stdout] help: consider unwrapping the `Result` with `?` to iterate over its contents [INFO] [stdout] | [INFO] [stdout] 100 | for entry in fs::read_dir(entry.path())? { [INFO] [stdout] | + [INFO] [stdout] help: consider using `if let` to clear intent [INFO] [stdout] | [INFO] [stdout] 100 - for entry in fs::read_dir(entry.path()) { [INFO] [stdout] 100 + if let Ok(entry) = fs::read_dir(entry.path()) { [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing [INFO] [stdout] --> src/verify_project.rs:117:19 [INFO] [stdout] | [INFO] [stdout] 117 | fn basename(path: &str, sep: char) -> Cow { [INFO] [stdout] | ^^^^ ^^^^^^^^ the same lifetime is hidden here [INFO] [stdout] | | [INFO] [stdout] | the lifetime is elided here [INFO] [stdout] | [INFO] [stdout] = help: the same lifetime is referred to in inconsistent ways, making the signature confusing [INFO] [stdout] = note: `#[warn(mismatched_lifetime_syntaxes)]` on by default [INFO] [stdout] help: use `'_` for type paths [INFO] [stdout] | [INFO] [stdout] 117 | fn basename(path: &str, sep: char) -> Cow<'_, str> { [INFO] [stdout] | +++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stdout] --> src/ser_de/manifest/manifest_utils.rs:76:37 [INFO] [stdout] | [INFO] [stdout] 76 | #[derive(Default, PartialEq, Debug, YaSerialize)] [INFO] [stdout] | ^---------- [INFO] [stdout] | | [INFO] [stdout] | `YaSerialize` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_IMPL_YA_SERIALIZE_FOR_CIQManifest` [INFO] [stdout] 77 | #[yaserde(rename = "manifest")] [INFO] [stdout] 78 | struct CIQManifest { [INFO] [stdout] | ----------- `CIQManifest` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `YaSerialize` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `YaSerialize` may come from an old version of the `yaserde_derive` crate, try updating your dependency with `cargo update -p yaserde_derive` [INFO] [stdout] = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl` [INFO] [stdout] = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint [INFO] [stdout] = note: `#[warn(non_local_definitions)]` on by default [INFO] [stdout] = note: this warning originates in the derive macro `YaSerialize` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stdout] --> src/ser_de/manifest/manifest_utils.rs:87:37 [INFO] [stdout] | [INFO] [stdout] 87 | #[derive(Default, PartialEq, Debug, YaSerialize)] [INFO] [stdout] | ^---------- [INFO] [stdout] | | [INFO] [stdout] | `YaSerialize` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_IMPL_YA_SERIALIZE_FOR_CIQApplication` [INFO] [stdout] 88 | struct CIQApplication { [INFO] [stdout] | -------------- `CIQApplication` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `YaSerialize` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `YaSerialize` may come from an old version of the `yaserde_derive` crate, try updating your dependency with `cargo update -p yaserde_derive` [INFO] [stdout] = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl` [INFO] [stdout] = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint [INFO] [stdout] = note: this warning originates in the derive macro `YaSerialize` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stdout] --> src/ser_de/manifest/manifest_utils.rs:113:37 [INFO] [stdout] | [INFO] [stdout] 113 | #[derive(Default, PartialEq, Debug, YaSerialize)] [INFO] [stdout] | ^---------- [INFO] [stdout] | | [INFO] [stdout] | `YaSerialize` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_IMPL_YA_SERIALIZE_FOR_CIQProducts` [INFO] [stdout] 114 | struct CIQProducts { [INFO] [stdout] | ----------- `CIQProducts` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `YaSerialize` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `YaSerialize` may come from an old version of the `yaserde_derive` crate, try updating your dependency with `cargo update -p yaserde_derive` [INFO] [stdout] = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl` [INFO] [stdout] = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint [INFO] [stdout] = note: this warning originates in the derive macro `YaSerialize` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stdout] --> src/ser_de/manifest/manifest_utils.rs:119:37 [INFO] [stdout] | [INFO] [stdout] 119 | #[derive(Default, PartialEq, Debug, YaSerialize)] [INFO] [stdout] | ^---------- [INFO] [stdout] | | [INFO] [stdout] | `YaSerialize` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_IMPL_YA_SERIALIZE_FOR_CIQProduct` [INFO] [stdout] 120 | struct CIQProduct { [INFO] [stdout] | ---------- `CIQProduct` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `YaSerialize` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `YaSerialize` may come from an old version of the `yaserde_derive` crate, try updating your dependency with `cargo update -p yaserde_derive` [INFO] [stdout] = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl` [INFO] [stdout] = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint [INFO] [stdout] = note: this warning originates in the derive macro `YaSerialize` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stdout] --> src/ser_de/manifest/manifest_utils.rs:125:37 [INFO] [stdout] | [INFO] [stdout] 125 | #[derive(Default, PartialEq, Debug, YaSerialize)] [INFO] [stdout] | ^---------- [INFO] [stdout] | | [INFO] [stdout] | `YaSerialize` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_IMPL_YA_SERIALIZE_FOR_CIQPermissions` [INFO] [stdout] 126 | struct CIQPermissions { [INFO] [stdout] | -------------- `CIQPermissions` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `YaSerialize` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `YaSerialize` may come from an old version of the `yaserde_derive` crate, try updating your dependency with `cargo update -p yaserde_derive` [INFO] [stdout] = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl` [INFO] [stdout] = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint [INFO] [stdout] = note: this warning originates in the derive macro `YaSerialize` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stdout] --> src/ser_de/manifest/manifest_utils.rs:132:37 [INFO] [stdout] | [INFO] [stdout] 132 | #[derive(Default, PartialEq, Debug, YaSerialize)] [INFO] [stdout] | ^---------- [INFO] [stdout] | | [INFO] [stdout] | `YaSerialize` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_IMPL_YA_SERIALIZE_FOR_CIQPermission` [INFO] [stdout] 133 | struct CIQPermission { [INFO] [stdout] | ------------- `CIQPermission` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `YaSerialize` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `YaSerialize` may come from an old version of the `yaserde_derive` crate, try updating your dependency with `cargo update -p yaserde_derive` [INFO] [stdout] = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl` [INFO] [stdout] = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint [INFO] [stdout] = note: this warning originates in the derive macro `YaSerialize` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stdout] --> src/ser_de/manifest/manifest_utils.rs:138:37 [INFO] [stdout] | [INFO] [stdout] 138 | #[derive(Default, PartialEq, Debug, YaSerialize)] [INFO] [stdout] | ^---------- [INFO] [stdout] | | [INFO] [stdout] | `YaSerialize` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_IMPL_YA_SERIALIZE_FOR_CIQLanguages` [INFO] [stdout] 139 | struct CIQLanguages { [INFO] [stdout] | ------------ `CIQLanguages` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `YaSerialize` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `YaSerialize` may come from an old version of the `yaserde_derive` crate, try updating your dependency with `cargo update -p yaserde_derive` [INFO] [stdout] = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl` [INFO] [stdout] = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint [INFO] [stdout] = note: this warning originates in the derive macro `YaSerialize` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stdout] --> src/ser_de/manifest/manifest_utils.rs:144:37 [INFO] [stdout] | [INFO] [stdout] 144 | #[derive(Default, PartialEq, Debug, YaSerialize)] [INFO] [stdout] | ^---------- [INFO] [stdout] | | [INFO] [stdout] | `YaSerialize` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_IMPL_YA_SERIALIZE_FOR_CIQDependencies` [INFO] [stdout] 145 | struct CIQDependencies { [INFO] [stdout] | --------------- `CIQDependencies` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `YaSerialize` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `YaSerialize` may come from an old version of the `yaserde_derive` crate, try updating your dependency with `cargo update -p yaserde_derive` [INFO] [stdout] = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl` [INFO] [stdout] = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint [INFO] [stdout] = note: this warning originates in the derive macro `YaSerialize` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stdout] --> src/ser_de/manifest/manifest_utils.rs:149:37 [INFO] [stdout] | [INFO] [stdout] 149 | #[derive(Default, PartialEq, Debug, YaSerialize)] [INFO] [stdout] | ^---------- [INFO] [stdout] | | [INFO] [stdout] | `YaSerialize` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_IMPL_YA_SERIALIZE_FOR_CIQDependency` [INFO] [stdout] 150 | struct CIQDependency { [INFO] [stdout] | ------------- `CIQDependency` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `YaSerialize` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `YaSerialize` may come from an old version of the `yaserde_derive` crate, try updating your dependency with `cargo update -p yaserde_derive` [INFO] [stdout] = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl` [INFO] [stdout] = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint [INFO] [stdout] = note: this warning originates in the derive macro `YaSerialize` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stderr] Finished `dev` profile [unoptimized + debuginfo] target(s) in 26.48s [INFO] [stderr] warning: the following packages contain code that will be rejected by a future version of Rust: quick-xml v0.20.0 [INFO] [stderr] note: to see what the problems were, use the option `--future-incompat-report`, or run `cargo report future-incompatibilities --id 3` [INFO] running `Command { std: "docker" "inspect" "c0f5d6389ac1c2f8d20a2c84572e43723f26df4221f3ea46ee4e77237d5fc996", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "c0f5d6389ac1c2f8d20a2c84572e43723f26df4221f3ea46ee4e77237d5fc996", kill_on_drop: false }` [INFO] [stdout] c0f5d6389ac1c2f8d20a2c84572e43723f26df4221f3ea46ee4e77237d5fc996 [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-2-tc2/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-2-tc2/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/cargo-home:/opt/rustwide/cargo-home:ro,Z" "-v" "/var/lib/crater-agent-workspace/rustup-home:/opt/rustwide/rustup-home:ro,Z" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-e" "CARGO_INCREMENTAL=0" "-e" "RUST_BACKTRACE=full" "-e" "RUSTFLAGS=--cap-lints=forbid" "-e" "RUSTDOCFLAGS=--cap-lints=forbid" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-w" "/opt/rustwide/workdir" "-m" "1610612736" "--user" "0:0" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:7ad1b28ee6f5f7f699f6cf7015098d6ccdd96d6f2d78dd06228f5b4c9faf309c" "/opt/rustwide/cargo-home/bin/cargo" "+b83b707f97d809763b7861afa7638871f3339a33" "test" "--frozen" "--no-run" "--message-format=json", kill_on_drop: false }` [INFO] [stdout] 2d803a2bbd2234b6f62853a1f0391e0187d1c1ed1864b23c828c8975760747fb [INFO] running `Command { std: "docker" "start" "-a" "2d803a2bbd2234b6f62853a1f0391e0187d1c1ed1864b23c828c8975760747fb", kill_on_drop: false }` [INFO] [stderr] Compiling kumitateru v0.4.0 (/opt/rustwide/workdir) [INFO] [stdout] warning: function `construct_selection` is never used [INFO] [stdout] --> src/utils/tui/multiple_selection.rs:8:4 [INFO] [stdout] | [INFO] [stdout] 8 | fn construct_selection(header: &str, items: Vec<&str>, highlighted: i64, selected: bool) -> String { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function `display_cli_multiple_selection` is never used [INFO] [stdout] --> src/utils/tui/multiple_selection.rs:41:8 [INFO] [stdout] | [INFO] [stdout] 41 | pub fn display_cli_multiple_selection(header: &str, items: Vec<&str>) -> Result { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function `reset_selection` is never used [INFO] [stdout] --> src/utils/tui/multiple_selection.rs:105:4 [INFO] [stdout] | [INFO] [stdout] 105 | fn reset_selection(len: i64) { [INFO] [stdout] | ^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: struct `BarrelConfig` is never constructed [INFO] [stdout] --> src/ser_de/config/barrel_config.rs:5:12 [INFO] [stdout] | [INFO] [stdout] 5 | pub struct BarrelConfig { [INFO] [stdout] | ^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: struct `BarrelConfigPackage` is never constructed [INFO] [stdout] --> src/ser_de/config/barrel_config.rs:13:12 [INFO] [stdout] | [INFO] [stdout] 13 | pub struct BarrelConfigPackage { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: struct `BarrelConfigPackageMeta` is never constructed [INFO] [stdout] --> src/ser_de/config/barrel_config.rs:23:12 [INFO] [stdout] | [INFO] [stdout] 23 | pub struct BarrelConfigPackageMeta { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: struct `BarrelConfigBuild` is never constructed [INFO] [stdout] --> src/ser_de/config/barrel_config.rs:35:12 [INFO] [stdout] | [INFO] [stdout] 35 | pub struct BarrelConfigBuild { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/main.rs:161:21 [INFO] [stdout] | [INFO] [stdout] 161 | io::stdin().read_line(&mut proj_name); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] = note: `#[warn(unused_must_use)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 161 | let _ = io::stdin().read_line(&mut proj_name); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/main.rs:260:5 [INFO] [stdout] | [INFO] [stdout] 260 | io::stdin().read_line(&mut version); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 260 | let _ = io::stdin().read_line(&mut version); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/prepare_build.rs:16:5 [INFO] [stdout] | [INFO] [stdout] 16 | fs::create_dir("build"); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 16 | let _ = fs::create_dir("build"); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/prepare_build.rs:17:5 [INFO] [stdout] | [INFO] [stdout] 17 | fs::create_dir("build/tmp"); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 17 | let _ = fs::create_dir("build/tmp"); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/prepare_build.rs:18:5 [INFO] [stdout] | [INFO] [stdout] 18 | fs::create_dir("build/tmp/source"); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 18 | let _ = fs::create_dir("build/tmp/source"); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/prepare_build.rs:19:5 [INFO] [stdout] | [INFO] [stdout] 19 | fs::create_dir("build/tmp/resources"); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 19 | let _ = fs::create_dir("build/tmp/resources"); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: for loop over a `Result`. This is more readably written as an `if let` statement [INFO] [stdout] --> src/prepare_build.rs:32:22 [INFO] [stdout] | [INFO] [stdout] 32 | for entry in fs::read_dir(PathBuf::from(resource)) { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(for_loops_over_fallibles)]` on by default [INFO] [stdout] help: to check pattern in a loop use `while let` [INFO] [stdout] | [INFO] [stdout] 32 - for entry in fs::read_dir(PathBuf::from(resource)) { [INFO] [stdout] 32 + while let Ok(entry) = fs::read_dir(PathBuf::from(resource)) { [INFO] [stdout] | [INFO] [stdout] help: consider unwrapping the `Result` with `?` to iterate over its contents [INFO] [stdout] | [INFO] [stdout] 32 | for entry in fs::read_dir(PathBuf::from(resource))? { [INFO] [stdout] | + [INFO] [stdout] help: consider using `if let` to clear intent [INFO] [stdout] | [INFO] [stdout] 32 - for entry in fs::read_dir(PathBuf::from(resource)) { [INFO] [stdout] 32 + if let Ok(entry) = fs::read_dir(PathBuf::from(resource)) { [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/prepare_build.rs:119:13 [INFO] [stdout] | [INFO] [stdout] 119 | fs::copy(entry.path(), end_dir.clone()).with_context(|| format!("Failed to copy {:?} to {:?}", entry.path(), end_dir)); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 119 | let _ = fs::copy(entry.path(), end_dir.clone()).with_context(|| format!("Failed to copy {:?} to {:?}", entry.path(), end_dir)); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: call to `.clone()` on a reference in this situation does nothing [INFO] [stdout] --> src/utils/tui/item_selection.rs:20:78 [INFO] [stdout] | [INFO] [stdout] 20 | result.push_str(&*format!("{}", format!("{}) {}", i + 1, items[i].clone()).bold())); [INFO] [stdout] | ^^^^^^^^ help: remove this redundant call [INFO] [stdout] | [INFO] [stdout] = note: the type `str` does not implement `Clone`, so calling `clone` on `&str` copies the reference, which does not do anything and can be removed [INFO] [stdout] = note: `#[warn(noop_method_call)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: call to `.clone()` on a reference in this situation does nothing [INFO] [stdout] --> src/utils/tui/item_selection.rs:22:64 [INFO] [stdout] | [INFO] [stdout] 22 | result.push_str(&*format!("{}) {}", i + 1, items[i].clone())); [INFO] [stdout] | ^^^^^^^^ help: remove this redundant call [INFO] [stdout] | [INFO] [stdout] = note: the type `str` does not implement `Clone`, so calling `clone` on `&str` copies the reference, which does not do anything and can be removed [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/utils/tui/item_selection.rs:107:9 [INFO] [stdout] | [INFO] [stdout] 107 | io::stdout().execute(terminal::Clear(terminal::ClearType::CurrentLine)); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 107 | let _ = io::stdout().execute(terminal::Clear(terminal::ClearType::CurrentLine)); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/utils/tui/item_selection.rs:108:9 [INFO] [stdout] | [INFO] [stdout] 108 | io::stdout().execute(cursor::MoveUp(1)); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 108 | let _ = io::stdout().execute(cursor::MoveUp(1)); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: call to `.clone()` on a reference in this situation does nothing [INFO] [stdout] --> src/utils/tui/multiple_selection.rs:20:78 [INFO] [stdout] | [INFO] [stdout] 20 | result.push_str(&*format!("{}", format!("{}) {}", i + 1, items[i].clone()).bold())); [INFO] [stdout] | ^^^^^^^^ help: remove this redundant call [INFO] [stdout] | [INFO] [stdout] = note: the type `str` does not implement `Clone`, so calling `clone` on `&str` copies the reference, which does not do anything and can be removed [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: call to `.clone()` on a reference in this situation does nothing [INFO] [stdout] --> src/utils/tui/multiple_selection.rs:22:64 [INFO] [stdout] | [INFO] [stdout] 22 | result.push_str(&*format!("{}) {}", i + 1, items[i].clone())); [INFO] [stdout] | ^^^^^^^^ help: remove this redundant call [INFO] [stdout] | [INFO] [stdout] = note: the type `str` does not implement `Clone`, so calling `clone` on `&str` copies the reference, which does not do anything and can be removed [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/utils/tui/multiple_selection.rs:107:9 [INFO] [stdout] | [INFO] [stdout] 107 | io::stdout().execute(terminal::Clear(terminal::ClearType::CurrentLine)); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 107 | let _ = io::stdout().execute(terminal::Clear(terminal::ClearType::CurrentLine)); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/utils/tui/multiple_selection.rs:108:9 [INFO] [stdout] | [INFO] [stdout] 108 | io::stdout().execute(cursor::MoveUp(1)); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 108 | let _ = io::stdout().execute(cursor::MoveUp(1)); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: for loop over a `Result`. This is more readably written as an `if let` statement [INFO] [stdout] --> src/verify_project.rs:90:18 [INFO] [stdout] | [INFO] [stdout] 90 | for entry in fs::read_dir("resources") { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] help: to check pattern in a loop use `while let` [INFO] [stdout] | [INFO] [stdout] 90 - for entry in fs::read_dir("resources") { [INFO] [stdout] 90 + while let Ok(entry) = fs::read_dir("resources") { [INFO] [stdout] | [INFO] [stdout] help: consider unwrapping the `Result` with `?` to iterate over its contents [INFO] [stdout] | [INFO] [stdout] 90 | for entry in fs::read_dir("resources")? { [INFO] [stdout] | + [INFO] [stdout] help: consider using `if let` to clear intent [INFO] [stdout] | [INFO] [stdout] 90 - for entry in fs::read_dir("resources") { [INFO] [stdout] 90 + if let Ok(entry) = fs::read_dir("resources") { [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: for loop over a `Result`. This is more readably written as an `if let` statement [INFO] [stdout] --> src/verify_project.rs:100:26 [INFO] [stdout] | [INFO] [stdout] 100 | for entry in fs::read_dir(entry.path()) { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] help: to check pattern in a loop use `while let` [INFO] [stdout] | [INFO] [stdout] 100 - for entry in fs::read_dir(entry.path()) { [INFO] [stdout] 100 + while let Ok(entry) = fs::read_dir(entry.path()) { [INFO] [stdout] | [INFO] [stdout] help: consider unwrapping the `Result` with `?` to iterate over its contents [INFO] [stdout] | [INFO] [stdout] 100 | for entry in fs::read_dir(entry.path())? { [INFO] [stdout] | + [INFO] [stdout] help: consider using `if let` to clear intent [INFO] [stdout] | [INFO] [stdout] 100 - for entry in fs::read_dir(entry.path()) { [INFO] [stdout] 100 + if let Ok(entry) = fs::read_dir(entry.path()) { [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing [INFO] [stdout] --> src/verify_project.rs:117:19 [INFO] [stdout] | [INFO] [stdout] 117 | fn basename(path: &str, sep: char) -> Cow { [INFO] [stdout] | ^^^^ ^^^^^^^^ the same lifetime is hidden here [INFO] [stdout] | | [INFO] [stdout] | the lifetime is elided here [INFO] [stdout] | [INFO] [stdout] = help: the same lifetime is referred to in inconsistent ways, making the signature confusing [INFO] [stdout] = note: `#[warn(mismatched_lifetime_syntaxes)]` on by default [INFO] [stdout] help: use `'_` for type paths [INFO] [stdout] | [INFO] [stdout] 117 | fn basename(path: &str, sep: char) -> Cow<'_, str> { [INFO] [stdout] | +++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stdout] --> src/ser_de/manifest/manifest_utils.rs:76:37 [INFO] [stdout] | [INFO] [stdout] 76 | #[derive(Default, PartialEq, Debug, YaSerialize)] [INFO] [stdout] | ^---------- [INFO] [stdout] | | [INFO] [stdout] | `YaSerialize` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_IMPL_YA_SERIALIZE_FOR_CIQManifest` [INFO] [stdout] 77 | #[yaserde(rename = "manifest")] [INFO] [stdout] 78 | struct CIQManifest { [INFO] [stdout] | ----------- `CIQManifest` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `YaSerialize` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `YaSerialize` may come from an old version of the `yaserde_derive` crate, try updating your dependency with `cargo update -p yaserde_derive` [INFO] [stdout] = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl` [INFO] [stdout] = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint [INFO] [stdout] = note: `#[warn(non_local_definitions)]` on by default [INFO] [stdout] = note: this warning originates in the derive macro `YaSerialize` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stdout] --> src/ser_de/manifest/manifest_utils.rs:87:37 [INFO] [stdout] | [INFO] [stdout] 87 | #[derive(Default, PartialEq, Debug, YaSerialize)] [INFO] [stdout] | ^---------- [INFO] [stdout] | | [INFO] [stdout] | `YaSerialize` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_IMPL_YA_SERIALIZE_FOR_CIQApplication` [INFO] [stdout] 88 | struct CIQApplication { [INFO] [stdout] | -------------- `CIQApplication` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `YaSerialize` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `YaSerialize` may come from an old version of the `yaserde_derive` crate, try updating your dependency with `cargo update -p yaserde_derive` [INFO] [stdout] = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl` [INFO] [stdout] = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint [INFO] [stdout] = note: this warning originates in the derive macro `YaSerialize` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stdout] --> src/ser_de/manifest/manifest_utils.rs:113:37 [INFO] [stdout] | [INFO] [stdout] 113 | #[derive(Default, PartialEq, Debug, YaSerialize)] [INFO] [stdout] | ^---------- [INFO] [stdout] | | [INFO] [stdout] | `YaSerialize` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_IMPL_YA_SERIALIZE_FOR_CIQProducts` [INFO] [stdout] 114 | struct CIQProducts { [INFO] [stdout] | ----------- `CIQProducts` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `YaSerialize` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `YaSerialize` may come from an old version of the `yaserde_derive` crate, try updating your dependency with `cargo update -p yaserde_derive` [INFO] [stdout] = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl` [INFO] [stdout] = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint [INFO] [stdout] = note: this warning originates in the derive macro `YaSerialize` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stdout] --> src/ser_de/manifest/manifest_utils.rs:119:37 [INFO] [stdout] | [INFO] [stdout] 119 | #[derive(Default, PartialEq, Debug, YaSerialize)] [INFO] [stdout] | ^---------- [INFO] [stdout] | | [INFO] [stdout] | `YaSerialize` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_IMPL_YA_SERIALIZE_FOR_CIQProduct` [INFO] [stdout] 120 | struct CIQProduct { [INFO] [stdout] | ---------- `CIQProduct` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `YaSerialize` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `YaSerialize` may come from an old version of the `yaserde_derive` crate, try updating your dependency with `cargo update -p yaserde_derive` [INFO] [stdout] = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl` [INFO] [stdout] = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint [INFO] [stdout] = note: this warning originates in the derive macro `YaSerialize` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stdout] --> src/ser_de/manifest/manifest_utils.rs:125:37 [INFO] [stdout] | [INFO] [stdout] 125 | #[derive(Default, PartialEq, Debug, YaSerialize)] [INFO] [stdout] | ^---------- [INFO] [stdout] | | [INFO] [stdout] | `YaSerialize` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_IMPL_YA_SERIALIZE_FOR_CIQPermissions` [INFO] [stdout] 126 | struct CIQPermissions { [INFO] [stdout] | -------------- `CIQPermissions` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `YaSerialize` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `YaSerialize` may come from an old version of the `yaserde_derive` crate, try updating your dependency with `cargo update -p yaserde_derive` [INFO] [stdout] = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl` [INFO] [stdout] = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint [INFO] [stdout] = note: this warning originates in the derive macro `YaSerialize` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stdout] --> src/ser_de/manifest/manifest_utils.rs:132:37 [INFO] [stdout] | [INFO] [stdout] 132 | #[derive(Default, PartialEq, Debug, YaSerialize)] [INFO] [stdout] | ^---------- [INFO] [stdout] | | [INFO] [stdout] | `YaSerialize` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_IMPL_YA_SERIALIZE_FOR_CIQPermission` [INFO] [stdout] 133 | struct CIQPermission { [INFO] [stdout] | ------------- `CIQPermission` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `YaSerialize` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `YaSerialize` may come from an old version of the `yaserde_derive` crate, try updating your dependency with `cargo update -p yaserde_derive` [INFO] [stdout] = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl` [INFO] [stdout] = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint [INFO] [stdout] = note: this warning originates in the derive macro `YaSerialize` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stdout] --> src/ser_de/manifest/manifest_utils.rs:138:37 [INFO] [stdout] | [INFO] [stdout] 138 | #[derive(Default, PartialEq, Debug, YaSerialize)] [INFO] [stdout] | ^---------- [INFO] [stdout] | | [INFO] [stdout] | `YaSerialize` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_IMPL_YA_SERIALIZE_FOR_CIQLanguages` [INFO] [stdout] 139 | struct CIQLanguages { [INFO] [stdout] | ------------ `CIQLanguages` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `YaSerialize` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `YaSerialize` may come from an old version of the `yaserde_derive` crate, try updating your dependency with `cargo update -p yaserde_derive` [INFO] [stdout] = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl` [INFO] [stdout] = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint [INFO] [stdout] = note: this warning originates in the derive macro `YaSerialize` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stdout] --> src/ser_de/manifest/manifest_utils.rs:144:37 [INFO] [stdout] | [INFO] [stdout] 144 | #[derive(Default, PartialEq, Debug, YaSerialize)] [INFO] [stdout] | ^---------- [INFO] [stdout] | | [INFO] [stdout] | `YaSerialize` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_IMPL_YA_SERIALIZE_FOR_CIQDependencies` [INFO] [stdout] 145 | struct CIQDependencies { [INFO] [stdout] | --------------- `CIQDependencies` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `YaSerialize` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `YaSerialize` may come from an old version of the `yaserde_derive` crate, try updating your dependency with `cargo update -p yaserde_derive` [INFO] [stdout] = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl` [INFO] [stdout] = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint [INFO] [stdout] = note: this warning originates in the derive macro `YaSerialize` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stdout] --> src/ser_de/manifest/manifest_utils.rs:149:37 [INFO] [stdout] | [INFO] [stdout] 149 | #[derive(Default, PartialEq, Debug, YaSerialize)] [INFO] [stdout] | ^---------- [INFO] [stdout] | | [INFO] [stdout] | `YaSerialize` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_IMPL_YA_SERIALIZE_FOR_CIQDependency` [INFO] [stdout] 150 | struct CIQDependency { [INFO] [stdout] | ------------- `CIQDependency` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `YaSerialize` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `YaSerialize` may come from an old version of the `yaserde_derive` crate, try updating your dependency with `cargo update -p yaserde_derive` [INFO] [stdout] = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl` [INFO] [stdout] = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint [INFO] [stdout] = note: this warning originates in the derive macro `YaSerialize` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stderr] Finished `test` profile [unoptimized + debuginfo] target(s) in 1.07s [INFO] [stderr] warning: the following packages contain code that will be rejected by a future version of Rust: quick-xml v0.20.0 [INFO] [stderr] note: to see what the problems were, use the option `--future-incompat-report`, or run `cargo report future-incompatibilities --id 3` [INFO] running `Command { std: "docker" "inspect" "2d803a2bbd2234b6f62853a1f0391e0187d1c1ed1864b23c828c8975760747fb", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "2d803a2bbd2234b6f62853a1f0391e0187d1c1ed1864b23c828c8975760747fb", kill_on_drop: false }` [INFO] [stdout] 2d803a2bbd2234b6f62853a1f0391e0187d1c1ed1864b23c828c8975760747fb [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-2-tc2/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-2-tc2/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/cargo-home:/opt/rustwide/cargo-home:ro,Z" "-v" "/var/lib/crater-agent-workspace/rustup-home:/opt/rustwide/rustup-home:ro,Z" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-e" "CARGO_INCREMENTAL=0" "-e" "RUST_BACKTRACE=full" "-e" "RUSTFLAGS=--cap-lints=forbid" "-e" "RUSTDOCFLAGS=--cap-lints=forbid" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-w" "/opt/rustwide/workdir" "-m" "1610612736" "--user" "0:0" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:7ad1b28ee6f5f7f699f6cf7015098d6ccdd96d6f2d78dd06228f5b4c9faf309c" "/opt/rustwide/cargo-home/bin/cargo" "+b83b707f97d809763b7861afa7638871f3339a33" "test" "--frozen", kill_on_drop: false }` [INFO] [stdout] cdb0cbb91c08d2d17c381a82aa616e2562d0d21be246a5661085119f18ba473f [INFO] running `Command { std: "docker" "start" "-a" "cdb0cbb91c08d2d17c381a82aa616e2562d0d21be246a5661085119f18ba473f", kill_on_drop: false }` [INFO] [stderr] warning: function `construct_selection` is never used [INFO] [stderr] --> src/utils/tui/multiple_selection.rs:8:4 [INFO] [stderr] | [INFO] [stderr] 8 | fn construct_selection(header: &str, items: Vec<&str>, highlighted: i64, selected: bool) -> String { [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default [INFO] [stderr] [INFO] [stderr] warning: function `display_cli_multiple_selection` is never used [INFO] [stderr] --> src/utils/tui/multiple_selection.rs:41:8 [INFO] [stderr] | [INFO] [stderr] 41 | pub fn display_cli_multiple_selection(header: &str, items: Vec<&str>) -> Result { [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] [INFO] [stderr] warning: function `reset_selection` is never used [INFO] [stderr] --> src/utils/tui/multiple_selection.rs:105:4 [INFO] [stderr] | [INFO] [stderr] 105 | fn reset_selection(len: i64) { [INFO] [stderr] | ^^^^^^^^^^^^^^^ [INFO] [stderr] [INFO] [stderr] warning: struct `BarrelConfig` is never constructed [INFO] [stderr] --> src/ser_de/config/barrel_config.rs:5:12 [INFO] [stderr] | [INFO] [stderr] 5 | pub struct BarrelConfig { [INFO] [stderr] | ^^^^^^^^^^^^ [INFO] [stderr] [INFO] [stderr] warning: struct `BarrelConfigPackage` is never constructed [INFO] [stderr] --> src/ser_de/config/barrel_config.rs:13:12 [INFO] [stderr] | [INFO] [stderr] 13 | pub struct BarrelConfigPackage { [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] [INFO] [stderr] warning: struct `BarrelConfigPackageMeta` is never constructed [INFO] [stderr] --> src/ser_de/config/barrel_config.rs:23:12 [INFO] [stderr] | [INFO] [stderr] 23 | pub struct BarrelConfigPackageMeta { [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] [INFO] [stderr] warning: struct `BarrelConfigBuild` is never constructed [INFO] [stderr] --> src/ser_de/config/barrel_config.rs:35:12 [INFO] [stderr] | [INFO] [stderr] 35 | pub struct BarrelConfigBuild { [INFO] [stderr] | ^^^^^^^^^^^^^^^^^ [INFO] [stderr] [INFO] [stderr] warning: unused `Result` that must be used [INFO] [stderr] --> src/main.rs:161:21 [INFO] [stderr] | [INFO] [stderr] 161 | io::stdin().read_line(&mut proj_name); [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stderr] = note: `#[warn(unused_must_use)]` (part of `#[warn(unused)]`) on by default [INFO] [stderr] help: use `let _ = ...` to ignore the resulting value [INFO] [stderr] | [INFO] [stderr] 161 | let _ = io::stdin().read_line(&mut proj_name); [INFO] [stderr] | +++++++ [INFO] [stderr] [INFO] [stderr] warning: unused `Result` that must be used [INFO] [stderr] --> src/main.rs:260:5 [INFO] [stderr] | [INFO] [stderr] 260 | io::stdin().read_line(&mut version); [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stderr] help: use `let _ = ...` to ignore the resulting value [INFO] [stderr] | [INFO] [stderr] 260 | let _ = io::stdin().read_line(&mut version); [INFO] [stderr] | +++++++ [INFO] [stderr] [INFO] [stderr] warning: unused `Result` that must be used [INFO] [stderr] --> src/prepare_build.rs:16:5 [INFO] [stderr] | [INFO] [stderr] 16 | fs::create_dir("build"); [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stderr] help: use `let _ = ...` to ignore the resulting value [INFO] [stderr] | [INFO] [stderr] 16 | let _ = fs::create_dir("build"); [INFO] [stderr] | +++++++ [INFO] [stderr] [INFO] [stderr] warning: unused `Result` that must be used [INFO] [stderr] --> src/prepare_build.rs:17:5 [INFO] [stderr] | [INFO] [stderr] 17 | fs::create_dir("build/tmp"); [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stderr] help: use `let _ = ...` to ignore the resulting value [INFO] [stderr] | [INFO] [stderr] 17 | let _ = fs::create_dir("build/tmp"); [INFO] [stderr] | +++++++ [INFO] [stderr] [INFO] [stderr] warning: unused `Result` that must be used [INFO] [stderr] --> src/prepare_build.rs:18:5 [INFO] [stderr] | [INFO] [stderr] 18 | fs::create_dir("build/tmp/source"); [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stderr] help: use `let _ = ...` to ignore the resulting value [INFO] [stderr] | [INFO] [stderr] 18 | let _ = fs::create_dir("build/tmp/source"); [INFO] [stderr] | +++++++ [INFO] [stderr] [INFO] [stderr] warning: unused `Result` that must be used [INFO] [stderr] --> src/prepare_build.rs:19:5 [INFO] [stderr] | [INFO] [stderr] 19 | fs::create_dir("build/tmp/resources"); [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stderr] help: use `let _ = ...` to ignore the resulting value [INFO] [stderr] | [INFO] [stderr] 19 | let _ = fs::create_dir("build/tmp/resources"); [INFO] [stderr] | +++++++ [INFO] [stderr] [INFO] [stderr] warning: for loop over a `Result`. This is more readably written as an `if let` statement [INFO] [stderr] --> src/prepare_build.rs:32:22 [INFO] [stderr] | [INFO] [stderr] 32 | for entry in fs::read_dir(PathBuf::from(resource)) { [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: `#[warn(for_loops_over_fallibles)]` on by default [INFO] [stderr] help: to check pattern in a loop use `while let` [INFO] [stderr] | [INFO] [stderr] 32 - for entry in fs::read_dir(PathBuf::from(resource)) { [INFO] [stderr] 32 + while let Ok(entry) = fs::read_dir(PathBuf::from(resource)) { [INFO] [stderr] | [INFO] [stderr] help: consider unwrapping the `Result` with `?` to iterate over its contents [INFO] [stderr] | [INFO] [stderr] 32 | for entry in fs::read_dir(PathBuf::from(resource))? { [INFO] [stderr] | + [INFO] [stderr] help: consider using `if let` to clear intent [INFO] [stderr] | [INFO] [stderr] 32 - for entry in fs::read_dir(PathBuf::from(resource)) { [INFO] [stderr] 32 + if let Ok(entry) = fs::read_dir(PathBuf::from(resource)) { [INFO] [stderr] | [INFO] [stderr] [INFO] [stderr] warning: unused `Result` that must be used [INFO] [stderr] --> src/prepare_build.rs:119:13 [INFO] [stderr] | [INFO] [stderr] 119 | fs::copy(entry.path(), end_dir.clone()).with_context(|| format!("Failed to copy {:?} to {:?}", entry.path(), end_dir)); [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stderr] help: use `let _ = ...` to ignore the resulting value [INFO] [stderr] | [INFO] [stderr] 119 | let _ = fs::copy(entry.path(), end_dir.clone()).with_context(|| format!("Failed to copy {:?} to {:?}", entry.path(), end_dir)); [INFO] [stderr] | +++++++ [INFO] [stderr] [INFO] [stderr] warning: call to `.clone()` on a reference in this situation does nothing [INFO] [stderr] --> src/utils/tui/item_selection.rs:20:78 [INFO] [stderr] | [INFO] [stderr] 20 | result.push_str(&*format!("{}", format!("{}) {}", i + 1, items[i].clone()).bold())); [INFO] [stderr] | ^^^^^^^^ help: remove this redundant call [INFO] [stderr] | [INFO] [stderr] = note: the type `str` does not implement `Clone`, so calling `clone` on `&str` copies the reference, which does not do anything and can be removed [INFO] [stderr] = note: `#[warn(noop_method_call)]` on by default [INFO] [stderr] [INFO] [stderr] warning: call to `.clone()` on a reference in this situation does nothing [INFO] [stderr] --> src/utils/tui/item_selection.rs:22:64 [INFO] [stderr] | [INFO] [stderr] 22 | result.push_str(&*format!("{}) {}", i + 1, items[i].clone())); [INFO] [stderr] | ^^^^^^^^ help: remove this redundant call [INFO] [stderr] | [INFO] [stderr] = note: the type `str` does not implement `Clone`, so calling `clone` on `&str` copies the reference, which does not do anything and can be removed [INFO] [stderr] [INFO] [stderr] warning: unused `Result` that must be used [INFO] [stderr] --> src/utils/tui/item_selection.rs:107:9 [INFO] [stderr] | [INFO] [stderr] 107 | io::stdout().execute(terminal::Clear(terminal::ClearType::CurrentLine)); [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stderr] help: use `let _ = ...` to ignore the resulting value [INFO] [stderr] | [INFO] [stderr] 107 | let _ = io::stdout().execute(terminal::Clear(terminal::ClearType::CurrentLine)); [INFO] [stderr] | +++++++ [INFO] [stderr] [INFO] [stderr] warning: unused `Result` that must be used [INFO] [stderr] --> src/utils/tui/item_selection.rs:108:9 [INFO] [stderr] | [INFO] [stderr] 108 | io::stdout().execute(cursor::MoveUp(1)); [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stderr] help: use `let _ = ...` to ignore the resulting value [INFO] [stderr] | [INFO] [stderr] 108 | let _ = io::stdout().execute(cursor::MoveUp(1)); [INFO] [stderr] | +++++++ [INFO] [stderr] [INFO] [stderr] warning: call to `.clone()` on a reference in this situation does nothing [INFO] [stderr] --> src/utils/tui/multiple_selection.rs:20:78 [INFO] [stderr] | [INFO] [stderr] 20 | result.push_str(&*format!("{}", format!("{}) {}", i + 1, items[i].clone()).bold())); [INFO] [stderr] | ^^^^^^^^ help: remove this redundant call [INFO] [stderr] | [INFO] [stderr] = note: the type `str` does not implement `Clone`, so calling `clone` on `&str` copies the reference, which does not do anything and can be removed [INFO] [stderr] [INFO] [stderr] warning: call to `.clone()` on a reference in this situation does nothing [INFO] [stderr] --> src/utils/tui/multiple_selection.rs:22:64 [INFO] [stderr] | [INFO] [stderr] 22 | result.push_str(&*format!("{}) {}", i + 1, items[i].clone())); [INFO] [stderr] | ^^^^^^^^ help: remove this redundant call [INFO] [stderr] | [INFO] [stderr] = note: the type `str` does not implement `Clone`, so calling `clone` on `&str` copies the reference, which does not do anything and can be removed [INFO] [stderr] [INFO] [stderr] warning: unused `Result` that must be used [INFO] [stderr] --> src/utils/tui/multiple_selection.rs:107:9 [INFO] [stderr] | [INFO] [stderr] 107 | io::stdout().execute(terminal::Clear(terminal::ClearType::CurrentLine)); [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stderr] help: use `let _ = ...` to ignore the resulting value [INFO] [stderr] | [INFO] [stderr] 107 | let _ = io::stdout().execute(terminal::Clear(terminal::ClearType::CurrentLine)); [INFO] [stderr] | +++++++ [INFO] [stderr] [INFO] [stderr] warning: unused `Result` that must be used [INFO] [stderr] --> src/utils/tui/multiple_selection.rs:108:9 [INFO] [stderr] | [INFO] [stderr] 108 | io::stdout().execute(cursor::MoveUp(1)); [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stderr] help: use `let _ = ...` to ignore the resulting value [INFO] [stderr] | [INFO] [stderr] 108 | let _ = io::stdout().execute(cursor::MoveUp(1)); [INFO] [stderr] | +++++++ [INFO] [stderr] [INFO] [stderr] warning: for loop over a `Result`. This is more readably written as an `if let` statement [INFO] [stderr] --> src/verify_project.rs:90:18 [INFO] [stderr] | [INFO] [stderr] 90 | for entry in fs::read_dir("resources") { [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] help: to check pattern in a loop use `while let` [INFO] [stderr] | [INFO] [stderr] 90 - for entry in fs::read_dir("resources") { [INFO] [stderr] 90 + while let Ok(entry) = fs::read_dir("resources") { [INFO] [stderr] | [INFO] [stderr] help: consider unwrapping the `Result` with `?` to iterate over its contents [INFO] [stderr] | [INFO] [stderr] 90 | for entry in fs::read_dir("resources")? { [INFO] [stderr] | + [INFO] [stderr] help: consider using `if let` to clear intent [INFO] [stderr] | [INFO] [stderr] 90 - for entry in fs::read_dir("resources") { [INFO] [stderr] 90 + if let Ok(entry) = fs::read_dir("resources") { [INFO] [stderr] | [INFO] [stderr] [INFO] [stderr] warning: for loop over a `Result`. This is more readably written as an `if let` statement [INFO] [stderr] --> src/verify_project.rs:100:26 [INFO] [stderr] | [INFO] [stderr] 100 | for entry in fs::read_dir(entry.path()) { [INFO] [stderr] | ^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stderr] | [INFO] [stderr] help: to check pattern in a loop use `while let` [INFO] [stderr] | [INFO] [stderr] 100 - for entry in fs::read_dir(entry.path()) { [INFO] [stderr] 100 + while let Ok(entry) = fs::read_dir(entry.path()) { [INFO] [stderr] | [INFO] [stderr] help: consider unwrapping the `Result` with `?` to iterate over its contents [INFO] [stderr] | [INFO] [stderr] 100 | for entry in fs::read_dir(entry.path())? { [INFO] [stderr] | + [INFO] [stderr] help: consider using `if let` to clear intent [INFO] [stderr] | [INFO] [stderr] 100 - for entry in fs::read_dir(entry.path()) { [INFO] [stderr] 100 + if let Ok(entry) = fs::read_dir(entry.path()) { [INFO] [stderr] | [INFO] [stderr] [INFO] [stderr] warning: hiding a lifetime that's elided elsewhere is confusing [INFO] [stderr] --> src/verify_project.rs:117:19 [INFO] [stderr] | [INFO] [stderr] 117 | fn basename(path: &str, sep: char) -> Cow { [INFO] [stderr] | ^^^^ ^^^^^^^^ the same lifetime is hidden here [INFO] [stderr] | | [INFO] [stderr] | the lifetime is elided here [INFO] [stderr] | [INFO] [stderr] = help: the same lifetime is referred to in inconsistent ways, making the signature confusing [INFO] [stderr] = note: `#[warn(mismatched_lifetime_syntaxes)]` on by default [INFO] [stderr] help: use `'_` for type paths [INFO] [stderr] | [INFO] [stderr] 117 | fn basename(path: &str, sep: char) -> Cow<'_, str> { [INFO] [stderr] | +++ [INFO] [stderr] [INFO] [stderr] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stderr] --> src/ser_de/manifest/manifest_utils.rs:76:37 [INFO] [stderr] | [INFO] [stderr] 76 | #[derive(Default, PartialEq, Debug, YaSerialize)] [INFO] [stderr] | ^---------- [INFO] [stderr] | | [INFO] [stderr] | `YaSerialize` is not local [INFO] [stderr] | move the `impl` block outside of this constant `_IMPL_YA_SERIALIZE_FOR_CIQManifest` [INFO] [stderr] 77 | #[yaserde(rename = "manifest")] [INFO] [stderr] 78 | struct CIQManifest { [INFO] [stderr] | ----------- `CIQManifest` is not local [INFO] [stderr] | [INFO] [stderr] = note: the derive macro `YaSerialize` defines the non-local `impl`, and may need to be changed [INFO] [stderr] = note: the derive macro `YaSerialize` may come from an old version of the `yaserde_derive` crate, try updating your dependency with `cargo update -p yaserde_derive` [INFO] [stderr] = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl` [INFO] [stderr] = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint [INFO] [stderr] = note: `#[warn(non_local_definitions)]` on by default [INFO] [stderr] = note: this warning originates in the derive macro `YaSerialize` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stderr] [INFO] [stderr] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stderr] --> src/ser_de/manifest/manifest_utils.rs:87:37 [INFO] [stderr] | [INFO] [stderr] 87 | #[derive(Default, PartialEq, Debug, YaSerialize)] [INFO] [stderr] | ^---------- [INFO] [stderr] | | [INFO] [stderr] | `YaSerialize` is not local [INFO] [stderr] | move the `impl` block outside of this constant `_IMPL_YA_SERIALIZE_FOR_CIQApplication` [INFO] [stderr] 88 | struct CIQApplication { [INFO] [stderr] | -------------- `CIQApplication` is not local [INFO] [stderr] | [INFO] [stderr] = note: the derive macro `YaSerialize` defines the non-local `impl`, and may need to be changed [INFO] [stderr] = note: the derive macro `YaSerialize` may come from an old version of the `yaserde_derive` crate, try updating your dependency with `cargo update -p yaserde_derive` [INFO] [stderr] = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl` [INFO] [stderr] = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint [INFO] [stderr] = note: this warning originates in the derive macro `YaSerialize` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stderr] [INFO] [stderr] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stderr] --> src/ser_de/manifest/manifest_utils.rs:113:37 [INFO] [stderr] | [INFO] [stderr] 113 | #[derive(Default, PartialEq, Debug, YaSerialize)] [INFO] [stderr] | ^---------- [INFO] [stderr] | | [INFO] [stderr] | `YaSerialize` is not local [INFO] [stderr] | move the `impl` block outside of this constant `_IMPL_YA_SERIALIZE_FOR_CIQProducts` [INFO] [stderr] 114 | struct CIQProducts { [INFO] [stderr] | ----------- `CIQProducts` is not local [INFO] [stderr] | [INFO] [stderr] = note: the derive macro `YaSerialize` defines the non-local `impl`, and may need to be changed [INFO] [stderr] = note: the derive macro `YaSerialize` may come from an old version of the `yaserde_derive` crate, try updating your dependency with `cargo update -p yaserde_derive` [INFO] [stderr] = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl` [INFO] [stderr] = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint [INFO] [stderr] = note: this warning originates in the derive macro `YaSerialize` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stderr] [INFO] [stderr] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stderr] --> src/ser_de/manifest/manifest_utils.rs:119:37 [INFO] [stderr] | [INFO] [stderr] 119 | #[derive(Default, PartialEq, Debug, YaSerialize)] [INFO] [stderr] | ^---------- [INFO] [stderr] | | [INFO] [stderr] | `YaSerialize` is not local [INFO] [stderr] | move the `impl` block outside of this constant `_IMPL_YA_SERIALIZE_FOR_CIQProduct` [INFO] [stderr] 120 | struct CIQProduct { [INFO] [stderr] | ---------- `CIQProduct` is not local [INFO] [stderr] | [INFO] [stderr] = note: the derive macro `YaSerialize` defines the non-local `impl`, and may need to be changed [INFO] [stderr] = note: the derive macro `YaSerialize` may come from an old version of the `yaserde_derive` crate, try updating your dependency with `cargo update -p yaserde_derive` [INFO] [stderr] = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl` [INFO] [stderr] = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint [INFO] [stderr] = note: this warning originates in the derive macro `YaSerialize` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stderr] [INFO] [stderr] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stderr] --> src/ser_de/manifest/manifest_utils.rs:125:37 [INFO] [stderr] | [INFO] [stderr] 125 | #[derive(Default, PartialEq, Debug, YaSerialize)] [INFO] [stderr] | ^---------- [INFO] [stderr] | | [INFO] [stderr] | `YaSerialize` is not local [INFO] [stderr] | move the `impl` block outside of this constant `_IMPL_YA_SERIALIZE_FOR_CIQPermissions` [INFO] [stderr] 126 | struct CIQPermissions { [INFO] [stderr] | -------------- `CIQPermissions` is not local [INFO] [stderr] | [INFO] [stderr] = note: the derive macro `YaSerialize` defines the non-local `impl`, and may need to be changed [INFO] [stderr] = note: the derive macro `YaSerialize` may come from an old version of the `yaserde_derive` crate, try updating your dependency with `cargo update -p yaserde_derive` [INFO] [stderr] = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl` [INFO] [stderr] = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint [INFO] [stderr] = note: this warning originates in the derive macro `YaSerialize` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stderr] [INFO] [stderr] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stderr] --> src/ser_de/manifest/manifest_utils.rs:132:37 [INFO] [stderr] | [INFO] [stderr] 132 | #[derive(Default, PartialEq, Debug, YaSerialize)] [INFO] [stderr] | ^---------- [INFO] [stderr] | | [INFO] [stderr] | `YaSerialize` is not local [INFO] [stderr] | move the `impl` block outside of this constant `_IMPL_YA_SERIALIZE_FOR_CIQPermission` [INFO] [stderr] 133 | struct CIQPermission { [INFO] [stderr] | ------------- `CIQPermission` is not local [INFO] [stderr] | [INFO] [stderr] = note: the derive macro `YaSerialize` defines the non-local `impl`, and may need to be changed [INFO] [stderr] = note: the derive macro `YaSerialize` may come from an old version of the `yaserde_derive` crate, try updating your dependency with `cargo update -p yaserde_derive` [INFO] [stderr] = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl` [INFO] [stderr] = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint [INFO] [stderr] = note: this warning originates in the derive macro `YaSerialize` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stderr] [INFO] [stderr] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stderr] --> src/ser_de/manifest/manifest_utils.rs:138:37 [INFO] [stderr] | [INFO] [stderr] 138 | #[derive(Default, PartialEq, Debug, YaSerialize)] [INFO] [stderr] | ^---------- [INFO] [stderr] | | [INFO] [stderr] | `YaSerialize` is not local [INFO] [stderr] | move the `impl` block outside of this constant `_IMPL_YA_SERIALIZE_FOR_CIQLanguages` [INFO] [stderr] 139 | struct CIQLanguages { [INFO] [stderr] | ------------ `CIQLanguages` is not local [INFO] [stderr] | [INFO] [stderr] = note: the derive macro `YaSerialize` defines the non-local `impl`, and may need to be changed [INFO] [stderr] = note: the derive macro `YaSerialize` may come from an old version of the `yaserde_derive` crate, try updating your dependency with `cargo update -p yaserde_derive` [INFO] [stderr] = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl` [INFO] [stderr] = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint [INFO] [stderr] = note: this warning originates in the derive macro `YaSerialize` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stderr] [INFO] [stderr] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stderr] --> src/ser_de/manifest/manifest_utils.rs:144:37 [INFO] [stderr] | [INFO] [stderr] 144 | #[derive(Default, PartialEq, Debug, YaSerialize)] [INFO] [stderr] | ^---------- [INFO] [stderr] | | [INFO] [stderr] | `YaSerialize` is not local [INFO] [stderr] | move the `impl` block outside of this constant `_IMPL_YA_SERIALIZE_FOR_CIQDependencies` [INFO] [stderr] 145 | struct CIQDependencies { [INFO] [stderr] | --------------- `CIQDependencies` is not local [INFO] [stderr] | [INFO] [stderr] = note: the derive macro `YaSerialize` defines the non-local `impl`, and may need to be changed [INFO] [stderr] = note: the derive macro `YaSerialize` may come from an old version of the `yaserde_derive` crate, try updating your dependency with `cargo update -p yaserde_derive` [INFO] [stderr] = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl` [INFO] [stderr] = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint [INFO] [stderr] = note: this warning originates in the derive macro `YaSerialize` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stderr] [INFO] [stderr] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stderr] --> src/ser_de/manifest/manifest_utils.rs:149:37 [INFO] [stderr] | [INFO] [stderr] 149 | #[derive(Default, PartialEq, Debug, YaSerialize)] [INFO] [stderr] | ^---------- [INFO] [stderr] | | [INFO] [stderr] | `YaSerialize` is not local [INFO] [stderr] | move the `impl` block outside of this constant `_IMPL_YA_SERIALIZE_FOR_CIQDependency` [INFO] [stderr] 150 | struct CIQDependency { [INFO] [stderr] | ------------- `CIQDependency` is not local [INFO] [stderr] | [INFO] [stderr] = note: the derive macro `YaSerialize` defines the non-local `impl`, and may need to be changed [INFO] [stderr] = note: the derive macro `YaSerialize` may come from an old version of the `yaserde_derive` crate, try updating your dependency with `cargo update -p yaserde_derive` [INFO] [stderr] = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl` [INFO] [stderr] = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint [INFO] [stderr] = note: this warning originates in the derive macro `YaSerialize` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stderr] [INFO] [stderr] warning: `kumitateru` (bin "kumitateru" test) generated 35 warnings (run `cargo fix --bin "kumitateru" --tests` to apply 5 suggestions) [INFO] [stderr] Finished `test` profile [unoptimized + debuginfo] target(s) in 0.06s [INFO] [stderr] warning: the following packages contain code that will be rejected by a future version of Rust: quick-xml v0.20.0 [INFO] [stderr] note: to see what the problems were, use the option `--future-incompat-report`, or run `cargo report future-incompatibilities --id 3` [INFO] [stdout] [INFO] [stderr] Running unittests src/main.rs (/opt/rustwide/target/debug/deps/kumitateru-5c3b15a490228b65) [INFO] [stdout] running 0 tests [INFO] [stdout] [INFO] [stdout] test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.00s [INFO] [stdout] [INFO] running `Command { std: "docker" "inspect" "cdb0cbb91c08d2d17c381a82aa616e2562d0d21be246a5661085119f18ba473f", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "cdb0cbb91c08d2d17c381a82aa616e2562d0d21be246a5661085119f18ba473f", kill_on_drop: false }` [INFO] [stdout] cdb0cbb91c08d2d17c381a82aa616e2562d0d21be246a5661085119f18ba473f