[INFO] cloning repository https://github.com/elipp/rs.core [INFO] running `Command { std: "git" "-c" "credential.helper=" "-c" "credential.helper=/workspace/cargo-home/bin/git-credential-null" "clone" "--bare" "https://github.com/elipp/rs.core" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Felipp%2Frs.core", kill_on_drop: false }` [INFO] [stderr] Cloning into bare repository '/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Felipp%2Frs.core'... [INFO] running `Command { std: "git" "rev-parse" "HEAD", kill_on_drop: false }` [INFO] [stdout] 7d8dc433983efde8091772bb33bfb7e0fde8c62c [INFO] checking elipp/rs.core against try#93086cfebfccb79083c59d0d3b2052944bba643c for pr-146440-1 [INFO] running `Command { std: "git" "clone" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Felipp%2Frs.core" "/workspace/builds/worker-1-tc2/source", kill_on_drop: false }` [INFO] [stderr] Cloning into '/workspace/builds/worker-1-tc2/source'... [INFO] [stderr] done. [INFO] started tweaking git repo https://github.com/elipp/rs.core [INFO] finished tweaking git repo https://github.com/elipp/rs.core [INFO] tweaked toml for git repo https://github.com/elipp/rs.core written to /workspace/builds/worker-1-tc2/source/Cargo.toml [INFO] validating manifest of git repo https://github.com/elipp/rs.core on toolchain 93086cfebfccb79083c59d0d3b2052944bba643c [INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+93086cfebfccb79083c59d0d3b2052944bba643c" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }` [INFO] crate git repo https://github.com/elipp/rs.core 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" "+93086cfebfccb79083c59d0d3b2052944bba643c" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }` [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-1-tc2/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-1-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:d429b63d4308055ea97f60fb1d3dfca48854a00942f1bd2ad806beaf015945ec" "/opt/rustwide/cargo-home/bin/cargo" "+93086cfebfccb79083c59d0d3b2052944bba643c" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }` [INFO] [stdout] 976f7b179e8924f0611a42c76e6ef16af705bc933d3fe93eed959c585f0fe59d [INFO] running `Command { std: "docker" "start" "-a" "976f7b179e8924f0611a42c76e6ef16af705bc933d3fe93eed959c585f0fe59d", kill_on_drop: false }` [INFO] running `Command { std: "docker" "inspect" "976f7b179e8924f0611a42c76e6ef16af705bc933d3fe93eed959c585f0fe59d", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "976f7b179e8924f0611a42c76e6ef16af705bc933d3fe93eed959c585f0fe59d", kill_on_drop: false }` [INFO] [stdout] 976f7b179e8924f0611a42c76e6ef16af705bc933d3fe93eed959c585f0fe59d [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-1-tc2/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-1-tc2/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/cargo-home:/opt/rustwide/cargo-home:ro,Z" "-v" "/var/lib/crater-agent-workspace/rustup-home:/opt/rustwide/rustup-home:ro,Z" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-e" "CARGO_INCREMENTAL=0" "-e" "RUST_BACKTRACE=full" "-e" "RUSTFLAGS=--cap-lints=forbid" "-e" "RUSTDOCFLAGS=--cap-lints=forbid" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-w" "/opt/rustwide/workdir" "-m" "1610612736" "--user" "0:0" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:d429b63d4308055ea97f60fb1d3dfca48854a00942f1bd2ad806beaf015945ec" "/opt/rustwide/cargo-home/bin/cargo" "+93086cfebfccb79083c59d0d3b2052944bba643c" "check" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }` [INFO] [stdout] f519004ed38a793d3418faf417ac9fac4cb10403e0323dd89a2a251153a12b1c [INFO] running `Command { std: "docker" "start" "-a" "f519004ed38a793d3418faf417ac9fac4cb10403e0323dd89a2a251153a12b1c", kill_on_drop: false }` [INFO] [stderr] Compiling proc-macro2 v1.0.92 [INFO] [stderr] Compiling libc v0.2.164 [INFO] [stderr] Compiling zerocopy v0.8.10 [INFO] [stderr] Checking inout v0.1.3 [INFO] [stderr] Checking digest v0.10.7 [INFO] [stderr] Checking unicode-properties v0.1.3 [INFO] [stderr] Checking unicode-bidi v0.3.17 [INFO] [stderr] Checking tracing-core v0.1.32 [INFO] [stderr] Checking crypto v0.5.1 [INFO] [stderr] Checking cipher v0.4.4 [INFO] [stderr] Checking fallible-iterator v0.2.0 [INFO] [stderr] Checking rc4 v0.1.0 [INFO] [stderr] Checking stringprep v0.1.5 [INFO] [stderr] Checking hmac v0.12.1 [INFO] [stderr] Checking sha1 v0.10.6 [INFO] [stderr] Checking md-5 v0.10.6 [INFO] [stderr] Checking sha2 v0.10.8 [INFO] [stderr] Checking anstyle-parse v0.2.6 [INFO] [stderr] Checking anstyle v1.0.10 [INFO] [stderr] Checking colorchoice v1.0.3 [INFO] [stderr] Checking anstyle-query v1.1.2 [INFO] [stderr] Checking phf v0.11.2 [INFO] [stderr] Checking futures-channel v0.3.31 [INFO] [stderr] Checking whoami v1.5.2 [INFO] [stderr] Checking clap_lex v0.7.3 [INFO] [stderr] Checking tracing-log v0.2.0 [INFO] [stderr] Checking anstream v0.6.18 [INFO] [stderr] Checking thread_local v1.1.8 [INFO] [stderr] Checking tracing-subscriber v0.3.18 [INFO] [stderr] Compiling quote v1.0.37 [INFO] [stderr] Checking clap_builder v4.5.21 [INFO] [stderr] Compiling syn v2.0.89 [INFO] [stderr] Checking getrandom v0.2.15 [INFO] [stderr] Checking parking_lot_core v0.9.10 [INFO] [stderr] Checking mio v1.0.2 [INFO] [stderr] Checking signal-hook-registry v1.4.2 [INFO] [stderr] Checking socket2 v0.5.7 [INFO] [stderr] Checking parking_lot v0.12.3 [INFO] [stderr] Checking num_cpus v1.16.0 [INFO] [stderr] Checking rand_core v0.6.4 [INFO] [stderr] Checking crypto-bigint v0.5.5 [INFO] [stderr] Compiling zerocopy-derive v0.7.35 [INFO] [stderr] Compiling tokio-macros v2.4.0 [INFO] [stderr] Compiling zerocopy-derive v0.8.10 [INFO] [stderr] Compiling futures-macro v0.3.31 [INFO] [stderr] Compiling async-trait v0.1.83 [INFO] [stderr] Compiling tracing-attributes v0.1.27 [INFO] [stderr] Compiling clap_derive v4.5.18 [INFO] [stderr] Checking tokio v1.41.1 [INFO] [stderr] Checking zerocopy v0.7.35 [INFO] [stderr] Checking futures-util v0.3.31 [INFO] [stderr] Checking tracing v0.1.40 [INFO] [stderr] Checking ppv-lite86 v0.2.20 [INFO] [stderr] Checking clap v4.5.21 [INFO] [stderr] Checking rand_chacha v0.3.1 [INFO] [stderr] Checking rand v0.8.5 [INFO] [stderr] Checking postgres-protocol v0.6.7 [INFO] [stderr] Checking postgres-types v0.2.8 [INFO] [stderr] Checking wow_proto v0.1.0 (/opt/rustwide/workdir/proto) [INFO] [stderr] Checking tokio-util v0.7.12 [INFO] [stderr] Checking deadpool-runtime v0.1.4 [INFO] [stderr] Checking deadpool v0.12.1 [INFO] [stdout] warning: unused import: `Digest` [INFO] [stdout] --> proto/src/lib.rs:11:12 [INFO] [stdout] | [INFO] [stdout] 11 | use sha1::{Digest, Sha1}; [INFO] [stdout] | ^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_imports)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `e` [INFO] [stdout] --> proto/src/lib.rs:415:23 [INFO] [stdout] | [INFO] [stdout] 415 | .map_err(|e| ProtocolError::Error(String::from("read() 2")))?; [INFO] [stdout] | ^ help: if this is intentional, prefix it with an underscore: `_e` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `e` [INFO] [stdout] --> proto/src/lib.rs:408:23 [INFO] [stdout] | [INFO] [stdout] 408 | .map_err(|e| ProtocolError::Error(String::from("read()")))?; [INFO] [stdout] | ^ help: if this is intentional, prefix it with an underscore: `_e` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `e` [INFO] [stdout] --> proto/src/lib.rs:446:23 [INFO] [stdout] | [INFO] [stdout] 446 | .map_err(|e| ProtocolError::Error(String::from("read()")))?; [INFO] [stdout] | ^ help: if this is intentional, prefix it with an underscore: `_e` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `e` [INFO] [stdout] --> proto/src/lib.rs:432:23 [INFO] [stdout] | [INFO] [stdout] 432 | .map_err(|e| ProtocolError::Error(String::from("read()")))?; [INFO] [stdout] | ^ help: if this is intentional, prefix it with an underscore: `_e` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `e` [INFO] [stdout] --> proto/src/lib.rs:472:23 [INFO] [stdout] | [INFO] [stdout] 472 | .map_err(|e| ProtocolError::Error(String::from("read()")))?; [INFO] [stdout] | ^ help: if this is intentional, prefix it with an underscore: `_e` [INFO] [stdout] [INFO] [stdout] [INFO] [stderr] Checking tokio-postgres v0.7.12 [INFO] [stderr] Checking client v0.1.0 (/opt/rustwide/workdir/client) [INFO] [stdout] warning: unused import: `Digest` [INFO] [stdout] --> proto/src/lib.rs:11:12 [INFO] [stdout] | [INFO] [stdout] 11 | use sha1::{Digest, Sha1}; [INFO] [stdout] | ^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_imports)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `e` [INFO] [stdout] --> proto/src/lib.rs:415:23 [INFO] [stdout] | [INFO] [stdout] 415 | .map_err(|e| ProtocolError::Error(String::from("read() 2")))?; [INFO] [stdout] | ^ help: if this is intentional, prefix it with an underscore: `_e` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `e` [INFO] [stdout] --> proto/src/lib.rs:408:23 [INFO] [stdout] | [INFO] [stdout] 408 | .map_err(|e| ProtocolError::Error(String::from("read()")))?; [INFO] [stdout] | ^ help: if this is intentional, prefix it with an underscore: `_e` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `e` [INFO] [stdout] --> proto/src/lib.rs:446:23 [INFO] [stdout] | [INFO] [stdout] 446 | .map_err(|e| ProtocolError::Error(String::from("read()")))?; [INFO] [stdout] | ^ help: if this is intentional, prefix it with an underscore: `_e` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `e` [INFO] [stdout] --> proto/src/lib.rs:432:23 [INFO] [stdout] | [INFO] [stdout] 432 | .map_err(|e| ProtocolError::Error(String::from("read()")))?; [INFO] [stdout] | ^ help: if this is intentional, prefix it with an underscore: `_e` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `e` [INFO] [stdout] --> proto/src/lib.rs:472:23 [INFO] [stdout] | [INFO] [stdout] 472 | .map_err(|e| ProtocolError::Error(String::from("read()")))?; [INFO] [stdout] | ^ help: if this is intentional, prefix it with an underscore: `_e` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> client/src/main.rs:434:14 [INFO] [stdout] | [INFO] [stdout] 434 | let (mut encrypt, mut decrypt) = init_crypto(&session_key)?; [INFO] [stdout] | ----^^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_mut)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `e` [INFO] [stdout] --> client/src/main.rs:390:23 [INFO] [stdout] | [INFO] [stdout] 390 | .map_err(|e| WowCliError::LoginFailed)?; [INFO] [stdout] | ^ help: if this is intentional, prefix it with an underscore: `_e` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `encrypt` [INFO] [stdout] --> client/src/main.rs:434:14 [INFO] [stdout] | [INFO] [stdout] 434 | let (mut encrypt, mut decrypt) = init_crypto(&session_key)?; [INFO] [stdout] | ^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_encrypt` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `packet` [INFO] [stdout] --> client/src/main.rs:436:17 [INFO] [stdout] | [INFO] [stdout] 436 | let packet = read_one_encrypted(&mut stream, &mut buf, &mut decrypt).await?; [INFO] [stdout] | ^^^^^^ help: if this is intentional, prefix it with an underscore: `_packet` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: field `0` is never read [INFO] [stdout] --> client/src/main.rs:52:13 [INFO] [stdout] | [INFO] [stdout] 52 | IoError(std::io::Error), [INFO] [stdout] | ------- ^^^^^^^^^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | field in this variant [INFO] [stdout] | [INFO] [stdout] = note: `WowCliError` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis [INFO] [stdout] = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default [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] 52 - IoError(std::io::Error), [INFO] [stdout] 52 + IoError(()), [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: field `0` is never read [INFO] [stdout] --> client/src/main.rs:58:16 [INFO] [stdout] | [INFO] [stdout] 58 | OtherError(String), [INFO] [stdout] | ---------- ^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | field in this variant [INFO] [stdout] | [INFO] [stdout] = note: `WowCliError` 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] 58 - OtherError(String), [INFO] [stdout] 58 + OtherError(()), [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: field `0` is never read [INFO] [stdout] --> client/src/main.rs:59:19 [INFO] [stdout] | [INFO] [stdout] 59 | ProtocolError(ProtocolError), [INFO] [stdout] | ------------- ^^^^^^^^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | field in this variant [INFO] [stdout] | [INFO] [stdout] = note: `WowCliError` 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] 59 - ProtocolError(ProtocolError), [INFO] [stdout] 59 + ProtocolError(()), [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variants `RealmServerConnectionFailed`, `UnexpectedResponseLength`, `BufferTooSmall`, `UnexpectedOpcode`, and `NetworkError` are never constructed [INFO] [stdout] --> client/src/main.rs:48:5 [INFO] [stdout] | [INFO] [stdout] 47 | enum WowCliError { [INFO] [stdout] | ----------- variants in this enum [INFO] [stdout] 48 | RealmServerConnectionFailed, [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 51 | UnexpectedResponseLength, [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] 52 | IoError(std::io::Error), [INFO] [stdout] 53 | BufferTooSmall, [INFO] [stdout] | ^^^^^^^^^^^^^^ [INFO] [stdout] 54 | RealmIsOffline, [INFO] [stdout] 55 | UnexpectedOpcode(u16), [INFO] [stdout] | ^^^^^^^^^^^^^^^^ [INFO] [stdout] 56 | CryptoInvalidLength(crypto::common::InvalidLength), [INFO] [stdout] 57 | NetworkError, [INFO] [stdout] | ^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `WowCliError` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function `reversed` is never used [INFO] [stdout] --> client/src/main.rs:62:4 [INFO] [stdout] | [INFO] [stdout] 62 | fn reversed(arg: &[T]) -> Vec { [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: struct `ServerPacketHeader` is never constructed [INFO] [stdout] --> client/src/main.rs:280:8 [INFO] [stdout] | [INFO] [stdout] 280 | struct ServerPacketHeader { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> client/src/main.rs:434:14 [INFO] [stdout] | [INFO] [stdout] 434 | let (mut encrypt, mut decrypt) = init_crypto(&session_key)?; [INFO] [stdout] | ----^^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_mut)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `e` [INFO] [stdout] --> client/src/main.rs:390:23 [INFO] [stdout] | [INFO] [stdout] 390 | .map_err(|e| WowCliError::LoginFailed)?; [INFO] [stdout] | ^ help: if this is intentional, prefix it with an underscore: `_e` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `encrypt` [INFO] [stdout] --> client/src/main.rs:434:14 [INFO] [stdout] | [INFO] [stdout] 434 | let (mut encrypt, mut decrypt) = init_crypto(&session_key)?; [INFO] [stdout] | ^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_encrypt` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `packet` [INFO] [stdout] --> client/src/main.rs:436:17 [INFO] [stdout] | [INFO] [stdout] 436 | let packet = read_one_encrypted(&mut stream, &mut buf, &mut decrypt).await?; [INFO] [stdout] | ^^^^^^ help: if this is intentional, prefix it with an underscore: `_packet` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: field `0` is never read [INFO] [stdout] --> client/src/main.rs:52:13 [INFO] [stdout] | [INFO] [stdout] 52 | IoError(std::io::Error), [INFO] [stdout] | ------- ^^^^^^^^^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | field in this variant [INFO] [stdout] | [INFO] [stdout] = note: `WowCliError` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis [INFO] [stdout] = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default [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] 52 - IoError(std::io::Error), [INFO] [stdout] 52 + IoError(()), [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: field `0` is never read [INFO] [stdout] --> client/src/main.rs:58:16 [INFO] [stdout] | [INFO] [stdout] 58 | OtherError(String), [INFO] [stdout] | ---------- ^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | field in this variant [INFO] [stdout] | [INFO] [stdout] = note: `WowCliError` 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] 58 - OtherError(String), [INFO] [stdout] 58 + OtherError(()), [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: field `0` is never read [INFO] [stdout] --> client/src/main.rs:59:19 [INFO] [stdout] | [INFO] [stdout] 59 | ProtocolError(ProtocolError), [INFO] [stdout] | ------------- ^^^^^^^^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | field in this variant [INFO] [stdout] | [INFO] [stdout] = note: `WowCliError` 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] 59 - ProtocolError(ProtocolError), [INFO] [stdout] 59 + ProtocolError(()), [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variants `RealmServerConnectionFailed`, `UnexpectedResponseLength`, `BufferTooSmall`, `UnexpectedOpcode`, and `NetworkError` are never constructed [INFO] [stdout] --> client/src/main.rs:48:5 [INFO] [stdout] | [INFO] [stdout] 47 | enum WowCliError { [INFO] [stdout] | ----------- variants in this enum [INFO] [stdout] 48 | RealmServerConnectionFailed, [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] ... [INFO] [stdout] 51 | UnexpectedResponseLength, [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] 52 | IoError(std::io::Error), [INFO] [stdout] 53 | BufferTooSmall, [INFO] [stdout] | ^^^^^^^^^^^^^^ [INFO] [stdout] 54 | RealmIsOffline, [INFO] [stdout] 55 | UnexpectedOpcode(u16), [INFO] [stdout] | ^^^^^^^^^^^^^^^^ [INFO] [stdout] 56 | CryptoInvalidLength(crypto::common::InvalidLength), [INFO] [stdout] 57 | NetworkError, [INFO] [stdout] | ^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `WowCliError` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function `reversed` is never used [INFO] [stdout] --> client/src/main.rs:62:4 [INFO] [stdout] | [INFO] [stdout] 62 | fn reversed(arg: &[T]) -> Vec { [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: struct `ServerPacketHeader` is never constructed [INFO] [stdout] --> client/src/main.rs:280:8 [INFO] [stdout] | [INFO] [stdout] 280 | struct ServerPacketHeader { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stderr] Checking deadpool-postgres v0.14.0 [INFO] [stderr] Checking authserver v0.1.0 (/opt/rustwide/workdir/authserver) [INFO] [stderr] Checking worldserver v0.1.0 (/opt/rustwide/workdir/worldserver) [INFO] [stdout] warning: unused import: `GenericClient` [INFO] [stdout] --> worldserver/src/main.rs:5:25 [INFO] [stdout] | [INFO] [stdout] 5 | use deadpool_postgres::{GenericClient, ManagerConfig, RecyclingMethod, Runtime}; [INFO] [stdout] | ^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_imports)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused imports: `AuthChallenge`, `AuthClientProof`, `AuthProtoPacketHeader`, `AuthServerProof`, `ProtoPacket`, `RecvPacket`, and `SendPacket` [INFO] [stdout] --> worldserver/src/main.rs:8:5 [INFO] [stdout] | [INFO] [stdout] 8 | AuthChallenge, AuthClientProof, AuthProtoPacketHeader, AuthServerProof, ProtoPacket, [INFO] [stdout] | ^^^^^^^^^^^^^ ^^^^^^^^^^^^^^^ ^^^^^^^^^^^^^^^^^^^^^ ^^^^^^^^^^^^^^^ ^^^^^^^^^^^ [INFO] [stdout] 9 | ProtocolError, RecvPacket, SendPacket, [INFO] [stdout] | ^^^^^^^^^^ ^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `core::str` [INFO] [stdout] --> worldserver/src/main.rs:12:5 [INFO] [stdout] | [INFO] [stdout] 12 | use core::str; [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> worldserver/src/main.rs:72:21 [INFO] [stdout] | [INFO] [stdout] 72 | let mut buf = vec![0u8; 16 * 1024]; [INFO] [stdout] | ----^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_mut)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> worldserver/src/main.rs:66:14 [INFO] [stdout] | [INFO] [stdout] 66 | let (mut socket, _) = listener.accept().await?; [INFO] [stdout] | ----^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `buf` [INFO] [stdout] --> worldserver/src/main.rs:72:21 [INFO] [stdout] | [INFO] [stdout] 72 | let mut buf = vec![0u8; 16 * 1024]; [INFO] [stdout] | ^^^^^^^ help: if this is intentional, prefix it with an underscore: `_buf` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `ip` [INFO] [stdout] --> worldserver/src/main.rs:73:21 [INFO] [stdout] | [INFO] [stdout] 73 | let ip = socket [INFO] [stdout] | ^^ help: if this is intentional, prefix it with an underscore: `_ip` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `pool_clone` [INFO] [stdout] --> worldserver/src/main.rs:67:13 [INFO] [stdout] | [INFO] [stdout] 67 | let pool_clone = pool.clone(); [INFO] [stdout] | ^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_pool_clone` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `GenericClient` [INFO] [stdout] --> worldserver/src/main.rs:5:25 [INFO] [stdout] | [INFO] [stdout] 5 | use deadpool_postgres::{GenericClient, ManagerConfig, RecyclingMethod, Runtime}; [INFO] [stdout] | ^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_imports)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused imports: `AuthChallenge`, `AuthClientProof`, `AuthProtoPacketHeader`, `AuthServerProof`, `ProtoPacket`, `RecvPacket`, and `SendPacket` [INFO] [stdout] --> worldserver/src/main.rs:8:5 [INFO] [stdout] | [INFO] [stdout] 8 | AuthChallenge, AuthClientProof, AuthProtoPacketHeader, AuthServerProof, ProtoPacket, [INFO] [stdout] | ^^^^^^^^^^^^^ ^^^^^^^^^^^^^^^ ^^^^^^^^^^^^^^^^^^^^^ ^^^^^^^^^^^^^^^ ^^^^^^^^^^^ [INFO] [stdout] 9 | ProtocolError, RecvPacket, SendPacket, [INFO] [stdout] | ^^^^^^^^^^ ^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `core::str` [INFO] [stdout] --> worldserver/src/main.rs:12:5 [INFO] [stdout] | [INFO] [stdout] 12 | use core::str; [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> worldserver/src/main.rs:72:21 [INFO] [stdout] | [INFO] [stdout] 72 | let mut buf = vec![0u8; 16 * 1024]; [INFO] [stdout] | ----^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_mut)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> worldserver/src/main.rs:66:14 [INFO] [stdout] | [INFO] [stdout] 66 | let (mut socket, _) = listener.accept().await?; [INFO] [stdout] | ----^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `buf` [INFO] [stdout] --> worldserver/src/main.rs:72:21 [INFO] [stdout] | [INFO] [stdout] 72 | let mut buf = vec![0u8; 16 * 1024]; [INFO] [stdout] | ^^^^^^^ help: if this is intentional, prefix it with an underscore: `_buf` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `ip` [INFO] [stdout] --> worldserver/src/main.rs:73:21 [INFO] [stdout] | [INFO] [stdout] 73 | let ip = socket [INFO] [stdout] | ^^ help: if this is intentional, prefix it with an underscore: `_ip` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `pool_clone` [INFO] [stdout] --> worldserver/src/main.rs:67:13 [INFO] [stdout] | [INFO] [stdout] 67 | let pool_clone = pool.clone(); [INFO] [stdout] | ^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_pool_clone` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `e` [INFO] [stdout] --> authserver/src/lib.rs:34:52 [INFO] [stdout] | [INFO] [stdout] 34 | verifier: verifier.try_into().map_err(|e| { [INFO] [stdout] | ^ help: if this is intentional, prefix it with an underscore: `_e` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `e` [INFO] [stdout] --> authserver/src/lib.rs:32:27 [INFO] [stdout] | [INFO] [stdout] 32 | .map_err(|e| ProtocolError::Error(format!("unexpected size for `salt` in db")))?, [INFO] [stdout] | ^ help: if this is intentional, prefix it with an underscore: `_e` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `verifier` [INFO] [stdout] --> authserver/src/auth.rs:232:13 [INFO] [stdout] | [INFO] [stdout] 232 | let verifier = calculate_verifier(&"tlipp".to_uppercase(), "kuusysi69", &salt); [INFO] [stdout] | ^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_verifier` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `e` [INFO] [stdout] --> authserver/src/lib.rs:34:52 [INFO] [stdout] | [INFO] [stdout] 34 | verifier: verifier.try_into().map_err(|e| { [INFO] [stdout] | ^ help: if this is intentional, prefix it with an underscore: `_e` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `e` [INFO] [stdout] --> authserver/src/lib.rs:32:27 [INFO] [stdout] | [INFO] [stdout] 32 | .map_err(|e| ProtocolError::Error(format!("unexpected size for `salt` in db")))?, [INFO] [stdout] | ^ help: if this is intentional, prefix it with an underscore: `_e` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `verifier` [INFO] [stdout] --> authserver/src/auth.rs:232:13 [INFO] [stdout] | [INFO] [stdout] 232 | let verifier = calculate_verifier(&"tlipp".to_uppercase(), "kuusysi69", &salt); [INFO] [stdout] | ^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_verifier` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: struct `Srp6` is never constructed [INFO] [stdout] --> authserver/src/utils/add_account.rs:22:12 [INFO] [stdout] | [INFO] [stdout] 22 | pub struct Srp6 { [INFO] [stdout] | ^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `pub` struct has private fields, no public constructor, and is not otherwise reachable through the external API, so consider providing a public constructor or removing it [INFO] [stdout] = note: `#[deny(unused_unconstructable_pub_structs)]` (part of `#[deny(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: fields `login`, `salt`, and `verifier` are never read [INFO] [stdout] --> authserver/src/utils/add_account.rs:23:5 [INFO] [stdout] | [INFO] [stdout] 22 | pub struct Srp6 { [INFO] [stdout] | ---- fields in this struct [INFO] [stdout] 23 | login: String, [INFO] [stdout] | ^^^^^ [INFO] [stdout] 24 | salt: Salt, [INFO] [stdout] | ^^^^ [INFO] [stdout] 25 | verifier: Verifier, [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: field `0` is never read [INFO] [stdout] --> authserver/src/utils/add_account.rs:30:26 [INFO] [stdout] | [INFO] [stdout] 30 | AccountAlreadyExists(i32), [INFO] [stdout] | -------------------- ^^^ [INFO] [stdout] | | [INFO] [stdout] | field in this variant [INFO] [stdout] | [INFO] [stdout] = note: `Error` 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] 30 - AccountAlreadyExists(i32), [INFO] [stdout] 30 + AccountAlreadyExists(()), [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: field `0` is never read [INFO] [stdout] --> authserver/src/utils/add_account.rs:31:13 [INFO] [stdout] | [INFO] [stdout] 31 | PgError(tokio_postgres::Error), [INFO] [stdout] | ------- ^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | field in this variant [INFO] [stdout] | [INFO] [stdout] = note: `Error` 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] 31 - PgError(tokio_postgres::Error), [INFO] [stdout] 31 + PgError(()), [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: struct `Srp6` is never constructed [INFO] [stdout] --> authserver/src/utils/add_account.rs:22:12 [INFO] [stdout] | [INFO] [stdout] 22 | pub struct Srp6 { [INFO] [stdout] | ^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `pub` struct has private fields, no public constructor, and is not otherwise reachable through the external API, so consider providing a public constructor or removing it [INFO] [stdout] = note: `#[deny(unused_unconstructable_pub_structs)]` (part of `#[deny(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: fields `login`, `salt`, and `verifier` are never read [INFO] [stdout] --> authserver/src/utils/add_account.rs:23:5 [INFO] [stdout] | [INFO] [stdout] 22 | pub struct Srp6 { [INFO] [stdout] | ---- fields in this struct [INFO] [stdout] 23 | login: String, [INFO] [stdout] | ^^^^^ [INFO] [stdout] 24 | salt: Salt, [INFO] [stdout] | ^^^^ [INFO] [stdout] 25 | verifier: Verifier, [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: field `0` is never read [INFO] [stdout] --> authserver/src/utils/add_account.rs:30:26 [INFO] [stdout] | [INFO] [stdout] 30 | AccountAlreadyExists(i32), [INFO] [stdout] | -------------------- ^^^ [INFO] [stdout] | | [INFO] [stdout] | field in this variant [INFO] [stdout] | [INFO] [stdout] = note: `Error` 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] 30 - AccountAlreadyExists(i32), [INFO] [stdout] 30 + AccountAlreadyExists(()), [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: field `0` is never read [INFO] [stdout] --> authserver/src/utils/add_account.rs:31:13 [INFO] [stdout] | [INFO] [stdout] 31 | PgError(tokio_postgres::Error), [INFO] [stdout] | ------- ^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | field in this variant [INFO] [stdout] | [INFO] [stdout] = note: `Error` 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] 31 - PgError(tokio_postgres::Error), [INFO] [stdout] 31 + PgError(()), [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stderr] error: could not compile `authserver` (bin "add_account" test) due to 1 previous error; 3 warnings emitted [INFO] [stderr] warning: build failed, waiting for other jobs to finish... [INFO] [stderr] error: could not compile `authserver` (bin "add_account") due to 1 previous error; 3 warnings emitted [INFO] running `Command { std: "docker" "inspect" "f519004ed38a793d3418faf417ac9fac4cb10403e0323dd89a2a251153a12b1c", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "f519004ed38a793d3418faf417ac9fac4cb10403e0323dd89a2a251153a12b1c", kill_on_drop: false }` [INFO] [stdout] f519004ed38a793d3418faf417ac9fac4cb10403e0323dd89a2a251153a12b1c