[INFO] fetching crate adb-rs 0.1.3...
[INFO] checking adb-rs-0.1.3 against master#3f1be1ec7ec3d8e80beb381ee82164a0aa3ca777 for pr-132289
[INFO] extracting crate adb-rs 0.1.3 into /workspace/builds/worker-2-tc1/source
[INFO] validating manifest of crates.io crate adb-rs 0.1.3 on toolchain 3f1be1ec7ec3d8e80beb381ee82164a0aa3ca777
[INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+3f1be1ec7ec3d8e80beb381ee82164a0aa3ca777" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }`
[INFO] started tweaking crates.io crate adb-rs 0.1.3
[INFO] finished tweaking crates.io crate adb-rs 0.1.3
[INFO] tweaked toml for crates.io crate adb-rs 0.1.3 written to /workspace/builds/worker-2-tc1/source/Cargo.toml
[INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+3f1be1ec7ec3d8e80beb381ee82164a0aa3ca777" "generate-lockfile" "--manifest-path" "Cargo.toml", kill_on_drop: false }`
[INFO] [stderr]     Updating crates.io index
[INFO] [stderr]      Locking 43 packages to latest compatible versions
[INFO] [stderr]       Adding bytes v0.4.12 (available: v1.8.0)
[INFO] [stderr]       Adding crossbeam-channel v0.3.9 (available: v0.5.13)
[INFO] [stderr]       Adding num-derive v0.2.5 (available: v0.4.2)
[INFO] [stderr]       Adding pretty-hex v0.1.1 (available: v0.4.1)
[INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+3f1be1ec7ec3d8e80beb381ee82164a0aa3ca777" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }`
[INFO] [stderr]  Downloading crates ...
[INFO] [stderr]   Downloaded pretty-hex v0.1.1
[INFO] [stderr]   Downloaded crossbeam-channel v0.3.9
[INFO] [stderr]   Downloaded num-derive v0.2.5
[INFO] [stderr]   Downloaded crossbeam-utils v0.6.6
[INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-2-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-2-tc1/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:4a844ea9eb2546a2d2c7022eacef16ef2e8229c7fbb2c7d4d55a9ceca922f72d" "/opt/rustwide/cargo-home/bin/cargo" "+3f1be1ec7ec3d8e80beb381ee82164a0aa3ca777" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }`
[INFO] [stdout] b019a762926eb7c0502e6659b06cdb5160b102a955e9f77e7da9c583abf9034d
[INFO] running `Command { std: "docker" "start" "-a" "b019a762926eb7c0502e6659b06cdb5160b102a955e9f77e7da9c583abf9034d", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "b019a762926eb7c0502e6659b06cdb5160b102a955e9f77e7da9c583abf9034d", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "b019a762926eb7c0502e6659b06cdb5160b102a955e9f77e7da9c583abf9034d", kill_on_drop: false }`
[INFO] [stdout] b019a762926eb7c0502e6659b06cdb5160b102a955e9f77e7da9c583abf9034d
[INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-2-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-2-tc1/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:4a844ea9eb2546a2d2c7022eacef16ef2e8229c7fbb2c7d4d55a9ceca922f72d" "/opt/rustwide/cargo-home/bin/cargo" "+3f1be1ec7ec3d8e80beb381ee82164a0aa3ca777" "check" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }`
[INFO] [stdout] f35cccf90e7be9228fc1c0fbbac1d7bd1658a69bb56dcaee656d035625061361
[INFO] running `Command { std: "docker" "start" "-a" "f35cccf90e7be9228fc1c0fbbac1d7bd1658a69bb56dcaee656d035625061361", kill_on_drop: false }`
[INFO] [stderr]    Compiling syn v1.0.109
[INFO] [stderr]    Compiling proc-macro2 v0.4.30
[INFO] [stderr]    Compiling unicode-xid v0.1.0
[INFO] [stderr]     Checking memchr v2.7.4
[INFO] [stderr]    Compiling failure_derive v0.1.8
[INFO] [stderr]    Compiling syn v0.15.44
[INFO] [stderr]    Compiling num-derive v0.2.5
[INFO] [stderr]    Compiling num-traits v0.2.19
[INFO] [stderr]     Checking crossbeam-utils v0.6.6
[INFO] [stderr]     Checking byteorder v1.5.0
[INFO] [stderr]     Checking iovec v0.1.4
[INFO] [stderr]     Checking pretty-hex v0.1.1
[INFO] [stderr]     Checking bytes v0.4.12
[INFO] [stderr]     Checking crossbeam-channel v0.3.9
[INFO] [stderr]     Checking object v0.36.5
[INFO] [stderr]    Compiling quote v0.6.13
[INFO] [stderr]    Compiling synstructure v0.12.6
[INFO] [stderr]     Checking backtrace v0.3.74
[INFO] [stderr]     Checking failure v0.1.8
[INFO] [stderr]     Checking adb-rs v0.1.3 (/opt/rustwide/workdir)
[INFO] [stdout] warning: field `magic` is never read
[INFO] [stdout]   --> src/message/mod.rs:32:7
[INFO] [stdout]    |
[INFO] [stdout] 26 | pub struct Header {
[INFO] [stdout]    |            ------ field in this struct
[INFO] [stdout] ...
[INFO] [stdout] 32 |   pub magic: u32,
[INFO] [stdout]    |       ^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `Header` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis
[INFO] [stdout]    = note: `#[warn(dead_code)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: fields `local_id` and `remote_id` are never read
[INFO] [stdout]    --> src/client.rs:272:3
[INFO] [stdout]     |
[INFO] [stdout] 271 | struct StreamContext {
[INFO] [stdout]     |        ------------- fields in this struct
[INFO] [stdout] 272 |   local_id: u32,
[INFO] [stdout]     |   ^^^^^^^^
[INFO] [stdout] 273 |   remote_id: u32,
[INFO] [stdout]     |   ^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: `StreamContext` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: fields `system_identity`, `device_system_identity`, and `device_version` are never read
[INFO] [stdout]    --> src/client.rs:280:3
[INFO] [stdout]     |
[INFO] [stdout] 279 | pub struct AdbConnection {
[INFO] [stdout]     |            ------------- fields in this struct
[INFO] [stdout] 280 |   system_identity: String,
[INFO] [stdout]     |   ^^^^^^^^^^^^^^^
[INFO] [stdout] 281 |   device_system_identity: String,
[INFO] [stdout]     |   ^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 282 |   device_version: u32,
[INFO] [stdout]     |   ^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: `AdbConnection` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item
[INFO] [stdout]  --> src/result.rs:3:17
[INFO] [stdout]   |
[INFO] [stdout] 3 | #[derive(Debug, Fail)]
[INFO] [stdout]   |                 ^---
[INFO] [stdout]   |                 |
[INFO] [stdout]   |                 `Fail` is not local
[INFO] [stdout]   |                 move the `impl` block outside of this constant `_DERIVE_failure_Fail_FOR_AdbError`
[INFO] [stdout] 4 | pub enum AdbError {
[INFO] [stdout]   |          -------- `AdbError` is not local
[INFO] [stdout]   |
[INFO] [stdout]   = note: the derive macro `Fail` defines the non-local `impl`, and may need to be changed
[INFO] [stdout]   = note: the derive macro `Fail` may come from an old version of the `failure_derive` crate, try updating your dependency with `cargo update -p failure_derive`
[INFO] [stdout]   = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl`
[INFO] [stdout]   = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint
[INFO] [stdout]   = note: `#[warn(non_local_definitions)]` on by default
[INFO] [stdout]   = note: this warning originates in the derive macro `Fail` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: field `magic` is never read
[INFO] [stdout]   --> src/message/mod.rs:32:7
[INFO] [stdout]    |
[INFO] [stdout] 26 | pub struct Header {
[INFO] [stdout]    |            ------ field in this struct
[INFO] [stdout] ...
[INFO] [stdout] 32 |   pub magic: u32,
[INFO] [stdout]    |       ^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `Header` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis
[INFO] [stdout]    = note: `#[warn(dead_code)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item
[INFO] [stdout]  --> src/result.rs:3:17
[INFO] [stdout]   |
[INFO] [stdout] 3 | #[derive(Debug, Fail)]
[INFO] [stdout]   |                 ^---
[INFO] [stdout]   |                 |
[INFO] [stdout]   |                 `Display` is not local
[INFO] [stdout]   |                 move the `impl` block outside of this constant `_DERIVE_failure_core_fmt_Display_FOR_AdbError`
[INFO] [stdout] 4 | pub enum AdbError {
[INFO] [stdout]   |          -------- `AdbError` is not local
[INFO] [stdout]   |
[INFO] [stdout]   = note: the derive macro `Fail` defines the non-local `impl`, and may need to be changed
[INFO] [stdout]   = note: the derive macro `Fail` may come from an old version of the `failure_derive` crate, try updating your dependency with `cargo update -p failure_derive`
[INFO] [stdout]   = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl`
[INFO] [stdout]   = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint
[INFO] [stdout]   = note: this warning originates in the derive macro `Fail` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: fields `local_id` and `remote_id` are never read
[INFO] [stdout]    --> src/client.rs:272:3
[INFO] [stdout]     |
[INFO] [stdout] 271 | struct StreamContext {
[INFO] [stdout]     |        ------------- fields in this struct
[INFO] [stdout] 272 |   local_id: u32,
[INFO] [stdout]     |   ^^^^^^^^
[INFO] [stdout] 273 |   remote_id: u32,
[INFO] [stdout]     |   ^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: `StreamContext` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item
[INFO] [stdout]   --> src/message/mod.rs:14:30
[INFO] [stdout]    |
[INFO] [stdout] 14 | #[derive(Debug, Copy, Clone, FromPrimitive, PartialEq)]
[INFO] [stdout]    |                              ^------------
[INFO] [stdout]    |                              |
[INFO] [stdout]    |                              `FromPrimitive` is not local
[INFO] [stdout]    |                              move the `impl` block outside of this constant `_IMPL_NUM_FromPrimitive_FOR_Command`
[INFO] [stdout] 15 | pub enum Command {
[INFO] [stdout]    |          ------- `Command` is not local
[INFO] [stdout]    |
[INFO] [stdout]    = note: the derive macro `FromPrimitive` defines the non-local `impl`, and may need to be changed
[INFO] [stdout]    = note: the derive macro `FromPrimitive` may come from an old version of the `num_derive` crate, try updating your dependency with `cargo update -p num_derive`
[INFO] [stdout]    = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl`
[INFO] [stdout]    = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint
[INFO] [stdout]    = note: this warning originates in the derive macro `FromPrimitive` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: fields `system_identity`, `device_system_identity`, and `device_version` are never read
[INFO] [stdout]    --> src/client.rs:280:3
[INFO] [stdout]     |
[INFO] [stdout] 279 | pub struct AdbConnection {
[INFO] [stdout]     |            ------------- fields in this struct
[INFO] [stdout] 280 |   system_identity: String,
[INFO] [stdout]     |   ^^^^^^^^^^^^^^^
[INFO] [stdout] 281 |   device_system_identity: String,
[INFO] [stdout]     |   ^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 282 |   device_version: u32,
[INFO] [stdout]     |   ^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: `AdbConnection` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item
[INFO] [stdout]  --> src/result.rs:3:17
[INFO] [stdout]   |
[INFO] [stdout] 3 | #[derive(Debug, Fail)]
[INFO] [stdout]   |                 ^---
[INFO] [stdout]   |                 |
[INFO] [stdout]   |                 `Fail` is not local
[INFO] [stdout]   |                 move the `impl` block outside of this constant `_DERIVE_failure_Fail_FOR_AdbError`
[INFO] [stdout] 4 | pub enum AdbError {
[INFO] [stdout]   |          -------- `AdbError` is not local
[INFO] [stdout]   |
[INFO] [stdout]   = note: the derive macro `Fail` defines the non-local `impl`, and may need to be changed
[INFO] [stdout]   = note: the derive macro `Fail` may come from an old version of the `failure_derive` crate, try updating your dependency with `cargo update -p failure_derive`
[INFO] [stdout]   = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl`
[INFO] [stdout]   = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint
[INFO] [stdout]   = note: `#[warn(non_local_definitions)]` on by default
[INFO] [stdout]   = note: this warning originates in the derive macro `Fail` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item
[INFO] [stdout]  --> src/result.rs:3:17
[INFO] [stdout]   |
[INFO] [stdout] 3 | #[derive(Debug, Fail)]
[INFO] [stdout]   |                 ^---
[INFO] [stdout]   |                 |
[INFO] [stdout]   |                 `Display` is not local
[INFO] [stdout]   |                 move the `impl` block outside of this constant `_DERIVE_failure_core_fmt_Display_FOR_AdbError`
[INFO] [stdout] 4 | pub enum AdbError {
[INFO] [stdout]   |          -------- `AdbError` is not local
[INFO] [stdout]   |
[INFO] [stdout]   = note: the derive macro `Fail` defines the non-local `impl`, and may need to be changed
[INFO] [stdout]   = note: the derive macro `Fail` may come from an old version of the `failure_derive` crate, try updating your dependency with `cargo update -p failure_derive`
[INFO] [stdout]   = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl`
[INFO] [stdout]   = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint
[INFO] [stdout]   = note: this warning originates in the derive macro `Fail` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: non-local `impl` definition, `impl` blocks should be written at the same level as their item
[INFO] [stdout]   --> src/message/mod.rs:14:30
[INFO] [stdout]    |
[INFO] [stdout] 14 | #[derive(Debug, Copy, Clone, FromPrimitive, PartialEq)]
[INFO] [stdout]    |                              ^------------
[INFO] [stdout]    |                              |
[INFO] [stdout]    |                              `FromPrimitive` is not local
[INFO] [stdout]    |                              move the `impl` block outside of this constant `_IMPL_NUM_FromPrimitive_FOR_Command`
[INFO] [stdout] 15 | pub enum Command {
[INFO] [stdout]    |          ------- `Command` is not local
[INFO] [stdout]    |
[INFO] [stdout]    = note: the derive macro `FromPrimitive` defines the non-local `impl`, and may need to be changed
[INFO] [stdout]    = note: the derive macro `FromPrimitive` may come from an old version of the `num_derive` crate, try updating your dependency with `cargo update -p num_derive`
[INFO] [stdout]    = note: an `impl` is never scoped, even when it is nested inside an item, as it may impact type checking outside of that item, which can be the case if neither the trait or the self type are at the same nesting level as the `impl`
[INFO] [stdout]    = note: items in an anonymous const item (`const _: () = { ... }`) are treated as in the same scope as the anonymous const's declaration for the purpose of this lint
[INFO] [stdout]    = note: this warning originates in the derive macro `FromPrimitive` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused return value of `std::mem::replace` that must be used
[INFO] [stdout]    --> src/client.rs:296:5
[INFO] [stdout]     |
[INFO] [stdout] 296 |     ::std::mem::replace(&mut self.conn_writer_s, conn_writer_s);
[INFO] [stdout]     |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: if you don't need the old value, you can just assign the new value directly
[INFO] [stdout]     = note: `#[warn(unused_must_use)]` on by default
[INFO] [stdout] help: use `let _ = ...` to ignore the resulting value
[INFO] [stdout]     |
[INFO] [stdout] 296 |     let _ = ::std::mem::replace(&mut self.conn_writer_s, conn_writer_s);
[INFO] [stdout]     |     +++++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused return value of `std::mem::replace` that must be used
[INFO] [stdout]    --> src/client.rs:296:5
[INFO] [stdout]     |
[INFO] [stdout] 296 |     ::std::mem::replace(&mut self.conn_writer_s, conn_writer_s);
[INFO] [stdout]     |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: if you don't need the old value, you can just assign the new value directly
[INFO] [stdout]     = note: `#[warn(unused_must_use)]` on by default
[INFO] [stdout] help: use `let _ = ...` to ignore the resulting value
[INFO] [stdout]     |
[INFO] [stdout] 296 |     let _ = ::std::mem::replace(&mut self.conn_writer_s, conn_writer_s);
[INFO] [stdout]     |     +++++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr]     Finished `dev` profile [unoptimized + debuginfo] target(s) in 3.23s
[INFO] running `Command { std: "docker" "inspect" "f35cccf90e7be9228fc1c0fbbac1d7bd1658a69bb56dcaee656d035625061361", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "f35cccf90e7be9228fc1c0fbbac1d7bd1658a69bb56dcaee656d035625061361", kill_on_drop: false }`
[INFO] [stdout] f35cccf90e7be9228fc1c0fbbac1d7bd1658a69bb56dcaee656d035625061361
