[INFO] cloning repository https://github.com/jmjoy/recorder [INFO] running `Command { std: "git" "-c" "credential.helper=" "-c" "credential.helper=/workspace/cargo-home/bin/git-credential-null" "clone" "--bare" "https://github.com/jmjoy/recorder" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fjmjoy%2Frecorder", kill_on_drop: false }` [INFO] [stderr] Cloning into bare repository '/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fjmjoy%2Frecorder'... [INFO] running `Command { std: "git" "rev-parse" "HEAD", kill_on_drop: false }` [INFO] [stdout] 12230fa58bc7ec097c0bc7b671283805b9fc0f60 [INFO] checking jmjoy/recorder against master#17ffbc81a30c094193836a5d7f90dff273b5df93 for pr-138785 [INFO] running `Command { std: "git" "clone" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fjmjoy%2Frecorder" "/workspace/builds/worker-5-tc2/source", kill_on_drop: false }` [INFO] [stderr] Cloning into '/workspace/builds/worker-5-tc2/source'... [INFO] [stderr] done. [INFO] validating manifest of git repo https://github.com/jmjoy/recorder on toolchain 17ffbc81a30c094193836a5d7f90dff273b5df93 [INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+17ffbc81a30c094193836a5d7f90dff273b5df93" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }` [INFO] started tweaking git repo https://github.com/jmjoy/recorder [INFO] finished tweaking git repo https://github.com/jmjoy/recorder [INFO] tweaked toml for git repo https://github.com/jmjoy/recorder written to /workspace/builds/worker-5-tc2/source/Cargo.toml [INFO] crate git repo https://github.com/jmjoy/recorder 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" "+17ffbc81a30c094193836a5d7f90dff273b5df93" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }` [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-5-tc2/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-5-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:b0b074c097205a61b89e8ad263052f976b2b332c4dc5f02aef1fe52501660d6e" "/opt/rustwide/cargo-home/bin/cargo" "+17ffbc81a30c094193836a5d7f90dff273b5df93" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }` [INFO] [stdout] d6bdb7786bd94c9eb23ddd7be3a47f5237a37899fd8fa730602ea361ec8f844e [INFO] running `Command { std: "docker" "start" "-a" "d6bdb7786bd94c9eb23ddd7be3a47f5237a37899fd8fa730602ea361ec8f844e", kill_on_drop: false }` [INFO] running `Command { std: "docker" "inspect" "d6bdb7786bd94c9eb23ddd7be3a47f5237a37899fd8fa730602ea361ec8f844e", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "d6bdb7786bd94c9eb23ddd7be3a47f5237a37899fd8fa730602ea361ec8f844e", kill_on_drop: false }` [INFO] [stdout] d6bdb7786bd94c9eb23ddd7be3a47f5237a37899fd8fa730602ea361ec8f844e [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-5-tc2/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-5-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:b0b074c097205a61b89e8ad263052f976b2b332c4dc5f02aef1fe52501660d6e" "/opt/rustwide/cargo-home/bin/cargo" "+17ffbc81a30c094193836a5d7f90dff273b5df93" "check" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }` [INFO] [stdout] d6d2b892887caab64f73ea83494b2a183d06b30b4736c2859ead92d413c1a275 [INFO] running `Command { std: "docker" "start" "-a" "d6d2b892887caab64f73ea83494b2a183d06b30b4736c2859ead92d413c1a275", kill_on_drop: false }` [INFO] [stderr] Compiling autocfg v0.1.6 [INFO] [stderr] Compiling proc-macro2 v1.0.4 [INFO] [stderr] Compiling unicode-xid v0.2.0 [INFO] [stderr] Compiling syn v1.0.5 [INFO] [stderr] Compiling libc v0.2.62 [INFO] [stderr] Compiling bitflags v1.2.0 [INFO] [stderr] Checking unicode-width v0.1.6 [INFO] [stderr] Compiling unicode-segmentation v1.3.0 [INFO] [stderr] Checking strsim v0.8.0 [INFO] [stderr] Checking ansi_term v0.11.0 [INFO] [stderr] Checking vec_map v0.8.1 [INFO] [stderr] Checking lazy_static v1.4.0 [INFO] [stderr] Checking textwrap v0.11.0 [INFO] [stderr] Compiling num-traits v0.2.8 [INFO] [stderr] Compiling num-integer v0.1.41 [INFO] [stderr] Compiling num-bigint v0.2.3 [INFO] [stderr] Compiling num-iter v0.1.39 [INFO] [stderr] Compiling num-rational v0.2.2 [INFO] [stderr] Compiling num-complex v0.2.3 [INFO] [stderr] Compiling heck v0.3.1 [INFO] [stderr] Compiling quote v1.0.2 [INFO] [stderr] Checking atty v0.2.13 [INFO] [stderr] Checking clap v2.33.0 [INFO] [stderr] Checking num v0.2.0 [INFO] [stderr] Compiling proc-macro-error v0.2.6 [INFO] [stderr] Compiling structopt-derive v0.3.2 [INFO] [stderr] Compiling num-derive v0.3.0 [INFO] [stderr] Checking structopt v0.3.2 [INFO] [stderr] Checking recorder v0.1.0 (/opt/rustwide/workdir) [INFO] [stdout] warning: unused import: `crate::tone::Tone` [INFO] [stdout] --> src/notation.rs:59:9 [INFO] [stdout] | [INFO] [stdout] 59 | use crate::tone::Tone; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_imports)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `crate::tone::Tone` [INFO] [stdout] --> src/notation.rs:59:9 [INFO] [stdout] | [INFO] [stdout] 59 | use crate::tone::Tone; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_imports)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `super::*` [INFO] [stdout] --> src/finger.rs:79:9 [INFO] [stdout] | [INFO] [stdout] 79 | use super::*; [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `n` [INFO] [stdout] --> src/tone.rs:441:33 [INFO] [stdout] | [INFO] [stdout] 441 | Token::Notation(n) => { [INFO] [stdout] | ^ help: if this is intentional, prefix it with an underscore: `_n` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `t` [INFO] [stdout] --> src/tone.rs:446:17 [INFO] [stdout] | [INFO] [stdout] 446 | t => s.to_owned(), [INFO] [stdout] | ^ help: if this is intentional, prefix it with an underscore: `_t` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `n` [INFO] [stdout] --> src/tone.rs:474:33 [INFO] [stdout] | [INFO] [stdout] 474 | Token::Notation(n) => { [INFO] [stdout] | ^ help: if this is intentional, prefix it with an underscore: `_n` [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/tone.rs:10:53 [INFO] [stdout] | [INFO] [stdout] 10 | #[derive(Debug, Clone, Copy, PartialEq, PartialOrd, FromPrimitive)] [INFO] [stdout] | ^------------ [INFO] [stdout] | | [INFO] [stdout] | `FromPrimitive` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_IMPL_NUM_FromPrimitive_FOR_Tone` [INFO] [stdout] 11 | #[repr(usize)] [INFO] [stdout] 12 | pub enum Tone { [INFO] [stdout] | ---- `Tone` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `FromPrimitive` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `FromPrimitive` may come from an old version of the `num_derive` crate, try updating your dependency with `cargo update -p num_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 `FromPrimitive` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `n` [INFO] [stdout] --> src/tone.rs:441:33 [INFO] [stdout] | [INFO] [stdout] 441 | Token::Notation(n) => { [INFO] [stdout] | ^ help: if this is intentional, prefix it with an underscore: `_n` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `t` [INFO] [stdout] --> src/tone.rs:446:17 [INFO] [stdout] | [INFO] [stdout] 446 | t => s.to_owned(), [INFO] [stdout] | ^ help: if this is intentional, prefix it with an underscore: `_t` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `n` [INFO] [stdout] --> src/tone.rs:474:33 [INFO] [stdout] | [INFO] [stdout] 474 | Token::Notation(n) => { [INFO] [stdout] | ^ help: if this is intentional, prefix it with an underscore: `_n` [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/tone.rs:10:53 [INFO] [stdout] | [INFO] [stdout] 10 | #[derive(Debug, Clone, Copy, PartialEq, PartialOrd, FromPrimitive)] [INFO] [stdout] | ^------------ [INFO] [stdout] | | [INFO] [stdout] | `FromPrimitive` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_IMPL_NUM_FromPrimitive_FOR_Tone` [INFO] [stdout] 11 | #[repr(usize)] [INFO] [stdout] 12 | pub enum Tone { [INFO] [stdout] | ---- `Tone` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `FromPrimitive` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `FromPrimitive` may come from an old version of the `num_derive` crate, try updating your dependency with `cargo update -p num_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 `FromPrimitive` (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 7.10s [INFO] running `Command { std: "docker" "inspect" "d6d2b892887caab64f73ea83494b2a183d06b30b4736c2859ead92d413c1a275", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "d6d2b892887caab64f73ea83494b2a183d06b30b4736c2859ead92d413c1a275", kill_on_drop: false }` [INFO] [stdout] d6d2b892887caab64f73ea83494b2a183d06b30b4736c2859ead92d413c1a275