[INFO] fetching crate thrussh 0.49.0...
[INFO] checking thrussh-0.49.0 against a5689b034ad287112f709693384a37f1377eb9cc for pr-129249-3
[INFO] extracting crate thrussh 0.49.0 into /workspace/builds/worker-1-tc2/source
[INFO] started tweaking crates.io crate thrussh 0.49.0
[INFO] finished tweaking crates.io crate thrussh 0.49.0
[INFO] tweaked toml for crates.io crate thrussh 0.49.0 written to /workspace/builds/worker-1-tc2/source/Cargo.toml
[INFO] validating manifest of crates.io crate thrussh 0.49.0 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 crates.io crate thrussh 0.49.0 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-1-tc2/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-1-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] 5105a6d38d1d3b9c7c63c6fb10b02acc33576f1db311f93160b91518cf15359c
[INFO] running `Command { std: "docker" "start" "5105a6d38d1d3b9c7c63c6fb10b02acc33576f1db311f93160b91518cf15359c", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "5105a6d38d1d3b9c7c63c6fb10b02acc33576f1db311f93160b91518cf15359c", 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" "5105a6d38d1d3b9c7c63c6fb10b02acc33576f1db311f93160b91518cf15359c" "/opt/rustwide/cargo-home/bin/cargo" "+a5689b034ad287112f709693384a37f1377eb9cc" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "5105a6d38d1d3b9c7c63c6fb10b02acc33576f1db311f93160b91518cf15359c", 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" "5105a6d38d1d3b9c7c63c6fb10b02acc33576f1db311f93160b91518cf15359c" "/opt/rustwide/cargo-home/bin/cargo" "+a5689b034ad287112f709693384a37f1377eb9cc" "check" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }`
[INFO] [stderr]    Compiling libc v0.2.175
[INFO] [stderr]    Compiling unicode-ident v1.0.19
[INFO] [stderr]     Checking cmov v0.5.4
[INFO] [stderr]    Compiling num-traits v0.2.19
[INFO] [stderr]     Checking crypto-common v0.1.6
[INFO] [stderr]    Compiling find-msvc-tools v0.1.1
[INFO] [stderr]    Compiling same-file v1.0.6
[INFO] [stderr]    Compiling zerocopy v0.8.27
[INFO] [stderr]    Compiling proc-macro2 v1.0.101
[INFO] [stderr]     Checking digest v0.10.7
[INFO] [stderr]    Compiling cc v1.2.37
[INFO] [stderr]     Checking ctutils v0.4.2
[INFO] [stderr]    Compiling walkdir v2.5.0
[INFO] [stderr]     Checking digest v0.9.0
[INFO] [stderr]     Checking cipher v0.3.0
[INFO] [stderr]     Checking hybrid-array v0.4.14
[INFO] [stderr]     Checking inout v0.1.4
[INFO] [stderr]    Compiling winapi v0.3.9
[INFO] [stderr]    Compiling serde_core v1.0.225
[INFO] [stderr]    Compiling quote v1.0.40
[INFO] [stderr]     Checking cipher v0.4.4
[INFO] [stderr]     Checking crypto-mac v0.11.1
[INFO] [stderr]     Checking block-buffer v0.9.0
[INFO] [stderr]     Checking getrandom v0.3.3
[INFO] [stderr]     Checking socket2 v0.6.0
[INFO] [stderr]    Compiling syn v2.0.106
[INFO] [stderr]     Checking rand_core v0.9.3
[INFO] [stderr]     Checking num-integer v0.1.46
[INFO] [stderr]     Checking mio v1.0.4
[INFO] [stderr]     Checking signal-hook-registry v1.4.6
[INFO] [stderr]     Checking log v0.4.28
[INFO] [stderr]     Checking base64ct v1.8.0
[INFO] [stderr]     Checking rand_core v0.6.4
[INFO] [stderr]    Compiling serde v1.0.225
[INFO] [stderr]     Checking num-bigint v0.4.6
[INFO] [stderr]     Checking password-hash v0.2.3
[INFO] [stderr]     Checking blowfish v0.9.1
[INFO] [stderr]     Checking regex-automata v0.4.10
[INFO] [stderr]     Checking sha2 v0.9.9
[INFO] [stderr]    Compiling libsodium-sys v0.2.7
[INFO] [stderr]     Checking hmac v0.11.0
[INFO] [stderr]     Checking dirs-sys v0.3.7
[INFO] [stderr]     Checking ppv-lite86 v0.2.21
[INFO] [stderr]     Checking crypto-common v0.2.2
[INFO] [stderr]     Checking block-buffer v0.12.1
[INFO] [stderr]     Checking ctr v0.8.0
[INFO] [stderr]     Checking pbkdf2 v0.12.2
[INFO] [stderr]     Checking sha2 v0.10.9
[INFO] [stderr]     Checking rand_chacha v0.9.0
[INFO] [stderr]     Checking keccak v0.2.1
[INFO] [stderr]     Checking block-padding v0.2.1
[INFO] [stderr]     Checking bit-vec v0.6.3
[INFO] [stderr]     Checking rand v0.9.2
[INFO] [stderr]     Checking block-modes v0.8.1
[INFO] [stderr]     Checking bcrypt-pbkdf v0.10.0
[INFO] [stderr]     Checking ssh-libsodium v0.4.0
[INFO] [stderr]     Checking aes v0.7.5
[INFO] [stderr]     Checking pbkdf2 v0.8.0
[INFO] [stderr]     Checking yasna v0.4.0
[INFO] [stderr]     Checking cryptovec v0.7.1
[INFO] [stderr]     Checking module-lattice v0.2.3
[INFO] [stderr]     Checking digest v0.11.3
[INFO] [stderr]     Checking sha3 v0.11.0
[INFO] [stderr]     Checking kem v0.3.0
[INFO] [stderr]     Checking dirs v3.0.2
[INFO] [stderr]     Checking crc32fast v1.5.0
[INFO] [stderr]     Checking anstream v0.6.20
[INFO] [stderr]     Checking jiff v0.2.15
[INFO] [stderr]     Checking flate2 v1.1.2
[INFO] [stderr]     Checking ml-kem v0.3.2
[INFO] [stderr]     Checking regex v1.11.2
[INFO] [stderr]     Checking env_filter v0.1.3
[INFO] [stderr]    Compiling futures-macro v0.3.31
[INFO] [stderr]    Compiling tokio-macros v2.5.0
[INFO] [stderr]    Compiling thiserror-impl v1.0.69
[INFO] [stderr]    Compiling serde_derive v1.0.225
[INFO] [stderr]     Checking tokio v1.47.1
[INFO] [stderr]     Checking futures-util v0.3.31
[INFO] [stderr]     Checking thiserror v1.0.69
[INFO] [stderr]     Checking env_logger v0.11.8
[INFO] [stderr]     Checking futures-executor v0.3.31
[INFO] [stderr]     Checking futures v0.3.31
[INFO] [stderr]     Checking tokio-stream v0.1.17
[INFO] [stderr]     Checking thrussh-keys v0.44.0
[INFO] [stderr]     Checking thrussh v0.49.0 (/opt/rustwide/workdir)
[INFO] [stdout] warning: unused import: `super::super::*`
[INFO] [stdout]   --> src/server/encrypted.rs:15:5
[INFO] [stdout]    |
[INFO] [stdout] 15 | use super::super::*;
[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 import: `super::super::*`
[INFO] [stdout]   --> src/server/encrypted.rs:15:5
[INFO] [stdout]    |
[INFO] [stdout] 15 | use super::super::*;
[INFO] [stdout]    |     ^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(unused_imports)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of deprecated constant `std::u32::MAX`: replaced by the `MAX` associated constant on this type
[INFO] [stdout]    --> src/cipher/mod.rs:215:44
[INFO] [stdout]     |
[INFO] [stdout] 215 |         assert!(packet_length <= std::u32::MAX as usize);
[INFO] [stdout]     |                                            ^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: `#[warn(deprecated)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of deprecated constant `std::u8::MAX`: replaced by the `MAX` associated constant on this type
[INFO] [stdout]    --> src/cipher/mod.rs:218:44
[INFO] [stdout]     |
[INFO] [stdout] 218 |         assert!(padding_length <= std::u8::MAX as usize);
[INFO] [stdout]     |                                            ^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of deprecated constant `std::u32::MAX`: replaced by the `MAX` associated constant on this type
[INFO] [stdout]    --> src/cipher/mod.rs:215:44
[INFO] [stdout]     |
[INFO] [stdout] 215 |         assert!(packet_length <= std::u32::MAX as usize);
[INFO] [stdout]     |                                            ^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: `#[warn(deprecated)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of deprecated constant `std::u8::MAX`: replaced by the `MAX` associated constant on this type
[INFO] [stdout]    --> src/cipher/mod.rs:218:44
[INFO] [stdout]     |
[INFO] [stdout] 218 |         assert!(padding_length <= std::u8::MAX as usize);
[INFO] [stdout]     |                                            ^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]    --> src/negotiation.rs:194:24
[INFO] [stdout]     |
[INFO] [stdout] 194 |             return Err(Error::NoCommonKexAlgo.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] 194 -             return Err(Error::NoCommonKexAlgo.into());
[INFO] [stdout] 194 +             return Err(Error::NoCommonKexAlgo);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 194 -             return Err(Error::NoCommonKexAlgo.into());
[INFO] [stdout] 194 +             return Err(<Error as Into<_>>::into(Error::NoCommonKexAlgo));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]    --> src/negotiation.rs:231:24
[INFO] [stdout]     |
[INFO] [stdout] 231 |             return Err(Error::NoCommonKeyAlgo.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] 231 -             return Err(Error::NoCommonKeyAlgo.into());
[INFO] [stdout] 231 +             return Err(Error::NoCommonKeyAlgo);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 231 -             return Err(Error::NoCommonKeyAlgo.into());
[INFO] [stdout] 231 +             return Err(<Error as Into<_>>::into(Error::NoCommonKeyAlgo));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]    --> src/negotiation.rs:242:24
[INFO] [stdout]     |
[INFO] [stdout] 242 |             return Err(Error::NoCommonCipher.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] 242 -             return Err(Error::NoCommonCipher.into());
[INFO] [stdout] 242 +             return Err(Error::NoCommonCipher);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 242 -             return Err(Error::NoCommonCipher.into());
[INFO] [stdout] 242 +             return Err(<Error as Into<_>>::into(Error::NoCommonCipher));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]    --> src/negotiation.rs:254:24
[INFO] [stdout]     |
[INFO] [stdout] 254 |             return Err(Error::NoCommonMac.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] 254 -             return Err(Error::NoCommonMac.into());
[INFO] [stdout] 254 +             return Err(Error::NoCommonMac);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 254 -             return Err(Error::NoCommonMac.into());
[INFO] [stdout] 254 +             return Err(<Error as Into<_>>::into(Error::NoCommonMac));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]    --> src/negotiation.rs:265:28
[INFO] [stdout]     |
[INFO] [stdout] 265 |                 return Err(Error::NoCommonCompression.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] 265 -                 return Err(Error::NoCommonCompression.into());
[INFO] [stdout] 265 +                 return Err(Error::NoCommonCompression);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 265 -                 return Err(Error::NoCommonCompression.into());
[INFO] [stdout] 265 +                 return Err(<Error as Into<_>>::into(Error::NoCommonCompression));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]    --> src/negotiation.rs:273:28
[INFO] [stdout]     |
[INFO] [stdout] 273 |                 return Err(Error::NoCommonCompression.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] 273 -                 return Err(Error::NoCommonCompression.into());
[INFO] [stdout] 273 +                 return Err(Error::NoCommonCompression);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 273 -                 return Err(Error::NoCommonCompression.into());
[INFO] [stdout] 273 +                 return Err(<Error as Into<_>>::into(Error::NoCommonCompression));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]    --> src/negotiation.rs:296:22
[INFO] [stdout]     |
[INFO] [stdout] 296 |             _ => Err(Error::KexInit.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] 296 -             _ => Err(Error::KexInit.into()),
[INFO] [stdout] 296 +             _ => Err(Error::KexInit),
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 296 -             _ => Err(Error::KexInit.into()),
[INFO] [stdout] 296 +             _ => Err(<Error as Into<_>>::into(Error::KexInit)),
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]    --> src/ssh_read.rs:125:28
[INFO] [stdout]     |
[INFO] [stdout] 125 |                 return Err(Error::Disconnect.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] 125 -                 return Err(Error::Disconnect.into());
[INFO] [stdout] 125 +                 return Err(Error::Disconnect);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 125 -                 return Err(Error::Disconnect.into());
[INFO] [stdout] 125 +                 return Err(<Error as Into<_>>::into(Error::Disconnect));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]   --> src/server/kex.rs:43:28
[INFO] [stdout]    |
[INFO] [stdout] 43 |                 return Err(Error::UnknownKey.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] 43 -                 return Err(Error::UnknownKey.into());
[INFO] [stdout] 43 +                 return Err(Error::UnknownKey);
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 43 -                 return Err(Error::UnknownKey.into());
[INFO] [stdout] 43 +                 return Err(<Error as Into<_>>::into(Error::UnknownKey));
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]   --> src/client/session.rs:29:33
[INFO] [stdout]    |
[INFO] [stdout] 29 |                 _ => return Err(Error::NotAuthenticated.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] 29 -                 _ => return Err(Error::NotAuthenticated.into()),
[INFO] [stdout] 29 +                 _ => return Err(Error::NotAuthenticated),
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 29 -                 _ => return Err(Error::NotAuthenticated.into()),
[INFO] [stdout] 29 +                 _ => return Err(<Error as Into<_>>::into(Error::NotAuthenticated)),
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]   --> src/client/session.rs:32:24
[INFO] [stdout]    |
[INFO] [stdout] 32 |             return Err(Error::Inconsistent.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] 32 -             return Err(Error::Inconsistent.into());
[INFO] [stdout] 32 +             return Err(Error::Inconsistent);
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 32 -             return Err(Error::Inconsistent.into());
[INFO] [stdout] 32 +             return Err(<Error as Into<_>>::into(Error::Inconsistent));
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]   --> src/client/session.rs:69:33
[INFO] [stdout]    |
[INFO] [stdout] 69 |                 _ => return Err(Error::NotAuthenticated.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] 69 -                 _ => return Err(Error::NotAuthenticated.into()),
[INFO] [stdout] 69 +                 _ => return Err(Error::NotAuthenticated),
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 69 -                 _ => return Err(Error::NotAuthenticated.into()),
[INFO] [stdout] 69 +                 _ => return Err(<Error as Into<_>>::into(Error::NotAuthenticated)),
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]   --> src/client/session.rs:72:24
[INFO] [stdout]    |
[INFO] [stdout] 72 |             return Err(Error::Inconsistent.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] 72 -             return Err(Error::Inconsistent.into());
[INFO] [stdout] 72 +             return Err(Error::Inconsistent);
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 72 -             return Err(Error::Inconsistent.into());
[INFO] [stdout] 72 +             return Err(<Error as Into<_>>::into(Error::Inconsistent));
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]    --> src/client/session.rs:113:33
[INFO] [stdout]     |
[INFO] [stdout] 113 |                 _ => return Err(Error::NotAuthenticated.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] 113 -                 _ => return Err(Error::NotAuthenticated.into()),
[INFO] [stdout] 113 +                 _ => return Err(Error::NotAuthenticated),
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 113 -                 _ => return Err(Error::NotAuthenticated.into()),
[INFO] [stdout] 113 +                 _ => return Err(<Error as Into<_>>::into(Error::NotAuthenticated)),
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]    --> src/client/session.rs:116:24
[INFO] [stdout]     |
[INFO] [stdout] 116 |             return Err(Error::Inconsistent.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] 116 -             return Err(Error::Inconsistent.into());
[INFO] [stdout] 116 +             return Err(Error::Inconsistent);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 116 -             return Err(Error::Inconsistent.into());
[INFO] [stdout] 116 +             return Err(<Error as Into<_>>::into(Error::Inconsistent));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `<S as Signer>::Error`
[INFO] [stdout]    --> src/client/mod.rs:292:55
[INFO] [stdout]     |
[INFO] [stdout] 292 |                         Err(e) => return (future, Err(e.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] 292 -                         Err(e) => return (future, Err(e.into())),
[INFO] [stdout] 292 +                         Err(e) => return (future, Err(e)),
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 292 -                         Err(e) => return (future, Err(e.into())),
[INFO] [stdout] 292 +                         Err(e) => return (future, Err(<<S as Signer>::Error as Into<_>>::into(e))),
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]    --> src/client/mod.rs:326:32
[INFO] [stdout]     |
[INFO] [stdout] 326 |                     return Err(Error::Disconnect.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] 326 -                     return Err(Error::Disconnect.into());
[INFO] [stdout] 326 +                     return Err(Error::Disconnect);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 326 -                     return Err(Error::Disconnect.into());
[INFO] [stdout] 326 +                     return Err(<Error as Into<_>>::into(Error::Disconnect));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr] error: could not compile `thrussh` (lib test) due to 17 previous errors; 3 warnings emitted
[INFO] [stderr] warning: build failed, waiting for other jobs to finish...
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]    --> src/negotiation.rs:194:24
[INFO] [stdout]     |
[INFO] [stdout] 194 |             return Err(Error::NoCommonKexAlgo.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] 194 -             return Err(Error::NoCommonKexAlgo.into());
[INFO] [stdout] 194 +             return Err(Error::NoCommonKexAlgo);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 194 -             return Err(Error::NoCommonKexAlgo.into());
[INFO] [stdout] 194 +             return Err(<Error as Into<_>>::into(Error::NoCommonKexAlgo));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]    --> src/negotiation.rs:231:24
[INFO] [stdout]     |
[INFO] [stdout] 231 |             return Err(Error::NoCommonKeyAlgo.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] 231 -             return Err(Error::NoCommonKeyAlgo.into());
[INFO] [stdout] 231 +             return Err(Error::NoCommonKeyAlgo);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 231 -             return Err(Error::NoCommonKeyAlgo.into());
[INFO] [stdout] 231 +             return Err(<Error as Into<_>>::into(Error::NoCommonKeyAlgo));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]    --> src/negotiation.rs:242:24
[INFO] [stdout]     |
[INFO] [stdout] 242 |             return Err(Error::NoCommonCipher.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] 242 -             return Err(Error::NoCommonCipher.into());
[INFO] [stdout] 242 +             return Err(Error::NoCommonCipher);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 242 -             return Err(Error::NoCommonCipher.into());
[INFO] [stdout] 242 +             return Err(<Error as Into<_>>::into(Error::NoCommonCipher));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]    --> src/negotiation.rs:254:24
[INFO] [stdout]     |
[INFO] [stdout] 254 |             return Err(Error::NoCommonMac.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] 254 -             return Err(Error::NoCommonMac.into());
[INFO] [stdout] 254 +             return Err(Error::NoCommonMac);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 254 -             return Err(Error::NoCommonMac.into());
[INFO] [stdout] 254 +             return Err(<Error as Into<_>>::into(Error::NoCommonMac));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]    --> src/negotiation.rs:265:28
[INFO] [stdout]     |
[INFO] [stdout] 265 |                 return Err(Error::NoCommonCompression.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] 265 -                 return Err(Error::NoCommonCompression.into());
[INFO] [stdout] 265 +                 return Err(Error::NoCommonCompression);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 265 -                 return Err(Error::NoCommonCompression.into());
[INFO] [stdout] 265 +                 return Err(<Error as Into<_>>::into(Error::NoCommonCompression));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]    --> src/negotiation.rs:273:28
[INFO] [stdout]     |
[INFO] [stdout] 273 |                 return Err(Error::NoCommonCompression.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] 273 -                 return Err(Error::NoCommonCompression.into());
[INFO] [stdout] 273 +                 return Err(Error::NoCommonCompression);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 273 -                 return Err(Error::NoCommonCompression.into());
[INFO] [stdout] 273 +                 return Err(<Error as Into<_>>::into(Error::NoCommonCompression));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]    --> src/negotiation.rs:296:22
[INFO] [stdout]     |
[INFO] [stdout] 296 |             _ => Err(Error::KexInit.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] 296 -             _ => Err(Error::KexInit.into()),
[INFO] [stdout] 296 +             _ => Err(Error::KexInit),
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 296 -             _ => Err(Error::KexInit.into()),
[INFO] [stdout] 296 +             _ => Err(<Error as Into<_>>::into(Error::KexInit)),
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]    --> src/ssh_read.rs:125:28
[INFO] [stdout]     |
[INFO] [stdout] 125 |                 return Err(Error::Disconnect.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] 125 -                 return Err(Error::Disconnect.into());
[INFO] [stdout] 125 +                 return Err(Error::Disconnect);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 125 -                 return Err(Error::Disconnect.into());
[INFO] [stdout] 125 +                 return Err(<Error as Into<_>>::into(Error::Disconnect));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]   --> src/server/kex.rs:43:28
[INFO] [stdout]    |
[INFO] [stdout] 43 |                 return Err(Error::UnknownKey.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] 43 -                 return Err(Error::UnknownKey.into());
[INFO] [stdout] 43 +                 return Err(Error::UnknownKey);
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 43 -                 return Err(Error::UnknownKey.into());
[INFO] [stdout] 43 +                 return Err(<Error as Into<_>>::into(Error::UnknownKey));
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]   --> src/client/session.rs:29:33
[INFO] [stdout]    |
[INFO] [stdout] 29 |                 _ => return Err(Error::NotAuthenticated.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] 29 -                 _ => return Err(Error::NotAuthenticated.into()),
[INFO] [stdout] 29 +                 _ => return Err(Error::NotAuthenticated),
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 29 -                 _ => return Err(Error::NotAuthenticated.into()),
[INFO] [stdout] 29 +                 _ => return Err(<Error as Into<_>>::into(Error::NotAuthenticated)),
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]   --> src/client/session.rs:32:24
[INFO] [stdout]    |
[INFO] [stdout] 32 |             return Err(Error::Inconsistent.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] 32 -             return Err(Error::Inconsistent.into());
[INFO] [stdout] 32 +             return Err(Error::Inconsistent);
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 32 -             return Err(Error::Inconsistent.into());
[INFO] [stdout] 32 +             return Err(<Error as Into<_>>::into(Error::Inconsistent));
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]   --> src/client/session.rs:69:33
[INFO] [stdout]    |
[INFO] [stdout] 69 |                 _ => return Err(Error::NotAuthenticated.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] 69 -                 _ => return Err(Error::NotAuthenticated.into()),
[INFO] [stdout] 69 +                 _ => return Err(Error::NotAuthenticated),
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 69 -                 _ => return Err(Error::NotAuthenticated.into()),
[INFO] [stdout] 69 +                 _ => return Err(<Error as Into<_>>::into(Error::NotAuthenticated)),
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]   --> src/client/session.rs:72:24
[INFO] [stdout]    |
[INFO] [stdout] 72 |             return Err(Error::Inconsistent.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] 72 -             return Err(Error::Inconsistent.into());
[INFO] [stdout] 72 +             return Err(Error::Inconsistent);
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 72 -             return Err(Error::Inconsistent.into());
[INFO] [stdout] 72 +             return Err(<Error as Into<_>>::into(Error::Inconsistent));
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]    --> src/client/session.rs:113:33
[INFO] [stdout]     |
[INFO] [stdout] 113 |                 _ => return Err(Error::NotAuthenticated.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] 113 -                 _ => return Err(Error::NotAuthenticated.into()),
[INFO] [stdout] 113 +                 _ => return Err(Error::NotAuthenticated),
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 113 -                 _ => return Err(Error::NotAuthenticated.into()),
[INFO] [stdout] 113 +                 _ => return Err(<Error as Into<_>>::into(Error::NotAuthenticated)),
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]    --> src/client/session.rs:116:24
[INFO] [stdout]     |
[INFO] [stdout] 116 |             return Err(Error::Inconsistent.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] 116 -             return Err(Error::Inconsistent.into());
[INFO] [stdout] 116 +             return Err(Error::Inconsistent);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 116 -             return Err(Error::Inconsistent.into());
[INFO] [stdout] 116 +             return Err(<Error as Into<_>>::into(Error::Inconsistent));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `<S as Signer>::Error`
[INFO] [stdout]    --> src/client/mod.rs:292:55
[INFO] [stdout]     |
[INFO] [stdout] 292 |                         Err(e) => return (future, Err(e.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] 292 -                         Err(e) => return (future, Err(e.into())),
[INFO] [stdout] 292 +                         Err(e) => return (future, Err(e)),
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 292 -                         Err(e) => return (future, Err(e.into())),
[INFO] [stdout] 292 +                         Err(e) => return (future, Err(<<S as Signer>::Error as Into<_>>::into(e))),
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]    --> src/client/mod.rs:326:32
[INFO] [stdout]     |
[INFO] [stdout] 326 |                     return Err(Error::Disconnect.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] 326 -                     return Err(Error::Disconnect.into());
[INFO] [stdout] 326 +                     return Err(Error::Disconnect);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 326 -                     return Err(Error::Disconnect.into());
[INFO] [stdout] 326 +                     return Err(<Error as Into<_>>::into(Error::Disconnect));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr] error: could not compile `thrussh` (lib) due to 17 previous errors; 3 warnings emitted
[INFO] running `Command { std: "docker" "inspect" "5105a6d38d1d3b9c7c63c6fb10b02acc33576f1db311f93160b91518cf15359c", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "5105a6d38d1d3b9c7c63c6fb10b02acc33576f1db311f93160b91518cf15359c", kill_on_drop: false }`
[INFO] [stdout] 5105a6d38d1d3b9c7c63c6fb10b02acc33576f1db311f93160b91518cf15359c
