[INFO] cloning repository https://github.com/nelsoon/sdr-hub
[INFO] running `Command { std: "git" "-c" "credential.helper=" "-c" "credential.helper=/workspace/cargo-home/bin/git-credential-null" "clone" "--bare" "https://github.com/nelsoon/sdr-hub" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fnelsoon%2Fsdr-hub", kill_on_drop: false }`
[INFO] [stderr] Cloning into bare repository '/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fnelsoon%2Fsdr-hub'...
[INFO] running `Command { std: "git" "rev-parse" "HEAD", kill_on_drop: false }`
[INFO] [stdout] 8ed5b472f3a8f3699559a5093d57c6e6c371bfeb
[INFO] checking nelsoon/sdr-hub against a5689b034ad287112f709693384a37f1377eb9cc for pr-129249-3
[INFO] running `Command { std: "git" "clone" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fnelsoon%2Fsdr-hub" "/workspace/builds/worker-2-tc2/source", kill_on_drop: false }`
[INFO] [stderr] Cloning into '/workspace/builds/worker-2-tc2/source'...
[INFO] [stderr] done.
[INFO] started tweaking git repo https://github.com/nelsoon/sdr-hub
[INFO] finished tweaking git repo https://github.com/nelsoon/sdr-hub
[INFO] tweaked toml for git repo https://github.com/nelsoon/sdr-hub written to /workspace/builds/worker-2-tc2/source/Cargo.toml
[INFO] validating manifest of git repo https://github.com/nelsoon/sdr-hub on toolchain a5689b034ad287112f709693384a37f1377eb9cc
[INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+a5689b034ad287112f709693384a37f1377eb9cc" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }`
[INFO] crate git repo https://github.com/nelsoon/sdr-hub 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" "+a5689b034ad287112f709693384a37f1377eb9cc" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-2-tc2/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-2-tc2/target:/opt/rustwide/target:rw,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" "-m" "1610612736" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:77db811e55d90add9212f6832c23229e3de5f1c1f9905e8cb07319ea15df73ac" "sleep" "infinity", kill_on_drop: false }`
[INFO] [stdout] 4c12b42d53526e495c2bca87cfa20786a93206c17a2d84d8ec7b5caf2268607b
[INFO] running `Command { std: "docker" "start" "4c12b42d53526e495c2bca87cfa20786a93206c17a2d84d8ec7b5caf2268607b", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "4c12b42d53526e495c2bca87cfa20786a93206c17a2d84d8ec7b5caf2268607b", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "exec" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-w" "/opt/rustwide/workdir" "--user" "0:0" "4c12b42d53526e495c2bca87cfa20786a93206c17a2d84d8ec7b5caf2268607b" "/opt/rustwide/cargo-home/bin/cargo" "+a5689b034ad287112f709693384a37f1377eb9cc" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "4c12b42d53526e495c2bca87cfa20786a93206c17a2d84d8ec7b5caf2268607b", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "exec" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-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" "-w" "/opt/rustwide/workdir" "--user" "0:0" "4c12b42d53526e495c2bca87cfa20786a93206c17a2d84d8ec7b5caf2268607b" "/opt/rustwide/cargo-home/bin/cargo" "+a5689b034ad287112f709693384a37f1377eb9cc" "check" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }`
[INFO] [stderr]    Compiling syn v3.0.6
[INFO] [stderr]    Compiling uhd-sys v0.1.0 (/opt/rustwide/workdir/uhd-sys)
[INFO] [stderr]     Checking mio v1.2.3
[INFO] [stderr]     Checking strength_reduce v0.2.4
[INFO] [stderr]     Checking futures-util v0.3.34
[INFO] [stderr]    Compiling syn v2.0.119
[INFO] [stderr]     Checking num-integer v0.1.47
[INFO] [stderr]     Checking num-complex v0.4.6
[INFO] [stderr]     Checking tracing v0.1.44
[INFO] [stderr]     Checking rand_chacha v0.3.1
[INFO] [stderr]     Checking serde_json v1.0.151
[INFO] [stderr]     Checking serde_path_to_error v0.1.20
[INFO] [stderr]     Checking bitflags v2.13.2
[INFO] [stderr]     Checking hub-api v0.1.0 (/opt/rustwide/workdir/hub-api)
[INFO] [stderr]     Checking transpose v0.2.3
[INFO] [stderr]     Checking primal-check v0.3.4
[INFO] [stderr]     Checking rand v0.8.8
[INFO] [stderr]     Checking rustfft v6.4.1
[INFO] [stderr]    Compiling thiserror-impl v1.0.69
[INFO] [stderr]     Checking thiserror v1.0.69
[INFO] [stderr]     Checking tungstenite v0.24.0
[INFO] [stderr]    Compiling tokio-macros v2.7.2
[INFO] [stderr]    Compiling serde_derive v1.0.229
[INFO] [stderr]    Compiling async-trait v0.1.92
[INFO] [stderr]     Checking tokio v1.53.1
[INFO] [stderr]     Checking axum-core v0.4.5
[INFO] [stderr]     Checking sdr-dsp v0.1.0 (/opt/rustwide/workdir/dsp)
[INFO] [stderr]     Checking serde v1.0.229
[INFO] [stdout] warning: unused variable: `n`
[INFO] [stdout]   --> dsp/tests/vectors.rs:43:9
[INFO] [stdout]    |
[INFO] [stdout] 43 |     let n = pcm.len() as f64;
[INFO] [stdout]    |         ^ help: if this is intentional, prefix it with an underscore: `_n`
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr]     Checking serde_spanned v0.6.9
[INFO] [stderr]     Checking toml_datetime v0.6.11
[INFO] [stderr]     Checking serde_urlencoded v0.7.1
[INFO] [stderr]     Checking toml_edit v0.22.27
[INFO] [stderr]     Checking toml v0.8.23
[INFO] [stderr]     Checking hyper v1.11.1
[INFO] [stderr]     Checking tower v0.5.3
[INFO] [stderr]     Checking tokio-tungstenite v0.24.0
[INFO] [stderr]     Checking tokio-util v0.7.19
[INFO] [stderr]     Checking tower-http v0.5.2
[INFO] [stderr]     Checking hyper-util v0.1.20
[INFO] [stderr]     Checking axum v0.7.9
[INFO] [stderr]     Checking sdr-hub v0.1.0 (/opt/rustwide/workdir/hub)
[INFO] [stdout] error: useless conversion to the same type: `Vec<u8>`
[INFO] [stdout]    --> hub/src/server.rs:374:37
[INFO] [stdout]     |
[INFO] [stdout] 374 |                     Message::Binary(encode_audio(&channel, seq, &pcm).into())
[INFO] [stdout]     |                                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout]     = note: `#[deny(self_type_conversion)]` on by default
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 374 -                     Message::Binary(encode_audio(&channel, seq, &pcm).into())
[INFO] [stdout] 374 +                     Message::Binary(encode_audio(&channel, seq, &pcm))
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 374 -                     Message::Binary(encode_audio(&channel, seq, &pcm).into())
[INFO] [stdout] 374 +                     Message::Binary(<Vec<u8> as Into<_>>::into(encode_audio(&channel, seq, &pcm)))
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `std::string::String`
[INFO] [stdout]    --> hub/src/server.rs:378:35
[INFO] [stdout]     |
[INFO] [stdout] 378 |                     Message::Text(t.into())
[INFO] [stdout]     |                                   ^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 378 -                     Message::Text(t.into())
[INFO] [stdout] 378 +                     Message::Text(t)
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 378 -                     Message::Text(t.into())
[INFO] [stdout] 378 +                     Message::Text(<std::string::String as Into<_>>::into(t))
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `std::string::String`
[INFO] [stdout]    --> hub/src/server.rs:411:47
[INFO] [stdout]     |
[INFO] [stdout] 411 |         let _ = out_tx.try_send(Message::Text(t.into()));
[INFO] [stdout]     |                                               ^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 411 -         let _ = out_tx.try_send(Message::Text(t.into()));
[INFO] [stdout] 411 +         let _ = out_tx.try_send(Message::Text(t));
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 411 -         let _ = out_tx.try_send(Message::Text(t.into()));
[INFO] [stdout] 411 +         let _ = out_tx.try_send(Message::Text(<std::string::String as Into<_>>::into(t)));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `std::string::String`
[INFO] [stdout]    --> hub/src/server.rs:426:46
[INFO] [stdout]     |
[INFO] [stdout] 426 |                     if tx.send(Message::Text(t.into())).await.is_err() {
[INFO] [stdout]     |                                              ^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 426 -                     if tx.send(Message::Text(t.into())).await.is_err() {
[INFO] [stdout] 426 +                     if tx.send(Message::Text(t)).await.is_err() {
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 426 -                     if tx.send(Message::Text(t.into())).await.is_err() {
[INFO] [stdout] 426 +                     if tx.send(Message::Text(<std::string::String as Into<_>>::into(t))).await.is_err() {
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: comparison is useless due to type limits
[INFO] [stdout]    --> hub/src/channel.rs:159:17
[INFO] [stdout]     |
[INFO] [stdout] 159 |         assert!(pcm_limit(3.0) <= 32767);
[INFO] [stdout]     |                 ^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: `#[warn(unused_comparisons)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Vec<u8>`
[INFO] [stdout]    --> hub/src/server.rs:374:37
[INFO] [stdout]     |
[INFO] [stdout] 374 |                     Message::Binary(encode_audio(&channel, seq, &pcm).into())
[INFO] [stdout]     |                                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout]     = note: `#[deny(self_type_conversion)]` on by default
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 374 -                     Message::Binary(encode_audio(&channel, seq, &pcm).into())
[INFO] [stdout] 374 +                     Message::Binary(encode_audio(&channel, seq, &pcm))
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 374 -                     Message::Binary(encode_audio(&channel, seq, &pcm).into())
[INFO] [stdout] 374 +                     Message::Binary(<Vec<u8> as Into<_>>::into(encode_audio(&channel, seq, &pcm)))
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `std::string::String`
[INFO] [stdout]    --> hub/src/server.rs:378:35
[INFO] [stdout]     |
[INFO] [stdout] 378 |                     Message::Text(t.into())
[INFO] [stdout]     |                                   ^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 378 -                     Message::Text(t.into())
[INFO] [stdout] 378 +                     Message::Text(t)
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 378 -                     Message::Text(t.into())
[INFO] [stdout] 378 +                     Message::Text(<std::string::String as Into<_>>::into(t))
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `std::string::String`
[INFO] [stdout]    --> hub/src/server.rs:411:47
[INFO] [stdout]     |
[INFO] [stdout] 411 |         let _ = out_tx.try_send(Message::Text(t.into()));
[INFO] [stdout]     |                                               ^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 411 -         let _ = out_tx.try_send(Message::Text(t.into()));
[INFO] [stdout] 411 +         let _ = out_tx.try_send(Message::Text(t));
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 411 -         let _ = out_tx.try_send(Message::Text(t.into()));
[INFO] [stdout] 411 +         let _ = out_tx.try_send(Message::Text(<std::string::String as Into<_>>::into(t)));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `std::string::String`
[INFO] [stdout]    --> hub/src/server.rs:426:46
[INFO] [stdout]     |
[INFO] [stdout] 426 |                     if tx.send(Message::Text(t.into())).await.is_err() {
[INFO] [stdout]     |                                              ^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 426 -                     if tx.send(Message::Text(t.into())).await.is_err() {
[INFO] [stdout] 426 +                     if tx.send(Message::Text(t)).await.is_err() {
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 426 -                     if tx.send(Message::Text(t.into())).await.is_err() {
[INFO] [stdout] 426 +                     if tx.send(Message::Text(<std::string::String as Into<_>>::into(t))).await.is_err() {
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr] error: could not compile `sdr-hub` (lib test) due to 4 previous errors; 1 warning emitted
[INFO] [stderr] warning: build failed, waiting for other jobs to finish...
[INFO] [stderr] error: could not compile `sdr-hub` (lib) due to 4 previous errors
[INFO] running `Command { std: "docker" "inspect" "4c12b42d53526e495c2bca87cfa20786a93206c17a2d84d8ec7b5caf2268607b", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "4c12b42d53526e495c2bca87cfa20786a93206c17a2d84d8ec7b5caf2268607b", kill_on_drop: false }`
[INFO] [stdout] 4c12b42d53526e495c2bca87cfa20786a93206c17a2d84d8ec7b5caf2268607b
