[INFO] cloning repository https://github.com/jpastuszek/microRTS-rs [INFO] running `Command { std: "git" "-c" "credential.helper=" "-c" "credential.helper=/workspace/cargo-home/bin/git-credential-null" "clone" "--bare" "https://github.com/jpastuszek/microRTS-rs" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fjpastuszek%2FmicroRTS-rs", kill_on_drop: false }` [INFO] [stderr] Cloning into bare repository '/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fjpastuszek%2FmicroRTS-rs'... [INFO] running `Command { std: "git" "rev-parse" "HEAD", kill_on_drop: false }` [INFO] [stdout] efa17147b015deb3db61c22ad6fecbd5511f22cc [INFO] checking jpastuszek/microRTS-rs against master#2e6fc425411280c658d852813de01c5157255088 for pr-128013 [INFO] running `Command { std: "git" "clone" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fjpastuszek%2FmicroRTS-rs" "/workspace/builds/worker-3-tc1/source", kill_on_drop: false }` [INFO] [stderr] Cloning into '/workspace/builds/worker-3-tc1/source'... [INFO] [stderr] done. [INFO] validating manifest of git repo https://github.com/jpastuszek/microRTS-rs on toolchain 2e6fc425411280c658d852813de01c5157255088 [INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+2e6fc425411280c658d852813de01c5157255088" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }` [INFO] started tweaking git repo https://github.com/jpastuszek/microRTS-rs [INFO] finished tweaking git repo https://github.com/jpastuszek/microRTS-rs [INFO] tweaked toml for git repo https://github.com/jpastuszek/microRTS-rs written to /workspace/builds/worker-3-tc1/source/Cargo.toml [INFO] crate git repo https://github.com/jpastuszek/microRTS-rs 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" "+2e6fc425411280c658d852813de01c5157255088" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }` [INFO] [stderr] warning: path `src/micro_rts.rs` was erroneously implicitly accepted for library `micro_rts`, [INFO] [stderr] please rename the file to `src/lib.rs` or set lib.path in Cargo.toml [INFO] [stderr] warning: no edition set: defaulting to the 2015 edition while the latest is 2021 [INFO] [stderr] Updating crates.io index [INFO] [stderr] Downloading crates ... [INFO] [stderr] Downloaded num-traits v0.1.40 [INFO] [stderr] Downloaded itertools v0.6.0 [INFO] [stderr] Downloaded pathfinding v0.1.13 [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-3-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-3-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:dff56e7819e73ed36160586b3445e93eb0be776c16704aeeded9c3fb668b2384" "/opt/rustwide/cargo-home/bin/cargo" "+2e6fc425411280c658d852813de01c5157255088" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }` [INFO] [stdout] 897e5d2a46fb45deb117aaac37f2cccf14f6a89ede9bad227fe54bc30bcf6dda [INFO] running `Command { std: "docker" "start" "-a" "897e5d2a46fb45deb117aaac37f2cccf14f6a89ede9bad227fe54bc30bcf6dda", kill_on_drop: false }` [INFO] running `Command { std: "docker" "inspect" "897e5d2a46fb45deb117aaac37f2cccf14f6a89ede9bad227fe54bc30bcf6dda", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "897e5d2a46fb45deb117aaac37f2cccf14f6a89ede9bad227fe54bc30bcf6dda", kill_on_drop: false }` [INFO] [stdout] 897e5d2a46fb45deb117aaac37f2cccf14f6a89ede9bad227fe54bc30bcf6dda [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-3-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-3-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:dff56e7819e73ed36160586b3445e93eb0be776c16704aeeded9c3fb668b2384" "/opt/rustwide/cargo-home/bin/cargo" "+2e6fc425411280c658d852813de01c5157255088" "check" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }` [INFO] [stdout] 6364cd5eb7a8ebfa03b15ede1d723f13a67765b7da0322f89ce6aaba43b38ccf [INFO] running `Command { std: "docker" "start" "-a" "6364cd5eb7a8ebfa03b15ede1d723f13a67765b7da0322f89ce6aaba43b38ccf", kill_on_drop: false }` [INFO] [stderr] warning: path `src/micro_rts.rs` was erroneously implicitly accepted for library `micro_rts`, [INFO] [stderr] please rename the file to `src/lib.rs` or set lib.path in Cargo.toml [INFO] [stderr] warning: no edition set: defaulting to the 2015 edition while the latest is 2021 [INFO] [stderr] Checking num-traits v0.1.40 [INFO] [stderr] Checking either v1.1.0 [INFO] [stderr] Checking ansi_term v0.9.0 [INFO] [stderr] Checking itertools v0.6.0 [INFO] [stderr] Checking pathfinding v0.1.13 [INFO] [stderr] Checking micro_rts_rs v0.1.0 (/opt/rustwide/workdir) [INFO] [stdout] warning: fields `name` and `round` are never read [INFO] [stdout] --> src/game/game_state.rs:14:5 [INFO] [stdout] | [INFO] [stdout] 13 | pub struct GameState<'p, 't> { [INFO] [stdout] | --------- fields in this struct [INFO] [stdout] 14 | name: String, [INFO] [stdout] | ^^^^ [INFO] [stdout] 15 | round: u32, [INFO] [stdout] | ^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `GameState` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis [INFO] [stdout] = note: `#[warn(dead_code)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: fields `0`, `1`, and `2` are never read [INFO] [stdout] --> src/game/game_state.rs:25:17 [INFO] [stdout] | [INFO] [stdout] 25 | InvalidMove(EntityID, Direction, InvalidMove<'t>), [INFO] [stdout] | ----------- ^^^^^^^^ ^^^^^^^^^ ^^^^^^^^^^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | fields in this variant [INFO] [stdout] | [INFO] [stdout] = note: `GameRuleViolation` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis [INFO] [stdout] help: consider changing the fields to be of unit type to suppress this warning while preserving the field numbering, or remove the fields [INFO] [stdout] | [INFO] [stdout] 25 | InvalidMove((), (), ()), [INFO] [stdout] | ~~ ~~ ~~ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: fields `0` and `1` are never read [INFO] [stdout] --> src/game/game_state.rs:26:20 [INFO] [stdout] | [INFO] [stdout] 26 | EntityNotOwned(EntityID, &'p Player), [INFO] [stdout] | -------------- ^^^^^^^^ ^^^^^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | fields in this variant [INFO] [stdout] | [INFO] [stdout] = note: `GameRuleViolation` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis [INFO] [stdout] help: consider changing the fields to be of unit type to suppress this warning while preserving the field numbering, or remove the fields [INFO] [stdout] | [INFO] [stdout] 26 | EntityNotOwned((), ()), [INFO] [stdout] | ~~ ~~ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: field `0` is never read [INFO] [stdout] --> src/game/game_state.rs:27:24 [INFO] [stdout] | [INFO] [stdout] 27 | EntityDoesNotExist(EntityID), [INFO] [stdout] | ------------------ ^^^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | field in this variant [INFO] [stdout] | [INFO] [stdout] = note: `GameRuleViolation` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis [INFO] [stdout] help: consider changing the field to be of unit type to suppress this warning while preserving the field numbering, or remove the field [INFO] [stdout] | [INFO] [stdout] 27 | EntityDoesNotExist(()), [INFO] [stdout] | ~~ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: field `0` is never read [INFO] [stdout] --> src/game/game_state.rs:32:17 [INFO] [stdout] | [INFO] [stdout] 32 | NotWalkable(Location<'t>), [INFO] [stdout] | ----------- ^^^^^^^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | field in this variant [INFO] [stdout] | [INFO] [stdout] = note: `InvalidMove` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis [INFO] [stdout] help: consider changing the field to be of unit type to suppress this warning while preserving the field numbering, or remove the field [INFO] [stdout] | [INFO] [stdout] 32 | NotWalkable(()), [INFO] [stdout] | ~~ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: fields `0` and `1` are never read [INFO] [stdout] --> src/game/game_state.rs:33:26 [INFO] [stdout] | [INFO] [stdout] 33 | LocationAlreadyTaken(Location<'t>, EntityID), [INFO] [stdout] | -------------------- ^^^^^^^^^^^^ ^^^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | fields in this variant [INFO] [stdout] | [INFO] [stdout] = note: `InvalidMove` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis [INFO] [stdout] help: consider changing the fields to be of unit type to suppress this warning while preserving the field numbering, or remove the fields [INFO] [stdout] | [INFO] [stdout] 33 | LocationAlreadyTaken((), ()), [INFO] [stdout] | ~~ ~~ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: call to `.clone()` on a reference in this situation does nothing [INFO] [stdout] --> src/game/game_state.rs:164:34 [INFO] [stdout] | [INFO] [stdout] 164 | terrain: self.terrain.clone(), [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: the type `Terrain` does not implement `Clone`, so calling `clone` on `&Terrain` copies the reference, which does not do anything and can be removed [INFO] [stdout] = note: `#[warn(noop_method_call)]` on by default [INFO] [stdout] help: remove this redundant call [INFO] [stdout] | [INFO] [stdout] 164 - terrain: self.terrain.clone(), [INFO] [stdout] 164 + terrain: self.terrain, [INFO] [stdout] | [INFO] [stdout] help: if you meant to clone `Terrain`, implement `Clone` for it [INFO] [stdout] --> src/game/terrain.rs:158:1 [INFO] [stdout] | [INFO] [stdout] 158 + #[derive(Clone)] [INFO] [stdout] 159 | pub struct Terrain { [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: fields `name` and `round` are never read [INFO] [stdout] --> src/game/game_state.rs:14:5 [INFO] [stdout] | [INFO] [stdout] 13 | pub struct GameState<'p, 't> { [INFO] [stdout] | --------- fields in this struct [INFO] [stdout] 14 | name: String, [INFO] [stdout] | ^^^^ [INFO] [stdout] 15 | round: u32, [INFO] [stdout] | ^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `GameState` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis [INFO] [stdout] = note: `#[warn(dead_code)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: fields `0`, `1`, and `2` are never read [INFO] [stdout] --> src/game/game_state.rs:25:17 [INFO] [stdout] | [INFO] [stdout] 25 | InvalidMove(EntityID, Direction, InvalidMove<'t>), [INFO] [stdout] | ----------- ^^^^^^^^ ^^^^^^^^^ ^^^^^^^^^^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | fields in this variant [INFO] [stdout] | [INFO] [stdout] = note: `GameRuleViolation` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis [INFO] [stdout] help: consider changing the fields to be of unit type to suppress this warning while preserving the field numbering, or remove the fields [INFO] [stdout] | [INFO] [stdout] 25 | InvalidMove((), (), ()), [INFO] [stdout] | ~~ ~~ ~~ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: fields `0` and `1` are never read [INFO] [stdout] --> src/game/game_state.rs:26:20 [INFO] [stdout] | [INFO] [stdout] 26 | EntityNotOwned(EntityID, &'p Player), [INFO] [stdout] | -------------- ^^^^^^^^ ^^^^^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | fields in this variant [INFO] [stdout] | [INFO] [stdout] = note: `GameRuleViolation` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis [INFO] [stdout] help: consider changing the fields to be of unit type to suppress this warning while preserving the field numbering, or remove the fields [INFO] [stdout] | [INFO] [stdout] 26 | EntityNotOwned((), ()), [INFO] [stdout] | ~~ ~~ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: field `0` is never read [INFO] [stdout] --> src/game/game_state.rs:27:24 [INFO] [stdout] | [INFO] [stdout] 27 | EntityDoesNotExist(EntityID), [INFO] [stdout] | ------------------ ^^^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | field in this variant [INFO] [stdout] | [INFO] [stdout] = note: `GameRuleViolation` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis [INFO] [stdout] help: consider changing the field to be of unit type to suppress this warning while preserving the field numbering, or remove the field [INFO] [stdout] | [INFO] [stdout] 27 | EntityDoesNotExist(()), [INFO] [stdout] | ~~ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: field `0` is never read [INFO] [stdout] --> src/game/game_state.rs:32:17 [INFO] [stdout] | [INFO] [stdout] 32 | NotWalkable(Location<'t>), [INFO] [stdout] | ----------- ^^^^^^^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | field in this variant [INFO] [stdout] | [INFO] [stdout] = note: `InvalidMove` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis [INFO] [stdout] help: consider changing the field to be of unit type to suppress this warning while preserving the field numbering, or remove the field [INFO] [stdout] | [INFO] [stdout] 32 | NotWalkable(()), [INFO] [stdout] | ~~ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: fields `0` and `1` are never read [INFO] [stdout] --> src/game/game_state.rs:33:26 [INFO] [stdout] | [INFO] [stdout] 33 | LocationAlreadyTaken(Location<'t>, EntityID), [INFO] [stdout] | -------------------- ^^^^^^^^^^^^ ^^^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | fields in this variant [INFO] [stdout] | [INFO] [stdout] = note: `InvalidMove` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis [INFO] [stdout] help: consider changing the fields to be of unit type to suppress this warning while preserving the field numbering, or remove the fields [INFO] [stdout] | [INFO] [stdout] 33 | LocationAlreadyTaken((), ()), [INFO] [stdout] | ~~ ~~ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: 7 warnings emitted [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: call to `.clone()` on a reference in this situation does nothing [INFO] [stdout] --> src/game/game_state.rs:164:34 [INFO] [stdout] | [INFO] [stdout] 164 | terrain: self.terrain.clone(), [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: the type `Terrain` does not implement `Clone`, so calling `clone` on `&Terrain` copies the reference, which does not do anything and can be removed [INFO] [stdout] = note: `#[warn(noop_method_call)]` on by default [INFO] [stdout] help: remove this redundant call [INFO] [stdout] | [INFO] [stdout] 164 - terrain: self.terrain.clone(), [INFO] [stdout] 164 + terrain: self.terrain, [INFO] [stdout] | [INFO] [stdout] help: if you meant to clone `Terrain`, implement `Clone` for it [INFO] [stdout] --> src/game/terrain.rs:158:1 [INFO] [stdout] | [INFO] [stdout] 158 + #[derive(Clone)] [INFO] [stdout] 159 | pub struct Terrain { [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: 7 warnings emitted [INFO] [stdout] [INFO] [stdout] [INFO] [stderr] Finished `dev` profile [unoptimized + debuginfo] target(s) in 0.63s [INFO] running `Command { std: "docker" "inspect" "6364cd5eb7a8ebfa03b15ede1d723f13a67765b7da0322f89ce6aaba43b38ccf", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "6364cd5eb7a8ebfa03b15ede1d723f13a67765b7da0322f89ce6aaba43b38ccf", kill_on_drop: false }` [INFO] [stdout] 6364cd5eb7a8ebfa03b15ede1d723f13a67765b7da0322f89ce6aaba43b38ccf