[INFO] fetching crate nsh 0.4.2... [INFO] checking nsh-0.4.2 against try#4f260f0f20b3133d20cfb50353c0221943af5796+rustflags=-Dunmustuse_in_always_ok for pr-148577 [INFO] extracting crate nsh 0.4.2 into /workspace/builds/worker-1-tc2/source [INFO] started tweaking crates.io crate nsh 0.4.2 [INFO] finished tweaking crates.io crate nsh 0.4.2 [INFO] tweaked toml for crates.io crate nsh 0.4.2 written to /workspace/builds/worker-1-tc2/source/Cargo.toml [INFO] validating manifest of crates.io crate nsh 0.4.2 on toolchain 4f260f0f20b3133d20cfb50353c0221943af5796 [INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+4f260f0f20b3133d20cfb50353c0221943af5796" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }` [INFO] crate crates.io crate nsh 0.4.2 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" "+4f260f0f20b3133d20cfb50353c0221943af5796" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }` [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-1-tc2/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-1-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:4848fb76d95f26979359cc7e45710b1dbc8f3acb7aeedee7c460d7702230f228" "/opt/rustwide/cargo-home/bin/cargo" "+4f260f0f20b3133d20cfb50353c0221943af5796" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }` [INFO] [stdout] e1b0ca366be11deb2f73bcd3331a0ecf8f228b46393263a79c155927a7497ae3 [INFO] running `Command { std: "docker" "start" "-a" "e1b0ca366be11deb2f73bcd3331a0ecf8f228b46393263a79c155927a7497ae3", kill_on_drop: false }` [INFO] running `Command { std: "docker" "inspect" "e1b0ca366be11deb2f73bcd3331a0ecf8f228b46393263a79c155927a7497ae3", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "e1b0ca366be11deb2f73bcd3331a0ecf8f228b46393263a79c155927a7497ae3", kill_on_drop: false }` [INFO] [stdout] e1b0ca366be11deb2f73bcd3331a0ecf8f228b46393263a79c155927a7497ae3 [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-1-tc2/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-1-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 -Dunmustuse_in_always_ok" "-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:4848fb76d95f26979359cc7e45710b1dbc8f3acb7aeedee7c460d7702230f228" "/opt/rustwide/cargo-home/bin/cargo" "+4f260f0f20b3133d20cfb50353c0221943af5796" "check" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }` [INFO] [stdout] 00cf02c9c234548274105e80bf131a9936b4cb4ea7d6da6ddfbc86dd386b62c5 [INFO] running `Command { std: "docker" "start" "-a" "00cf02c9c234548274105e80bf131a9936b4cb4ea7d6da6ddfbc86dd386b62c5", kill_on_drop: false }` [INFO] [stderr] Compiling libc v0.2.98 [INFO] [stderr] Compiling proc-macro2 v1.0.27 [INFO] [stderr] Compiling syn v1.0.73 [INFO] [stderr] Compiling log v0.4.14 [INFO] [stderr] Compiling proc-macro-error-attr v1.0.4 [INFO] [stderr] Compiling memchr v2.4.0 [INFO] [stderr] Compiling ppv-lite86 v0.2.10 [INFO] [stderr] Compiling proc-macro-error v1.0.4 [INFO] [stderr] Compiling miniz_oxide v0.4.4 [INFO] [stderr] Compiling cc v1.0.69 [INFO] [stderr] Compiling siphasher v0.3.5 [INFO] [stderr] Compiling proc-macro-hack v0.5.19 [INFO] [stderr] Compiling phf_shared v0.9.0 [INFO] [stderr] Checking instant v0.1.10 [INFO] [stderr] Compiling memoffset v0.6.4 [INFO] [stderr] Compiling maplit v1.0.2 [INFO] [stderr] Compiling unicode-segmentation v1.8.0 [INFO] [stderr] Checking gimli v0.24.0 [INFO] [stderr] Checking smallvec v1.6.1 [INFO] [stderr] Checking object v0.25.3 [INFO] [stderr] Compiling heck v0.3.3 [INFO] [stderr] Checking lock_api v0.4.4 [INFO] [stderr] Compiling quote v1.0.9 [INFO] [stderr] Compiling pest_meta v2.1.3 [INFO] [stderr] Compiling backtrace v0.3.60 [INFO] [stderr] Checking textwrap v0.11.0 [INFO] [stderr] Checking atty v0.2.14 [INFO] [stderr] Checking getrandom v0.2.3 [INFO] [stderr] Checking signal-hook-registry v1.4.0 [INFO] [stderr] Checking rand_core v0.6.3 [INFO] [stderr] Checking mio v0.7.13 [INFO] [stderr] Checking parking_lot_core v0.8.3 [INFO] [stderr] Compiling rand_chacha v0.3.1 [INFO] [stderr] Checking ansi_term v0.11.0 [INFO] [stderr] Checking rustc-demangle v0.1.20 [INFO] [stderr] Compiling rand v0.8.4 [INFO] [stderr] Checking clap v2.33.3 [INFO] [stderr] Checking parking_lot v0.11.1 [INFO] [stderr] Checking signal-hook v0.1.17 [INFO] [stderr] Checking addr2line v0.15.2 [INFO] [stderr] Checking colored v1.9.3 [INFO] [stderr] Checking ucd-trie v0.1.3 [INFO] [stderr] Checking crossterm v0.18.2 [INFO] [stderr] Checking nix v0.22.0 [INFO] [stderr] Checking pest v2.1.3 [INFO] [stderr] Checking fern v0.6.0 [INFO] [stderr] Compiling phf_generator v0.9.0 [INFO] [stderr] Checking signal-hook v0.2.3 [INFO] [stderr] Checking dirs v1.0.5 [INFO] [stderr] Checking glob v0.3.0 [INFO] [stderr] Checking diff v0.1.12 [INFO] [stderr] Checking tempfile v3.2.0 [INFO] [stderr] Checking pretty_assertions v0.7.2 [INFO] [stderr] Compiling synstructure v0.12.5 [INFO] [stderr] Compiling pest_generator v2.1.3 [INFO] [stderr] Compiling structopt-derive v0.4.15 [INFO] [stderr] Compiling phf_macros v0.9.0 [INFO] [stderr] Compiling pest_derive v2.1.0 [INFO] [stderr] Compiling failure_derive v0.1.8 [INFO] [stderr] Checking failure v0.1.8 [INFO] [stderr] Checking phf v0.9.0 [INFO] [stderr] Checking structopt v0.3.22 [INFO] [stderr] Checking nsh v0.4.2 (/opt/rustwide/workdir) [INFO] [stdout] warning: field `login` is never read [INFO] [stdout] --> src/main.rs:153:5 [INFO] [stdout] | [INFO] [stdout] 144 | struct Opt { [INFO] [stdout] | --- field in this struct [INFO] [stdout] ... [INFO] [stdout] 153 | login: bool, [INFO] [stdout] | ^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `Opt` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis [INFO] [stdout] = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: this type will no longer be must used: std::result::Result [INFO] [stdout] --> src/main.rs:145:5 [INFO] [stdout] | [INFO] [stdout] 145 | / /// The command to be executed. [INFO] [stdout] 146 | | #[structopt(short = "c")] [INFO] [stdout] 147 | | command: Option, [INFO] [stdout] | |___________________________^ [INFO] [stdout] | [INFO] [stdout] = note: requested on the command line with `-D unmustuse-in-always-ok` [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/builtins/mod.rs:34:17 [INFO] [stdout] | [INFO] [stdout] 34 | #[derive(Debug, Fail)] [INFO] [stdout] | ^--- [INFO] [stdout] | | [INFO] [stdout] | `Fail` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_DERIVE_failure_Fail_FOR_InternalCommandError` [INFO] [stdout] 35 | pub enum InternalCommandError { [INFO] [stdout] | -------------------- `InternalCommandError` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `Fail` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `Fail` may come from an old version of the `failure_derive` crate, try updating your dependency with `cargo update -p failure_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 `Fail` (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/builtins/mod.rs:34:17 [INFO] [stdout] | [INFO] [stdout] 34 | #[derive(Debug, Fail)] [INFO] [stdout] | ^--- [INFO] [stdout] | | [INFO] [stdout] | `Display` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_DERIVE_failure_core_fmt_Display_FOR_InternalCommandError` [INFO] [stdout] 35 | pub enum InternalCommandError { [INFO] [stdout] | -------------------- `InternalCommandError` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `Fail` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `Fail` may come from an old version of the `failure_derive` crate, try updating your dependency with `cargo update -p failure_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 `Fail` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: this type will no longer be must used: std::result::Result [INFO] [stdout] --> src/builtins/bg.rs:10:5 [INFO] [stdout] | [INFO] [stdout] 10 | / #[structopt(name = "job_id")] [INFO] [stdout] 11 | | job_id: Option, [INFO] [stdout] | |__________________________^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: this type will no longer be must used: std::result::Result [INFO] [stdout] --> src/builtins/fg.rs:10:5 [INFO] [stdout] | [INFO] [stdout] 10 | / #[structopt(name = "job_id")] [INFO] [stdout] 11 | | job_id: Option, [INFO] [stdout] | |__________________________^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: this type will no longer be must used: std::result::Result [INFO] [stdout] --> src/builtins/pushd.rs:9:5 [INFO] [stdout] | [INFO] [stdout] 9 | / #[structopt(name = "dir")] [INFO] [stdout] 10 | | dir: Option, [INFO] [stdout] | |_______________________^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: this type will no longer be must used: std::result::Result [INFO] [stdout] --> src/builtins/read.rs:10:5 [INFO] [stdout] | [INFO] [stdout] 10 | / #[structopt(short = "p")] [INFO] [stdout] 11 | | prompt: Option, [INFO] [stdout] | |__________________________^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: this type will no longer be must used: std::result::Result [INFO] [stdout] --> src/builtins/read.rs:12:5 [INFO] [stdout] | [INFO] [stdout] 12 | / #[structopt(name = "VAR")] [INFO] [stdout] 13 | | var_name: String, [INFO] [stdout] | |____________________^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: this type will no longer be must used: std::result::Result [INFO] [stdout] --> src/builtins/unset.rs:9:5 [INFO] [stdout] | [INFO] [stdout] 9 | / #[structopt(name = "NAME")] [INFO] [stdout] 10 | | name: String, [INFO] [stdout] | |________________^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: this type will no longer be must used: std::result::Result [INFO] [stdout] --> src/builtins/wait.rs:11:5 [INFO] [stdout] | [INFO] [stdout] 11 | / #[structopt(name = "job_id")] [INFO] [stdout] 12 | | job_id: Option, [INFO] [stdout] | |__________________________^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing [INFO] [stdout] --> src/fuzzy.rs:37:17 [INFO] [stdout] | [INFO] [stdout] 37 | pub fn iter(&self) -> std::slice::Iter<(Option, String)> { [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] 37 | pub fn iter(&self) -> std::slice::Iter<'_, (Option, String)> { [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/pattern.rs:4:17 [INFO] [stdout] | [INFO] [stdout] 4 | #[derive(Debug, Fail)] [INFO] [stdout] | ^--- [INFO] [stdout] | | [INFO] [stdout] | `Fail` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_DERIVE_failure_Fail_FOR_NoMatchesError` [INFO] [stdout] 5 | #[fail(display = "no matches")] [INFO] [stdout] 6 | pub struct NoMatchesError; [INFO] [stdout] | -------------- `NoMatchesError` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `Fail` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `Fail` may come from an old version of the `failure_derive` crate, try updating your dependency with `cargo update -p failure_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 `Fail` (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/pattern.rs:4:17 [INFO] [stdout] | [INFO] [stdout] 4 | #[derive(Debug, Fail)] [INFO] [stdout] | ^--- [INFO] [stdout] | | [INFO] [stdout] | `Display` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_DERIVE_failure_core_fmt_Display_FOR_NoMatchesError` [INFO] [stdout] 5 | #[fail(display = "no matches")] [INFO] [stdout] 6 | pub struct NoMatchesError; [INFO] [stdout] | -------------- `NoMatchesError` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `Fail` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `Fail` may come from an old version of the `failure_derive` crate, try updating your dependency with `cargo update -p failure_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 `Fail` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing [INFO] [stdout] --> src/shell.rs:227:27 [INFO] [stdout] | [INFO] [stdout] 227 | pub fn exported_names(&self) -> std::collections::hash_set::Iter { [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] help: use `'_` for type paths [INFO] [stdout] | [INFO] [stdout] 227 | pub fn exported_names(&self) -> std::collections::hash_set::Iter<'_, String> { [INFO] [stdout] | +++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing [INFO] [stdout] --> src/shell.rs:231:20 [INFO] [stdout] | [INFO] [stdout] 231 | pub fn aliases(&self) -> std::collections::hash_map::Iter { [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] help: use `'_` for type paths [INFO] [stdout] | [INFO] [stdout] 231 | pub fn aliases(&self) -> std::collections::hash_map::Iter<'_, String, String> { [INFO] [stdout] | +++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: field `login` is never read [INFO] [stdout] --> src/main.rs:153:5 [INFO] [stdout] | [INFO] [stdout] 144 | struct Opt { [INFO] [stdout] | --- field in this struct [INFO] [stdout] ... [INFO] [stdout] 153 | login: bool, [INFO] [stdout] | ^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `Opt` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis [INFO] [stdout] = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: this type will no longer be must used: std::result::Result [INFO] [stdout] --> src/main.rs:145:5 [INFO] [stdout] | [INFO] [stdout] 145 | / /// The command to be executed. [INFO] [stdout] 146 | | #[structopt(short = "c")] [INFO] [stdout] 147 | | command: Option, [INFO] [stdout] | |___________________________^ [INFO] [stdout] | [INFO] [stdout] = note: requested on the command line with `-D unmustuse-in-always-ok` [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/builtins/mod.rs:34:17 [INFO] [stdout] | [INFO] [stdout] 34 | #[derive(Debug, Fail)] [INFO] [stdout] | ^--- [INFO] [stdout] | | [INFO] [stdout] | `Fail` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_DERIVE_failure_Fail_FOR_InternalCommandError` [INFO] [stdout] 35 | pub enum InternalCommandError { [INFO] [stdout] | -------------------- `InternalCommandError` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `Fail` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `Fail` may come from an old version of the `failure_derive` crate, try updating your dependency with `cargo update -p failure_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 `Fail` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stderr] error: could not compile `nsh` (bin "nsh" test) due to 8 previous errors; 8 warnings emitted [INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item [INFO] [stdout] --> src/builtins/mod.rs:34:17 [INFO] [stdout] | [INFO] [stdout] 34 | #[derive(Debug, Fail)] [INFO] [stdout] | ^--- [INFO] [stdout] | | [INFO] [stdout] | `Display` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_DERIVE_failure_core_fmt_Display_FOR_InternalCommandError` [INFO] [stdout] 35 | pub enum InternalCommandError { [INFO] [stdout] | -------------------- `InternalCommandError` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `Fail` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `Fail` may come from an old version of the `failure_derive` crate, try updating your dependency with `cargo update -p failure_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 `Fail` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stderr] warning: build failed, waiting for other jobs to finish... [INFO] [stdout] error: this type will no longer be must used: std::result::Result [INFO] [stdout] --> src/builtins/bg.rs:10:5 [INFO] [stdout] | [INFO] [stdout] 10 | / #[structopt(name = "job_id")] [INFO] [stdout] 11 | | job_id: Option, [INFO] [stdout] | |__________________________^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: this type will no longer be must used: std::result::Result [INFO] [stdout] --> src/builtins/fg.rs:10:5 [INFO] [stdout] | [INFO] [stdout] 10 | / #[structopt(name = "job_id")] [INFO] [stdout] 11 | | job_id: Option, [INFO] [stdout] | |__________________________^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: this type will no longer be must used: std::result::Result [INFO] [stdout] --> src/builtins/pushd.rs:9:5 [INFO] [stdout] | [INFO] [stdout] 9 | / #[structopt(name = "dir")] [INFO] [stdout] 10 | | dir: Option, [INFO] [stdout] | |_______________________^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: this type will no longer be must used: std::result::Result [INFO] [stdout] --> src/builtins/read.rs:10:5 [INFO] [stdout] | [INFO] [stdout] 10 | / #[structopt(short = "p")] [INFO] [stdout] 11 | | prompt: Option, [INFO] [stdout] | |__________________________^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: this type will no longer be must used: std::result::Result [INFO] [stdout] --> src/builtins/read.rs:12:5 [INFO] [stdout] | [INFO] [stdout] 12 | / #[structopt(name = "VAR")] [INFO] [stdout] 13 | | var_name: String, [INFO] [stdout] | |____________________^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: this type will no longer be must used: std::result::Result [INFO] [stdout] --> src/builtins/unset.rs:9:5 [INFO] [stdout] | [INFO] [stdout] 9 | / #[structopt(name = "NAME")] [INFO] [stdout] 10 | | name: String, [INFO] [stdout] | |________________^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: this type will no longer be must used: std::result::Result [INFO] [stdout] --> src/builtins/wait.rs:11:5 [INFO] [stdout] | [INFO] [stdout] 11 | / #[structopt(name = "job_id")] [INFO] [stdout] 12 | | job_id: Option, [INFO] [stdout] | |__________________________^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing [INFO] [stdout] --> src/fuzzy.rs:37:17 [INFO] [stdout] | [INFO] [stdout] 37 | pub fn iter(&self) -> std::slice::Iter<(Option, String)> { [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] 37 | pub fn iter(&self) -> std::slice::Iter<'_, (Option, String)> { [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/pattern.rs:4:17 [INFO] [stdout] | [INFO] [stdout] 4 | #[derive(Debug, Fail)] [INFO] [stdout] | ^--- [INFO] [stdout] | | [INFO] [stdout] | `Fail` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_DERIVE_failure_Fail_FOR_NoMatchesError` [INFO] [stdout] 5 | #[fail(display = "no matches")] [INFO] [stdout] 6 | pub struct NoMatchesError; [INFO] [stdout] | -------------- `NoMatchesError` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `Fail` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `Fail` may come from an old version of the `failure_derive` crate, try updating your dependency with `cargo update -p failure_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 `Fail` (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/pattern.rs:4:17 [INFO] [stdout] | [INFO] [stdout] 4 | #[derive(Debug, Fail)] [INFO] [stdout] | ^--- [INFO] [stdout] | | [INFO] [stdout] | `Display` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_DERIVE_failure_core_fmt_Display_FOR_NoMatchesError` [INFO] [stdout] 5 | #[fail(display = "no matches")] [INFO] [stdout] 6 | pub struct NoMatchesError; [INFO] [stdout] | -------------- `NoMatchesError` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `Fail` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `Fail` may come from an old version of the `failure_derive` crate, try updating your dependency with `cargo update -p failure_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 `Fail` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing [INFO] [stdout] --> src/shell.rs:227:27 [INFO] [stdout] | [INFO] [stdout] 227 | pub fn exported_names(&self) -> std::collections::hash_set::Iter { [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] help: use `'_` for type paths [INFO] [stdout] | [INFO] [stdout] 227 | pub fn exported_names(&self) -> std::collections::hash_set::Iter<'_, String> { [INFO] [stdout] | +++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing [INFO] [stdout] --> src/shell.rs:231:20 [INFO] [stdout] | [INFO] [stdout] 231 | pub fn aliases(&self) -> std::collections::hash_map::Iter { [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] help: use `'_` for type paths [INFO] [stdout] | [INFO] [stdout] 231 | pub fn aliases(&self) -> std::collections::hash_map::Iter<'_, String, String> { [INFO] [stdout] | +++ [INFO] [stdout] [INFO] [stdout] [INFO] [stderr] error: could not compile `nsh` (bin "nsh") due to 8 previous errors; 8 warnings emitted [INFO] running `Command { std: "docker" "inspect" "00cf02c9c234548274105e80bf131a9936b4cb4ea7d6da6ddfbc86dd386b62c5", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "00cf02c9c234548274105e80bf131a9936b4cb4ea7d6da6ddfbc86dd386b62c5", kill_on_drop: false }` [INFO] [stdout] 00cf02c9c234548274105e80bf131a9936b4cb4ea7d6da6ddfbc86dd386b62c5