[INFO] cloning repository https://github.com/sgrif/cotli-helper
[INFO] running `Command { std: "git" "-c" "credential.helper=" "-c" "credential.helper=/workspace/cargo-home/bin/git-credential-null" "clone" "--bare" "https://github.com/sgrif/cotli-helper" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fsgrif%2Fcotli-helper", kill_on_drop: false }`
[INFO] [stderr] Cloning into bare repository '/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fsgrif%2Fcotli-helper'...
[INFO] running `Command { std: "git" "rev-parse" "HEAD", kill_on_drop: false }`
[INFO] [stdout] 510f8f14d73390921a4e22a24eab9fb0a64554ad
[INFO] checking sgrif/cotli-helper against master#b56aaec52bc0fa35591a872fb4aac81f606e265c for pr-122661-1
[INFO] running `Command { std: "git" "clone" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fsgrif%2Fcotli-helper" "/workspace/builds/worker-7-tc1/source", kill_on_drop: false }`
[INFO] [stderr] Cloning into '/workspace/builds/worker-7-tc1/source'...
[INFO] [stderr] done.
[INFO] started tweaking git repo https://github.com/sgrif/cotli-helper
[INFO] removed 0 missing tests
[INFO] finished tweaking git repo https://github.com/sgrif/cotli-helper
[INFO] tweaked toml for git repo https://github.com/sgrif/cotli-helper written to /workspace/builds/worker-7-tc1/source/Cargo.toml
[INFO] validating manifest of git repo https://github.com/sgrif/cotli-helper on toolchain b56aaec52bc0fa35591a872fb4aac81f606e265c
[INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+b56aaec52bc0fa35591a872fb4aac81f606e265c" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }`
[INFO] crate git repo https://github.com/sgrif/cotli-helper 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" "+b56aaec52bc0fa35591a872fb4aac81f606e265c" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }`
[INFO] [stderr] warning: An explicit [[test]] section is specified in Cargo.toml which currently
[INFO] [stderr] disables Cargo from automatically inferring other test targets.
[INFO] [stderr] This inference behavior will change in the Rust 2018 edition and the following
[INFO] [stderr] files will be included as a test target:
[INFO] [stderr] 
[INFO] [stderr] * tests/support.rs
[INFO] [stderr] 
[INFO] [stderr] This is likely to break cargo build or cargo test as these files may not be
[INFO] [stderr] ready to be compiled as a test target today. You can future-proof yourself
[INFO] [stderr] and disable this warning by adding `autotests = false` to your [package]
[INFO] [stderr] section. You may also move the files to a location where Cargo would not
[INFO] [stderr] automatically infer them to be a target, such as in subfolders.
[INFO] [stderr] 
[INFO] [stderr] For more information on this warning you can consult
[INFO] [stderr] https://github.com/rust-lang/cargo/issues/5330
[INFO] [stderr] warning: no edition set: defaulting to the 2015 edition while the latest is 2024
[INFO] [stderr]  Downloading crates ...
[INFO] [stderr]   Downloaded toml v0.4.1
[INFO] [stderr]   Downloaded itertools v0.6.0
[INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-7-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-7-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:cf8efcab8866f2cf4285301c7418e2f4f2a9b088c91ba69c40d5b659f81557f7" "/opt/rustwide/cargo-home/bin/cargo" "+b56aaec52bc0fa35591a872fb4aac81f606e265c" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }`
[INFO] [stdout] e9f6db22271e296ba4d47622c092e7d94f4b6d4ebf44da662bfeaf6749c8fbc4
[INFO] running `Command { std: "docker" "start" "-a" "e9f6db22271e296ba4d47622c092e7d94f4b6d4ebf44da662bfeaf6749c8fbc4", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "e9f6db22271e296ba4d47622c092e7d94f4b6d4ebf44da662bfeaf6749c8fbc4", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "e9f6db22271e296ba4d47622c092e7d94f4b6d4ebf44da662bfeaf6749c8fbc4", kill_on_drop: false }`
[INFO] [stdout] e9f6db22271e296ba4d47622c092e7d94f4b6d4ebf44da662bfeaf6749c8fbc4
[INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-7-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-7-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:cf8efcab8866f2cf4285301c7418e2f4f2a9b088c91ba69c40d5b659f81557f7" "/opt/rustwide/cargo-home/bin/cargo" "+b56aaec52bc0fa35591a872fb4aac81f606e265c" "check" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }`
[INFO] [stdout] 68d91b9c345831d0821a2c0d30f7308de6506bfdacc5572c3600ac5ca3c45b97
[INFO] running `Command { std: "docker" "start" "-a" "68d91b9c345831d0821a2c0d30f7308de6506bfdacc5572c3600ac5ca3c45b97", kill_on_drop: false }`
[INFO] [stderr] warning: An explicit [[test]] section is specified in Cargo.toml which currently
[INFO] [stderr] disables Cargo from automatically inferring other test targets.
[INFO] [stderr] This inference behavior will change in the Rust 2018 edition and the following
[INFO] [stderr] files will be included as a test target:
[INFO] [stderr] 
[INFO] [stderr] * tests/support.rs
[INFO] [stderr] 
[INFO] [stderr] This is likely to break cargo build or cargo test as these files may not be
[INFO] [stderr] ready to be compiled as a test target today. You can future-proof yourself
[INFO] [stderr] and disable this warning by adding `autotests = false` to your [package]
[INFO] [stderr] section. You may also move the files to a location where Cargo would not
[INFO] [stderr] automatically infer them to be a target, such as in subfolders.
[INFO] [stderr] 
[INFO] [stderr] For more information on this warning you can consult
[INFO] [stderr] https://github.com/rust-lang/cargo/issues/5330
[INFO] [stderr] warning: no edition set: defaulting to the 2015 edition while the latest is 2024
[INFO] [stderr]    Compiling unicode-xid v0.0.4
[INFO] [stderr]     Checking libc v0.2.22
[INFO] [stderr]    Compiling quote v0.3.15
[INFO] [stderr]     Checking strsim v0.6.0
[INFO] [stderr]     Checking unicode-width v0.1.4
[INFO] [stderr]     Checking vec_map v0.8.0
[INFO] [stderr]     Checking serde v1.0.8
[INFO] [stderr]     Checking bitflags v0.8.2
[INFO] [stderr]     Checking ansi_term v0.9.0
[INFO] [stderr]     Checking either v1.1.0
[INFO] [stderr]     Checking unicode-segmentation v1.2.0
[INFO] [stderr]    Compiling synom v0.11.3
[INFO] [stderr]     Checking byteorder v1.0.0
[INFO] [stderr]     Checking ordermap v0.2.10
[INFO] [stderr]     Checking itertools v0.6.0
[INFO] [stderr]    Compiling syn v0.11.11
[INFO] [stderr]     Checking term_size v0.3.0
[INFO] [stderr]     Checking atty v0.2.2
[INFO] [stderr]     Checking rand v0.3.15
[INFO] [stderr]     Checking clap v2.24.2
[INFO] [stderr]    Compiling serde_derive_internals v0.15.1
[INFO] [stderr]    Compiling serde_derive v1.0.8
[INFO] [stderr]     Checking toml v0.4.1
[INFO] [stderr]     Checking cotli-helper v0.1.0 (/opt/rustwide/workdir)
[INFO] [stdout] warning: use of deprecated macro `try`: use the `?` operator instead
[INFO] [stdout]     --> src/crusader.rs:2371:1
[INFO] [stdout]      |
[INFO] [stdout] 2371 | / bitflags! {
[INFO] [stdout] 2372 | |     pub flags Slot: u32 {
[INFO] [stdout] 2373 | |         const SLOT_1   = 1,
[INFO] [stdout] 2374 | |         const SLOT_2   = 1 << 1,
[INFO] [stdout] ...    |
[INFO] [stdout] 2397 | | }
[INFO] [stdout]      | |_^
[INFO] [stdout]      |
[INFO] [stdout]      = note: `#[warn(deprecated)]` on by default
[INFO] [stdout]      = note: this warning originates in the macro `bitflags` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of deprecated macro `try`: use the `?` operator instead
[INFO] [stdout]     --> src/crusader.rs:2371:1
[INFO] [stdout]      |
[INFO] [stdout] 2371 | / bitflags! {
[INFO] [stdout] 2372 | |     pub flags Slot: u32 {
[INFO] [stdout] 2373 | |         const SLOT_1   = 1,
[INFO] [stdout] 2374 | |         const SLOT_2   = 1 << 1,
[INFO] [stdout] ...    |
[INFO] [stdout] 2397 | | }
[INFO] [stdout]      | |_^
[INFO] [stdout]      |
[INFO] [stdout]      = note: this warning originates in the macro `bitflags` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of deprecated macro `try`: use the `?` operator instead
[INFO] [stdout]     --> src/crusader.rs:2399:1
[INFO] [stdout]      |
[INFO] [stdout] 2399 | / bitflags! {
[INFO] [stdout] 2400 | |     pub flags Tags: u32 {
[INFO] [stdout] 2401 | |         const MALE            = 0b000000000000000000000000000000001,
[INFO] [stdout] 2402 | |         const FEMALE          = 0b000000000000000000000000000000010,
[INFO] [stdout] ...    |
[INFO] [stdout] 2425 | | }
[INFO] [stdout]      | |_^
[INFO] [stdout]      |
[INFO] [stdout]      = note: this warning originates in the macro `bitflags` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of deprecated macro `try`: use the `?` operator instead
[INFO] [stdout]     --> src/crusader.rs:2399:1
[INFO] [stdout]      |
[INFO] [stdout] 2399 | / bitflags! {
[INFO] [stdout] 2400 | |     pub flags Tags: u32 {
[INFO] [stdout] 2401 | |         const MALE            = 0b000000000000000000000000000000001,
[INFO] [stdout] 2402 | |         const FEMALE          = 0b000000000000000000000000000000010,
[INFO] [stdout] ...    |
[INFO] [stdout] 2425 | | }
[INFO] [stdout]      | |_^
[INFO] [stdout]      |
[INFO] [stdout]      = note: this warning originates in the macro `bitflags` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unnecessary parentheses around block return value
[INFO] [stdout]   --> src/formation/coordinate.rs:32:17
[INFO] [stdout]    |
[INFO] [stdout] 32 |                 (coord.y == self.y || coord.y == self.y + 1)
[INFO] [stdout]    |                 ^                                          ^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(unused_parens)]` on by default
[INFO] [stdout] help: remove these parentheses
[INFO] [stdout]    |
[INFO] [stdout] 32 -                 (coord.y == self.y || coord.y == self.y + 1)
[INFO] [stdout] 32 +                 coord.y == self.y || coord.y == self.y + 1
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unnecessary parentheses around block return value
[INFO] [stdout]   --> src/formation/coordinate.rs:34:17
[INFO] [stdout]    |
[INFO] [stdout] 34 |                 (coord.y == self.y.saturating_sub(1) || coord.y == self.y)
[INFO] [stdout]    |                 ^                                                        ^
[INFO] [stdout]    |
[INFO] [stdout] help: remove these parentheses
[INFO] [stdout]    |
[INFO] [stdout] 34 -                 (coord.y == self.y.saturating_sub(1) || coord.y == self.y)
[INFO] [stdout] 34 +                 coord.y == self.y.saturating_sub(1) || coord.y == self.y
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of deprecated macro `try`: use the `?` operator instead
[INFO] [stdout]     --> src/crusader.rs:2371:1
[INFO] [stdout]      |
[INFO] [stdout] 2371 | / bitflags! {
[INFO] [stdout] 2372 | |     pub flags Slot: u32 {
[INFO] [stdout] 2373 | |         const SLOT_1   = 1,
[INFO] [stdout] 2374 | |         const SLOT_2   = 1 << 1,
[INFO] [stdout] ...    |
[INFO] [stdout] 2397 | | }
[INFO] [stdout]      | |_^
[INFO] [stdout]      |
[INFO] [stdout]      = note: `#[warn(deprecated)]` on by default
[INFO] [stdout]      = note: this warning originates in the macro `bitflags` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of deprecated macro `try`: use the `?` operator instead
[INFO] [stdout]     --> src/crusader.rs:2371:1
[INFO] [stdout]      |
[INFO] [stdout] 2371 | / bitflags! {
[INFO] [stdout] 2372 | |     pub flags Slot: u32 {
[INFO] [stdout] 2373 | |         const SLOT_1   = 1,
[INFO] [stdout] 2374 | |         const SLOT_2   = 1 << 1,
[INFO] [stdout] ...    |
[INFO] [stdout] 2397 | | }
[INFO] [stdout]      | |_^
[INFO] [stdout]      |
[INFO] [stdout]      = note: this warning originates in the macro `bitflags` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of deprecated macro `try`: use the `?` operator instead
[INFO] [stdout]     --> src/crusader.rs:2399:1
[INFO] [stdout]      |
[INFO] [stdout] 2399 | / bitflags! {
[INFO] [stdout] 2400 | |     pub flags Tags: u32 {
[INFO] [stdout] 2401 | |         const MALE            = 0b000000000000000000000000000000001,
[INFO] [stdout] 2402 | |         const FEMALE          = 0b000000000000000000000000000000010,
[INFO] [stdout] ...    |
[INFO] [stdout] 2425 | | }
[INFO] [stdout]      | |_^
[INFO] [stdout]      |
[INFO] [stdout]      = note: this warning originates in the macro `bitflags` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of deprecated macro `try`: use the `?` operator instead
[INFO] [stdout]     --> src/crusader.rs:2399:1
[INFO] [stdout]      |
[INFO] [stdout] 2399 | / bitflags! {
[INFO] [stdout] 2400 | |     pub flags Tags: u32 {
[INFO] [stdout] 2401 | |         const MALE            = 0b000000000000000000000000000000001,
[INFO] [stdout] 2402 | |         const FEMALE          = 0b000000000000000000000000000000010,
[INFO] [stdout] ...    |
[INFO] [stdout] 2425 | | }
[INFO] [stdout]      | |_^
[INFO] [stdout]      |
[INFO] [stdout]      = note: this warning originates in the macro `bitflags` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unnecessary parentheses around block return value
[INFO] [stdout]   --> src/formation/coordinate.rs:32:17
[INFO] [stdout]    |
[INFO] [stdout] 32 |                 (coord.y == self.y || coord.y == self.y + 1)
[INFO] [stdout]    |                 ^                                          ^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(unused_parens)]` on by default
[INFO] [stdout] help: remove these parentheses
[INFO] [stdout]    |
[INFO] [stdout] 32 -                 (coord.y == self.y || coord.y == self.y + 1)
[INFO] [stdout] 32 +                 coord.y == self.y || coord.y == self.y + 1
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unnecessary parentheses around block return value
[INFO] [stdout]   --> src/formation/coordinate.rs:34:17
[INFO] [stdout]    |
[INFO] [stdout] 34 |                 (coord.y == self.y.saturating_sub(1) || coord.y == self.y)
[INFO] [stdout]    |                 ^                                                        ^
[INFO] [stdout]    |
[INFO] [stdout] help: remove these parentheses
[INFO] [stdout]    |
[INFO] [stdout] 34 -                 (coord.y == self.y.saturating_sub(1) || coord.y == self.y)
[INFO] [stdout] 34 +                 coord.y == self.y.saturating_sub(1) || coord.y == self.y
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: the feature `conservative_impl_trait` has been stable since 1.26.0 and no longer requires an attribute to enable
[INFO] [stdout]  --> src/lib.rs:2:12
[INFO] [stdout]   |
[INFO] [stdout] 2 | #![feature(conservative_impl_trait)]
[INFO] [stdout]   |            ^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]   |
[INFO] [stdout]   = note: `#[warn(stable_features)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: the feature `conservative_impl_trait` has been stable since 1.26.0 and no longer requires an attribute to enable
[INFO] [stdout]  --> src/lib.rs:2:12
[INFO] [stdout]   |
[INFO] [stdout] 2 | #![feature(conservative_impl_trait)]
[INFO] [stdout]   |            ^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]   |
[INFO] [stdout]   = note: `#[warn(stable_features)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: panic message contains an unused formatting placeholder
[INFO] [stdout]   --> src/cli.rs:74:47
[INFO] [stdout]    |
[INFO] [stdout] 74 |                     panic!("Invalid placement {}. Placements must be in the \
[INFO] [stdout]    |                                               ^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: this message is not used as a format string when given without arguments, but will be in Rust 2021
[INFO] [stdout]    = note: `#[warn(non_fmt_panics)]` on by default
[INFO] [stdout] help: add the missing argument
[INFO] [stdout]    |
[INFO] [stdout] 76 |                         '4 Artaxes the Lion'`", ...)
[INFO] [stdout]    |                                               +++++
[INFO] [stdout] help: or add a "{}" format string to use the message literally
[INFO] [stdout]    |
[INFO] [stdout] 74 |                     panic!("{}", "Invalid placement {}. Placements must be in the \
[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/crusader.rs:10:68
[INFO] [stdout]    |
[INFO] [stdout] 10 | #[derive(Hash, Debug, Clone, Copy, PartialEq, Eq, PartialOrd, Ord, Deserialize, Serialize)]
[INFO] [stdout]    |                                                                    -^^^^^^^^^^
[INFO] [stdout]    |                                                                    |
[INFO] [stdout]    |                                                                    `CrusaderName` is not local
[INFO] [stdout]    |                                                                    `Deserialize` is not local
[INFO] [stdout]    |                                                                    move the `impl` block outside of this constant `_IMPL_DESERIALIZE_FOR_CrusaderName`
[INFO] [stdout]    |
[INFO] [stdout]    = note: the derive macro `Deserialize` defines the non-local `impl`, and may need to be changed
[INFO] [stdout]    = note: the derive macro `Deserialize` may come from an old version of the `serde_derive` crate, try updating your dependency with `cargo update -p serde_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 `Deserialize` (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/crusader.rs:10:81
[INFO] [stdout]    |
[INFO] [stdout] 10 | #[derive(Hash, Debug, Clone, Copy, PartialEq, Eq, PartialOrd, Ord, Deserialize, Serialize)]
[INFO] [stdout]    |                                                                                 ^^^^^^^^^
[INFO] [stdout]    |                                                                                 |
[INFO] [stdout]    |                                                                                 `CrusaderName` is not local
[INFO] [stdout]    |                                                                                 `Serialize` is not local
[INFO] [stdout]    |                                                                                 move the `impl` block outside of this constant `_IMPL_SERIALIZE_FOR_CrusaderName`
[INFO] [stdout]    |
[INFO] [stdout]    = note: the derive macro `Serialize` defines the non-local `impl`, and may need to be changed
[INFO] [stdout]    = note: the derive macro `Serialize` may come from an old version of the `serde_derive` crate, try updating your dependency with `cargo update -p serde_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 `Serialize` (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/dps.rs:9:10
[INFO] [stdout]   |
[INFO] [stdout] 9 | #[derive(Deserialize)]
[INFO] [stdout]   |          -^^^^^^^^^^
[INFO] [stdout]   |          |
[INFO] [stdout]   |          `Level` is not local
[INFO] [stdout]   |          `Deserialize` is not local
[INFO] [stdout]   |          move the `impl` block outside of this constant `_IMPL_DESERIALIZE_FOR_Level`
[INFO] [stdout]   |
[INFO] [stdout]   = note: the derive macro `Deserialize` defines the non-local `impl`, and may need to be changed
[INFO] [stdout]   = note: the derive macro `Deserialize` may come from an old version of the `serde_derive` crate, try updating your dependency with `cargo update -p serde_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 `Deserialize` (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/user_data/from_toml.rs:26:10
[INFO] [stdout]    |
[INFO] [stdout] 26 | #[derive(Deserialize)]
[INFO] [stdout]    |          -^^^^^^^^^^
[INFO] [stdout]    |          |
[INFO] [stdout]    |          `UserDataToml` is not local
[INFO] [stdout]    |          `Deserialize` is not local
[INFO] [stdout]    |          move the `impl` block outside of this constant `_IMPL_DESERIALIZE_FOR_UserDataToml`
[INFO] [stdout]    |
[INFO] [stdout]    = note: the derive macro `Deserialize` defines the non-local `impl`, and may need to be changed
[INFO] [stdout]    = note: the derive macro `Deserialize` may come from an old version of the `serde_derive` crate, try updating your dependency with `cargo update -p serde_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 `Deserialize` (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/user_data/from_toml.rs:43:10
[INFO] [stdout]    |
[INFO] [stdout] 43 | #[derive(Deserialize)]
[INFO] [stdout]    |          -^^^^^^^^^^
[INFO] [stdout]    |          |
[INFO] [stdout]    |          `TalentFromToml` is not local
[INFO] [stdout]    |          `Deserialize` is not local
[INFO] [stdout]    |          move the `impl` block outside of this constant `_IMPL_DESERIALIZE_FOR_TalentFromToml`
[INFO] [stdout]    |
[INFO] [stdout]    = note: the derive macro `Deserialize` defines the non-local `impl`, and may need to be changed
[INFO] [stdout]    = note: the derive macro `Deserialize` may come from an old version of the `serde_derive` crate, try updating your dependency with `cargo update -p serde_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 `Deserialize` (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/user_data/from_toml.rs:106:10
[INFO] [stdout]     |
[INFO] [stdout] 106 | #[derive(Deserialize)]
[INFO] [stdout]     |          -^^^^^^^^^^
[INFO] [stdout]     |          |
[INFO] [stdout]     |          `CrusaderFromToml` is not local
[INFO] [stdout]     |          `Deserialize` is not local
[INFO] [stdout]     |          move the `impl` block outside of this constant `_IMPL_DESERIALIZE_FOR_CrusaderFromToml`
[INFO] [stdout]     |
[INFO] [stdout]     = note: the derive macro `Deserialize` defines the non-local `impl`, and may need to be changed
[INFO] [stdout]     = note: the derive macro `Deserialize` may come from an old version of the `serde_derive` crate, try updating your dependency with `cargo update -p serde_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 `Deserialize` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: panic message contains an unused formatting placeholder
[INFO] [stdout]   --> src/cli.rs:74:47
[INFO] [stdout]    |
[INFO] [stdout] 74 |                     panic!("Invalid placement {}. Placements must be in the \
[INFO] [stdout]    |                                               ^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: this message is not used as a format string when given without arguments, but will be in Rust 2021
[INFO] [stdout]    = note: `#[warn(non_fmt_panics)]` on by default
[INFO] [stdout] help: add the missing argument
[INFO] [stdout]    |
[INFO] [stdout] 76 |                         '4 Artaxes the Lion'`", ...)
[INFO] [stdout]    |                                               +++++
[INFO] [stdout] help: or add a "{}" format string to use the message literally
[INFO] [stdout]    |
[INFO] [stdout] 74 |                     panic!("{}", "Invalid placement {}. Placements must be in the \
[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/crusader.rs:10:68
[INFO] [stdout]    |
[INFO] [stdout] 10 | #[derive(Hash, Debug, Clone, Copy, PartialEq, Eq, PartialOrd, Ord, Deserialize, Serialize)]
[INFO] [stdout]    |                                                                    -^^^^^^^^^^
[INFO] [stdout]    |                                                                    |
[INFO] [stdout]    |                                                                    `CrusaderName` is not local
[INFO] [stdout]    |                                                                    `Deserialize` is not local
[INFO] [stdout]    |                                                                    move the `impl` block outside of this constant `_IMPL_DESERIALIZE_FOR_CrusaderName`
[INFO] [stdout]    |
[INFO] [stdout]    = note: the derive macro `Deserialize` defines the non-local `impl`, and may need to be changed
[INFO] [stdout]    = note: the derive macro `Deserialize` may come from an old version of the `serde_derive` crate, try updating your dependency with `cargo update -p serde_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 `Deserialize` (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/crusader.rs:10:81
[INFO] [stdout]    |
[INFO] [stdout] 10 | #[derive(Hash, Debug, Clone, Copy, PartialEq, Eq, PartialOrd, Ord, Deserialize, Serialize)]
[INFO] [stdout]    |                                                                                 ^^^^^^^^^
[INFO] [stdout]    |                                                                                 |
[INFO] [stdout]    |                                                                                 `CrusaderName` is not local
[INFO] [stdout]    |                                                                                 `Serialize` is not local
[INFO] [stdout]    |                                                                                 move the `impl` block outside of this constant `_IMPL_SERIALIZE_FOR_CrusaderName`
[INFO] [stdout]    |
[INFO] [stdout]    = note: the derive macro `Serialize` defines the non-local `impl`, and may need to be changed
[INFO] [stdout]    = note: the derive macro `Serialize` may come from an old version of the `serde_derive` crate, try updating your dependency with `cargo update -p serde_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 `Serialize` (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/dps.rs:9:10
[INFO] [stdout]   |
[INFO] [stdout] 9 | #[derive(Deserialize)]
[INFO] [stdout]   |          -^^^^^^^^^^
[INFO] [stdout]   |          |
[INFO] [stdout]   |          `Level` is not local
[INFO] [stdout]   |          `Deserialize` is not local
[INFO] [stdout]   |          move the `impl` block outside of this constant `_IMPL_DESERIALIZE_FOR_Level`
[INFO] [stdout]   |
[INFO] [stdout]   = note: the derive macro `Deserialize` defines the non-local `impl`, and may need to be changed
[INFO] [stdout]   = note: the derive macro `Deserialize` may come from an old version of the `serde_derive` crate, try updating your dependency with `cargo update -p serde_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 `Deserialize` (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/formation_search/mod.rs:367:28
[INFO] [stdout]     |
[INFO] [stdout] 367 | fn search_state(crusaders: &[Crusader]) -> State {
[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] 367 | fn search_state(crusaders: &[Crusader]) -> State<'_> {
[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/user_data/from_toml.rs:26:10
[INFO] [stdout]    |
[INFO] [stdout] 26 | #[derive(Deserialize)]
[INFO] [stdout]    |          -^^^^^^^^^^
[INFO] [stdout]    |          |
[INFO] [stdout]    |          `UserDataToml` is not local
[INFO] [stdout]    |          `Deserialize` is not local
[INFO] [stdout]    |          move the `impl` block outside of this constant `_IMPL_DESERIALIZE_FOR_UserDataToml`
[INFO] [stdout]    |
[INFO] [stdout]    = note: the derive macro `Deserialize` defines the non-local `impl`, and may need to be changed
[INFO] [stdout]    = note: the derive macro `Deserialize` may come from an old version of the `serde_derive` crate, try updating your dependency with `cargo update -p serde_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 `Deserialize` (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/user_data/from_toml.rs:43:10
[INFO] [stdout]    |
[INFO] [stdout] 43 | #[derive(Deserialize)]
[INFO] [stdout]    |          -^^^^^^^^^^
[INFO] [stdout]    |          |
[INFO] [stdout]    |          `TalentFromToml` is not local
[INFO] [stdout]    |          `Deserialize` is not local
[INFO] [stdout]    |          move the `impl` block outside of this constant `_IMPL_DESERIALIZE_FOR_TalentFromToml`
[INFO] [stdout]    |
[INFO] [stdout]    = note: the derive macro `Deserialize` defines the non-local `impl`, and may need to be changed
[INFO] [stdout]    = note: the derive macro `Deserialize` may come from an old version of the `serde_derive` crate, try updating your dependency with `cargo update -p serde_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 `Deserialize` (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/user_data/from_toml.rs:106:10
[INFO] [stdout]     |
[INFO] [stdout] 106 | #[derive(Deserialize)]
[INFO] [stdout]     |          -^^^^^^^^^^
[INFO] [stdout]     |          |
[INFO] [stdout]     |          `CrusaderFromToml` is not local
[INFO] [stdout]     |          `Deserialize` is not local
[INFO] [stdout]     |          move the `impl` block outside of this constant `_IMPL_DESERIALIZE_FOR_CrusaderFromToml`
[INFO] [stdout]     |
[INFO] [stdout]     = note: the derive macro `Deserialize` defines the non-local `impl`, and may need to be changed
[INFO] [stdout]     = note: the derive macro `Deserialize` may come from an old version of the `serde_derive` crate, try updating your dependency with `cargo update -p serde_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 `Deserialize` (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 8.44s
[INFO] [stderr] warning: the following packages contain code that will be rejected by a future version of Rust: bitflags v0.8.2
[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" "68d91b9c345831d0821a2c0d30f7308de6506bfdacc5572c3600ac5ca3c45b97", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "68d91b9c345831d0821a2c0d30f7308de6506bfdacc5572c3600ac5ca3c45b97", kill_on_drop: false }`
[INFO] [stdout] 68d91b9c345831d0821a2c0d30f7308de6506bfdacc5572c3600ac5ca3c45b97