[INFO] cloning repository https://github.com/tailhook/nginx-config-mod [INFO] running `Command { std: "git" "-c" "credential.helper=" "-c" "credential.helper=/workspace/cargo-home/bin/git-credential-null" "clone" "--bare" "https://github.com/tailhook/nginx-config-mod" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Ftailhook%2Fnginx-config-mod", kill_on_drop: false }` [INFO] [stderr] Cloning into bare repository '/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Ftailhook%2Fnginx-config-mod'... [INFO] running `Command { std: "git" "rev-parse" "HEAD", kill_on_drop: false }` [INFO] [stdout] 4be0df3b7390f79536ca05b3ac27c72897746007 [INFO] checking tailhook/nginx-config-mod/4be0df3b7390f79536ca05b3ac27c72897746007 against master#c720f49c46ec3e70bbc2bbb3e0dd1fe7fc0a47f2 for pr-142712 [INFO] running `Command { std: "git" "clone" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Ftailhook%2Fnginx-config-mod" "/workspace/builds/worker-5-tc1/source", kill_on_drop: false }` [INFO] [stderr] Cloning into '/workspace/builds/worker-5-tc1/source'... [INFO] [stderr] done. [INFO] started tweaking git repo https://github.com/tailhook/nginx-config-mod [INFO] finished tweaking git repo https://github.com/tailhook/nginx-config-mod [INFO] tweaked toml for git repo https://github.com/tailhook/nginx-config-mod written to /workspace/builds/worker-5-tc1/source/Cargo.toml [INFO] validating manifest of git repo https://github.com/tailhook/nginx-config-mod on toolchain c720f49c46ec3e70bbc2bbb3e0dd1fe7fc0a47f2 [INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+c720f49c46ec3e70bbc2bbb3e0dd1fe7fc0a47f2" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }` [INFO] crate git repo https://github.com/tailhook/nginx-config-mod 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" "+c720f49c46ec3e70bbc2bbb3e0dd1fe7fc0a47f2" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }` [INFO] [stderr] warning: no edition set: defaulting to the 2015 edition while the latest is 2024 [INFO] [stderr] Updating crates.io index [INFO] [stderr] Downloading crates ... [INFO] [stderr] Downloaded structopt v0.2.10 [INFO] [stderr] Downloaded failure_derive v0.1.2 [INFO] [stderr] Downloaded wincolor v1.0.0 [INFO] [stderr] Downloaded structopt-derive v0.2.10 [INFO] [stderr] Downloaded aho-corasick v0.6.6 [INFO] [stderr] Downloaded synstructure v0.9.0 [INFO] [stderr] Downloaded termcolor v1.0.1 [INFO] [stderr] Downloaded failure v0.1.2 [INFO] [stderr] Downloaded nginx-config v0.13.2 [INFO] [stderr] Downloaded env_logger v0.5.12 [INFO] [stderr] Downloaded combine v3.5.1 [INFO] [stderr] Downloaded regex v1.0.2 [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-5-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-5-tc1/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:90999bfc7ae267e83380e433d8e61a7c072ca6729e92edbae886d3423b3a6f4c" "/opt/rustwide/cargo-home/bin/cargo" "+c720f49c46ec3e70bbc2bbb3e0dd1fe7fc0a47f2" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }` [INFO] [stdout] 6afb2c6492d66af58fc937ab0cf7ac937f866be3edfa379cf51d1a8e29a4a74f [INFO] running `Command { std: "docker" "start" "-a" "6afb2c6492d66af58fc937ab0cf7ac937f866be3edfa379cf51d1a8e29a4a74f", kill_on_drop: false }` [INFO] running `Command { std: "docker" "inspect" "6afb2c6492d66af58fc937ab0cf7ac937f866be3edfa379cf51d1a8e29a4a74f", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "6afb2c6492d66af58fc937ab0cf7ac937f866be3edfa379cf51d1a8e29a4a74f", kill_on_drop: false }` [INFO] [stdout] 6afb2c6492d66af58fc937ab0cf7ac937f866be3edfa379cf51d1a8e29a4a74f [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-5-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-5-tc1/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:90999bfc7ae267e83380e433d8e61a7c072ca6729e92edbae886d3423b3a6f4c" "/opt/rustwide/cargo-home/bin/cargo" "+c720f49c46ec3e70bbc2bbb3e0dd1fe7fc0a47f2" "check" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }` [INFO] [stdout] bdd162f04948e03128df4151d6ec2fcd8d40e28af9323de09952fd436a2e57e5 [INFO] running `Command { std: "docker" "start" "-a" "bdd162f04948e03128df4151d6ec2fcd8d40e28af9323de09952fd436a2e57e5", kill_on_drop: false }` [INFO] [stderr] warning: no edition set: defaulting to the 2015 edition while the latest is 2024 [INFO] [stderr] Checking libc v0.2.43 [INFO] [stderr] Compiling proc-macro2 v0.4.12 [INFO] [stderr] Checking void v1.0.2 [INFO] [stderr] Compiling version_check v0.1.4 [INFO] [stderr] Compiling cc v1.0.18 [INFO] [stderr] Checking cfg-if v0.1.5 [INFO] [stderr] Compiling failure_derive v0.1.2 [INFO] [stderr] Checking matches v0.1.7 [INFO] [stderr] Checking strsim v0.7.0 [INFO] [stderr] Checking rustc-demangle v0.1.9 [INFO] [stderr] Checking unreachable v1.0.0 [INFO] [stderr] Checking ucd-util v0.1.1 [INFO] [stderr] Compiling regex v1.0.2 [INFO] [stderr] Checking textwrap v0.10.0 [INFO] [stderr] Checking unicode-bidi v0.3.4 [INFO] [stderr] Checking regex-syntax v0.6.2 [INFO] [stderr] Compiling lazy_static v1.1.0 [INFO] [stderr] Checking unicode-normalization v0.1.7 [INFO] [stderr] Checking quick-error v1.2.2 [INFO] [stderr] Checking bitflags v1.0.3 [INFO] [stderr] Compiling quote v0.6.6 [INFO] [stderr] Checking byteorder v1.2.4 [INFO] [stderr] Checking memchr v2.0.1 [INFO] [stderr] Checking atty v0.2.11 [INFO] [stderr] Compiling syn v0.14.8 [INFO] [stderr] Checking aho-corasick v0.6.6 [INFO] [stderr] Checking thread_local v0.3.5 [INFO] [stderr] Checking utf8-ranges v1.0.0 [INFO] [stderr] Checking either v1.5.0 [INFO] [stderr] Checking ascii v0.7.1 [INFO] [stderr] Checking clap v2.32.0 [INFO] [stderr] Checking humantime v1.1.1 [INFO] [stderr] Checking log v0.4.3 [INFO] [stderr] Checking combine v3.5.1 [INFO] [stderr] Checking termcolor v1.0.1 [INFO] [stderr] Compiling backtrace-sys v0.1.24 [INFO] [stderr] Checking idna v0.1.5 [INFO] [stderr] Checking backtrace v0.3.9 [INFO] [stderr] Checking url v1.7.1 [INFO] [stderr] Checking env_logger v0.5.12 [INFO] [stderr] Compiling synstructure v0.9.0 [INFO] [stderr] Compiling structopt-derive v0.2.10 [INFO] [stderr] Checking failure v0.1.2 [INFO] [stderr] Checking structopt v0.2.10 [INFO] [stderr] Checking nginx-config v0.13.2 [INFO] [stderr] Checking nginx-config-mod v0.2.9 (/opt/rustwide/workdir) [INFO] [stdout] warning: unnecessary trailing semicolon [INFO] [stdout] --> src/checks/proxy_pass.rs:30:33 [INFO] [stdout] | [INFO] [stdout] 30 | let mut errors = Vec::new();; [INFO] [stdout] | ^ help: remove this semicolon [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(redundant_semicolons)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary trailing semicolon [INFO] [stdout] --> src/checks/proxy_pass.rs:30:33 [INFO] [stdout] | [INFO] [stdout] 30 | let mut errors = Vec::new();; [INFO] [stdout] | ^ help: remove this semicolon [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(redundant_semicolons)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: lifetime flowing from input to output with different syntax can be confusing [INFO] [stdout] --> src/config.rs:69:27 [INFO] [stdout] | [INFO] [stdout] 69 | pub fn all_directives(&self) -> DirectiveIter { [INFO] [stdout] | ^^^^^ ------------- the lifetime gets resolved as `'_` [INFO] [stdout] | | [INFO] [stdout] | this lifetime flows to the output [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(mismatched_lifetime_syntaxes)]` on by default [INFO] [stdout] help: one option is to remove the lifetime for references and use the anonymous lifetime for paths [INFO] [stdout] | [INFO] [stdout] 69 | pub fn all_directives(&self) -> DirectiveIter<'_> { [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/errors.rs:4:17 [INFO] [stdout] | [INFO] [stdout] 4 | #[derive(Debug, Fail)] [INFO] [stdout] | ^^^^ [INFO] [stdout] | | [INFO] [stdout] | `ReadError` is not local [INFO] [stdout] | `Fail` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_DERIVE_failure_Fail_FOR_ReadError` [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/errors.rs:4:17 [INFO] [stdout] | [INFO] [stdout] 4 | #[derive(Debug, Fail)] [INFO] [stdout] | ^^^^ [INFO] [stdout] | | [INFO] [stdout] | `ReadError` is not local [INFO] [stdout] | `Display` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_DERIVE_failure_core_fmt_Display_FOR_ReadError` [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/errors.rs:8:17 [INFO] [stdout] | [INFO] [stdout] 8 | #[derive(Debug, Fail)] [INFO] [stdout] | ^^^^ [INFO] [stdout] | | [INFO] [stdout] | `ReadEnum` is not local [INFO] [stdout] | `Fail` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_DERIVE_failure_Fail_FOR_ReadEnum` [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/errors.rs:8:17 [INFO] [stdout] | [INFO] [stdout] 8 | #[derive(Debug, Fail)] [INFO] [stdout] | ^^^^ [INFO] [stdout] | | [INFO] [stdout] | `ReadEnum` is not local [INFO] [stdout] | `Display` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_DERIVE_failure_core_fmt_Display_FOR_ReadEnum` [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/checks/proxy_pass.rs:8:10 [INFO] [stdout] | [INFO] [stdout] 8 | #[derive(Fail, Debug)] [INFO] [stdout] | ^^^^ [INFO] [stdout] | | [INFO] [stdout] | `Error` is not local [INFO] [stdout] | `Fail` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_DERIVE_failure_Fail_FOR_Error` [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/checks/proxy_pass.rs:8:10 [INFO] [stdout] | [INFO] [stdout] 8 | #[derive(Fail, Debug)] [INFO] [stdout] | ^^^^ [INFO] [stdout] | | [INFO] [stdout] | `Error` is not local [INFO] [stdout] | `Display` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_DERIVE_failure_core_fmt_Display_FOR_Error` [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: lifetime flowing from input to output with different syntax can be confusing [INFO] [stdout] --> src/config.rs:69:27 [INFO] [stdout] | [INFO] [stdout] 69 | pub fn all_directives(&self) -> DirectiveIter { [INFO] [stdout] | ^^^^^ ------------- the lifetime gets resolved as `'_` [INFO] [stdout] | | [INFO] [stdout] | this lifetime flows to the output [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(mismatched_lifetime_syntaxes)]` on by default [INFO] [stdout] help: one option is to remove the lifetime for references and use the anonymous lifetime for paths [INFO] [stdout] | [INFO] [stdout] 69 | pub fn all_directives(&self) -> DirectiveIter<'_> { [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/errors.rs:4:17 [INFO] [stdout] | [INFO] [stdout] 4 | #[derive(Debug, Fail)] [INFO] [stdout] | ^^^^ [INFO] [stdout] | | [INFO] [stdout] | `ReadError` is not local [INFO] [stdout] | `Fail` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_DERIVE_failure_Fail_FOR_ReadError` [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/errors.rs:4:17 [INFO] [stdout] | [INFO] [stdout] 4 | #[derive(Debug, Fail)] [INFO] [stdout] | ^^^^ [INFO] [stdout] | | [INFO] [stdout] | `ReadError` is not local [INFO] [stdout] | `Display` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_DERIVE_failure_core_fmt_Display_FOR_ReadError` [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/errors.rs:8:17 [INFO] [stdout] | [INFO] [stdout] 8 | #[derive(Debug, Fail)] [INFO] [stdout] | ^^^^ [INFO] [stdout] | | [INFO] [stdout] | `ReadEnum` is not local [INFO] [stdout] | `Fail` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_DERIVE_failure_Fail_FOR_ReadEnum` [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/errors.rs:8:17 [INFO] [stdout] | [INFO] [stdout] 8 | #[derive(Debug, Fail)] [INFO] [stdout] | ^^^^ [INFO] [stdout] | | [INFO] [stdout] | `ReadEnum` is not local [INFO] [stdout] | `Display` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_DERIVE_failure_core_fmt_Display_FOR_ReadEnum` [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/checks/proxy_pass.rs:8:10 [INFO] [stdout] | [INFO] [stdout] 8 | #[derive(Fail, Debug)] [INFO] [stdout] | ^^^^ [INFO] [stdout] | | [INFO] [stdout] | `Error` is not local [INFO] [stdout] | `Fail` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_DERIVE_failure_Fail_FOR_Error` [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/checks/proxy_pass.rs:8:10 [INFO] [stdout] | [INFO] [stdout] 8 | #[derive(Fail, Debug)] [INFO] [stdout] | ^^^^ [INFO] [stdout] | | [INFO] [stdout] | `Error` is not local [INFO] [stdout] | `Display` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_DERIVE_failure_core_fmt_Display_FOR_Error` [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: variable does not need to be mutable [INFO] [stdout] --> src/bin/nginx_config_mod/modify.rs:340:21 [INFO] [stdout] | [INFO] [stdout] 340 | let mut s = replacement.to_string(); [INFO] [stdout] | ----^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_mut)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/bin/nginx_config_mod/modify.rs:340:21 [INFO] [stdout] | [INFO] [stdout] 340 | let mut s = replacement.to_string(); [INFO] [stdout] | ----^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_mut)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stderr] Finished `dev` profile [unoptimized + debuginfo] target(s) in 22.22s [INFO] [stderr] warning: the following packages contain code that will be rejected by a future version of Rust: syn v0.14.8 [INFO] [stderr] note: to see what the problems were, use the option `--future-incompat-report`, or run `cargo report future-incompatibilities --id 1` [INFO] running `Command { std: "docker" "inspect" "bdd162f04948e03128df4151d6ec2fcd8d40e28af9323de09952fd436a2e57e5", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "bdd162f04948e03128df4151d6ec2fcd8d40e28af9323de09952fd436a2e57e5", kill_on_drop: false }` [INFO] [stdout] bdd162f04948e03128df4151d6ec2fcd8d40e28af9323de09952fd436a2e57e5