[INFO] cloning repository https://github.com/ThomWright/tit
[INFO] running `Command { std: "git" "-c" "credential.helper=" "-c" "credential.helper=/workspace/cargo-home/bin/git-credential-null" "clone" "--bare" "https://github.com/ThomWright/tit" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2FThomWright%2Ftit", kill_on_drop: false }`
[INFO] [stderr] Cloning into bare repository '/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2FThomWright%2Ftit'...
[INFO] running `Command { std: "git" "rev-parse" "HEAD", kill_on_drop: false }`
[INFO] [stdout] 5f1743917eb569bdf3e865bfc811af72e933a5ce
[INFO] checking ThomWright/tit against a5689b034ad287112f709693384a37f1377eb9cc for pr-129249-3
[INFO] running `Command { std: "git" "clone" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2FThomWright%2Ftit" "/workspace/builds/worker-6-tc2/source", kill_on_drop: false }`
[INFO] [stderr] Cloning into '/workspace/builds/worker-6-tc2/source'...
[INFO] [stderr] done.
[INFO] started tweaking git repo https://github.com/ThomWright/tit
[INFO] finished tweaking git repo https://github.com/ThomWright/tit
[INFO] tweaked toml for git repo https://github.com/ThomWright/tit written to /workspace/builds/worker-6-tc2/source/Cargo.toml
[INFO] validating manifest of git repo https://github.com/ThomWright/tit 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/ThomWright/tit 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-6-tc2/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-6-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] 794f3b6b8e73d998b1e38911929d38ca70ede244c646d9e6380e7f458087675b
[INFO] running `Command { std: "docker" "start" "794f3b6b8e73d998b1e38911929d38ca70ede244c646d9e6380e7f458087675b", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "794f3b6b8e73d998b1e38911929d38ca70ede244c646d9e6380e7f458087675b", 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" "794f3b6b8e73d998b1e38911929d38ca70ede244c646d9e6380e7f458087675b" "/opt/rustwide/cargo-home/bin/cargo" "+a5689b034ad287112f709693384a37f1377eb9cc" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "794f3b6b8e73d998b1e38911929d38ca70ede244c646d9e6380e7f458087675b", 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" "794f3b6b8e73d998b1e38911929d38ca70ede244c646d9e6380e7f458087675b" "/opt/rustwide/cargo-home/bin/cargo" "+a5689b034ad287112f709693384a37f1377eb9cc" "check" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }`
[INFO] [stderr]    Compiling autocfg v1.0.1
[INFO] [stderr]    Compiling libc v0.2.80
[INFO] [stderr]    Compiling maybe-uninit v2.0.0
[INFO] [stderr]    Compiling log v0.4.11
[INFO] [stderr]    Compiling semver-parser v0.7.0
[INFO] [stderr]     Checking futures v0.1.30
[INFO] [stderr]    Compiling byteorder v1.3.4
[INFO] [stderr]     Checking slab v0.4.2
[INFO] [stderr]     Checking lock_api v0.3.4
[INFO] [stderr]    Compiling semver v0.9.0
[INFO] [stderr]     Checking smallvec v0.6.13
[INFO] [stderr]    Compiling cc v1.0.65
[INFO] [stderr]    Compiling arraydeque v0.4.5
[INFO] [stderr]    Compiling crossbeam-utils v0.7.2
[INFO] [stderr]    Compiling memoffset v0.5.6
[INFO] [stderr]    Compiling rustc_version v0.2.3
[INFO] [stderr]    Compiling crossbeam-epoch v0.8.2
[INFO] [stderr]    Compiling crossbeam-utils v0.8.1
[INFO] [stderr]     Checking scoped-tls v0.1.2
[INFO] [stderr]     Checking etherparse v0.9.0
[INFO] [stderr]    Compiling parking_lot_core v0.6.2
[INFO] [stderr]     Checking iovec v0.1.4
[INFO] [stderr]     Checking bytes v0.4.12
[INFO] [stderr]     Checking num_cpus v1.13.0
[INFO] [stderr]     Checking net2 v0.2.36
[INFO] [stderr]    Compiling parking_lot v0.9.0
[INFO] [stderr]     Checking mio v0.6.22
[INFO] [stderr]     Checking crossbeam-queue v0.2.3
[INFO] [stderr]     Checking crossbeam-deque v0.7.3
[INFO] [stderr]     Checking ansi_term v0.12.1
[INFO] [stderr]    Compiling tun-tap v0.1.2
[INFO] [stderr]     Checking crossbeam-channel v0.5.0
[INFO] [stderr]     Checking tokio-executor v0.1.10
[INFO] [stderr]     Checking tokio-io v0.1.13
[INFO] [stderr]     Checking tokio-sync v0.1.8
[INFO] [stderr]     Checking mio-uds v0.6.8
[INFO] [stderr]     Checking tokio-threadpool v0.1.18
[INFO] [stderr]     Checking tokio-timer v0.2.13
[INFO] [stderr]     Checking tokio-current-thread v0.1.7
[INFO] [stderr]     Checking tokio-reactor v0.1.12
[INFO] [stderr]     Checking tokio-codec v0.1.2
[INFO] [stderr]     Checking tokio-fs v0.1.7
[INFO] [stderr]     Checking tokio-tcp v0.1.4
[INFO] [stderr]     Checking tokio-udp v0.1.6
[INFO] [stderr]     Checking tokio-uds v0.2.7
[INFO] [stderr]     Checking tokio v0.1.22
[INFO] [stderr]     Checking tokio-core v0.1.17
[INFO] [stderr]     Checking tit v0.1.0 (/opt/rustwide/workdir)
[INFO] [stdout] warning: use of deprecated constant `std::u32::MAX`: replaced by the `MAX` associated constant on this type
[INFO] [stdout]     --> src/tcp/connection.rs:1034:33
[INFO] [stdout]      |
[INFO] [stdout] 1034 |         let snd_una = std::u32::MAX - 5;
[INFO] [stdout]      |                                 ^^^
[INFO] [stdout]      |
[INFO] [stdout]      = note: `#[warn(deprecated)]` 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/tcp/connection.rs:1036:33
[INFO] [stdout]      |
[INFO] [stdout] 1036 |         let ack_num = std::u32::MAX - 1;
[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/tcp/connection.rs:1042:33
[INFO] [stdout]      |
[INFO] [stdout] 1042 |         let snd_una = std::u32::MAX - 5;
[INFO] [stdout]      |                                 ^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: field `0` is never read
[INFO] [stdout]   --> src/errors.rs:33:25
[INFO] [stdout]    |
[INFO] [stdout] 33 | pub struct TcpSendError(crossbeam_channel::SendError<tcp::TcpPacket>);
[INFO] [stdout]    |            ------------ ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |            |
[INFO] [stdout]    |            field in this struct
[INFO] [stdout]    |
[INFO] [stdout]    = help: consider removing this field
[INFO] [stdout]    = note: `TcpSendError` 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] 
[INFO] [stdout] 
[INFO] [stdout] warning: variant `Active` is never constructed
[INFO] [stdout]   --> src/tcp/connection.rs:99:5
[INFO] [stdout]    |
[INFO] [stdout] 97 | enum OpenType {
[INFO] [stdout]    |      -------- variant in this enum
[INFO] [stdout] 98 |     Passive,
[INFO] [stdout] 99 |     Active,
[INFO] [stdout]    |     ^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `OpenType` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `crossbeam_channel::SendError<TcpCommand>`
[INFO] [stdout]   --> src/tcp/interfaces.rs:34:58
[INFO] [stdout]    |
[INFO] [stdout] 34 |             .map_err(|e| TitError::TcpCommandSendFailure(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]    = note: `#[deny(self_type_conversion)]` on by default
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]    |
[INFO] [stdout] 34 -             .map_err(|e| TitError::TcpCommandSendFailure(e.into()))?;
[INFO] [stdout] 34 +             .map_err(|e| TitError::TcpCommandSendFailure(e))?;
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 34 -             .map_err(|e| TitError::TcpCommandSendFailure(e.into()))?;
[INFO] [stdout] 34 +             .map_err(|e| TitError::TcpCommandSendFailure(<crossbeam_channel::SendError<TcpCommand> as Into<_>>::into(e)))?;
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `TcpError`
[INFO] [stdout]    --> src/tcp/tcp.rs:180:31
[INFO] [stdout]     |
[INFO] [stdout] 180 |                     .send(Err(TcpError::NoConnection.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] 180 -                     .send(Err(TcpError::NoConnection.into()))
[INFO] [stdout] 180 +                     .send(Err(TcpError::NoConnection))
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 180 -                     .send(Err(TcpError::NoConnection.into()))
[INFO] [stdout] 180 +                     .send(Err(<TcpError as Into<_>>::into(TcpError::NoConnection)))
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr] error: could not compile `tit` (lib) due to 2 previous errors; 2 warnings emitted
[INFO] [stderr] warning: build failed, waiting for other jobs to finish...
[INFO] [stdout] warning: field `0` is never read
[INFO] [stdout]   --> src/errors.rs:33:25
[INFO] [stdout]    |
[INFO] [stdout] 33 | pub struct TcpSendError(crossbeam_channel::SendError<tcp::TcpPacket>);
[INFO] [stdout]    |            ------------ ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |            |
[INFO] [stdout]    |            field in this struct
[INFO] [stdout]    |
[INFO] [stdout]    = help: consider removing this field
[INFO] [stdout]    = note: `TcpSendError` 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] 
[INFO] [stdout] 
[INFO] [stdout] warning: variant `Active` is never constructed
[INFO] [stdout]   --> src/tcp/connection.rs:99:5
[INFO] [stdout]    |
[INFO] [stdout] 97 | enum OpenType {
[INFO] [stdout]    |      -------- variant in this enum
[INFO] [stdout] 98 |     Passive,
[INFO] [stdout] 99 |     Active,
[INFO] [stdout]    |     ^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `OpenType` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `crossbeam_channel::SendError<TcpCommand>`
[INFO] [stdout]   --> src/tcp/interfaces.rs:34:58
[INFO] [stdout]    |
[INFO] [stdout] 34 |             .map_err(|e| TitError::TcpCommandSendFailure(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]    = note: `#[deny(self_type_conversion)]` on by default
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]    |
[INFO] [stdout] 34 -             .map_err(|e| TitError::TcpCommandSendFailure(e.into()))?;
[INFO] [stdout] 34 +             .map_err(|e| TitError::TcpCommandSendFailure(e))?;
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 34 -             .map_err(|e| TitError::TcpCommandSendFailure(e.into()))?;
[INFO] [stdout] 34 +             .map_err(|e| TitError::TcpCommandSendFailure(<crossbeam_channel::SendError<TcpCommand> as Into<_>>::into(e)))?;
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `TcpError`
[INFO] [stdout]    --> src/tcp/tcp.rs:180:31
[INFO] [stdout]     |
[INFO] [stdout] 180 |                     .send(Err(TcpError::NoConnection.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] 180 -                     .send(Err(TcpError::NoConnection.into()))
[INFO] [stdout] 180 +                     .send(Err(TcpError::NoConnection))
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 180 -                     .send(Err(TcpError::NoConnection.into()))
[INFO] [stdout] 180 +                     .send(Err(<TcpError as Into<_>>::into(TcpError::NoConnection)))
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr] error: could not compile `tit` (lib test) due to 2 previous errors; 5 warnings emitted
[INFO] running `Command { std: "docker" "inspect" "794f3b6b8e73d998b1e38911929d38ca70ede244c646d9e6380e7f458087675b", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "794f3b6b8e73d998b1e38911929d38ca70ede244c646d9e6380e7f458087675b", kill_on_drop: false }`
[INFO] [stdout] 794f3b6b8e73d998b1e38911929d38ca70ede244c646d9e6380e7f458087675b
