[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 try#6db1a6a7ebb2ad243051f3ff949f987dd70cde2b+rustflags=-Dreturn-local-variable-ptr for pr-134218 [INFO] running `Command { std: "git" "clone" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fjpastuszek%2FmicroRTS-rs" "/workspace/builds/worker-6-tc2/source", kill_on_drop: false }` [INFO] [stderr] Cloning into '/workspace/builds/worker-6-tc2/source'... [INFO] [stderr] done. [INFO] validating manifest of git repo https://github.com/jpastuszek/microRTS-rs on toolchain 6db1a6a7ebb2ad243051f3ff949f987dd70cde2b [INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+6db1a6a7ebb2ad243051f3ff949f987dd70cde2b" "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-6-tc2/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" "+6db1a6a7ebb2ad243051f3ff949f987dd70cde2b" "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 2024 [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-6-tc2/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-6-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:f3a9d4ad9d972b27faf3965f35b62e55ba32bbce8f20bc8fe909558a86702fde" "/opt/rustwide/cargo-home/bin/cargo" "+6db1a6a7ebb2ad243051f3ff949f987dd70cde2b" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }` [INFO] [stdout] 24780e61948f82504cbce77e5b26fe32ed965ce04011d329610742c5f35236bc [INFO] running `Command { std: "docker" "start" "-a" "24780e61948f82504cbce77e5b26fe32ed965ce04011d329610742c5f35236bc", kill_on_drop: false }` [INFO] running `Command { std: "docker" "inspect" "24780e61948f82504cbce77e5b26fe32ed965ce04011d329610742c5f35236bc", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "24780e61948f82504cbce77e5b26fe32ed965ce04011d329610742c5f35236bc", kill_on_drop: false }` [INFO] [stdout] 24780e61948f82504cbce77e5b26fe32ed965ce04011d329610742c5f35236bc [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-6-tc2/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-6-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 -Dreturn-local-variable-ptr" "-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:f3a9d4ad9d972b27faf3965f35b62e55ba32bbce8f20bc8fe909558a86702fde" "/opt/rustwide/cargo-home/bin/cargo" "+6db1a6a7ebb2ad243051f3ff949f987dd70cde2b" "check" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }` [INFO] [stdout] 7dfc22216a72f6eac38f633788c723965cef1b73989fa3a7963f7638349c0793 [INFO] running `Command { std: "docker" "start" "-a" "7dfc22216a72f6eac38f633788c723965cef1b73989fa3a7963f7638349c0793", 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 2024 [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: 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] [stderr] Finished `dev` profile [unoptimized + debuginfo] target(s) in 0.89s [INFO] running `Command { std: "docker" "inspect" "7dfc22216a72f6eac38f633788c723965cef1b73989fa3a7963f7638349c0793", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "7dfc22216a72f6eac38f633788c723965cef1b73989fa3a7963f7638349c0793", kill_on_drop: false }` [INFO] [stdout] 7dfc22216a72f6eac38f633788c723965cef1b73989fa3a7963f7638349c0793