[INFO] fetching crate juliet 0.3.0... [INFO] checking juliet-0.3.0 against try#c659ee110de67e82444e4b6c8407c1a9af9c2cf6 for pr-145608-1 [INFO] extracting crate juliet 0.3.0 into /workspace/builds/worker-4-tc2/source [INFO] started tweaking crates.io crate juliet 0.3.0 [INFO] removed 0 missing examples [INFO] finished tweaking crates.io crate juliet 0.3.0 [INFO] tweaked toml for crates.io crate juliet 0.3.0 written to /workspace/builds/worker-4-tc2/source/Cargo.toml [INFO] validating manifest of crates.io crate juliet 0.3.0 on toolchain c659ee110de67e82444e4b6c8407c1a9af9c2cf6 [INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+c659ee110de67e82444e4b6c8407c1a9af9c2cf6" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }` [INFO] crate crates.io crate juliet 0.3.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" "+c659ee110de67e82444e4b6c8407c1a9af9c2cf6" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }` [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-4-tc2/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-4-tc2/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/cargo-home:/opt/rustwide/cargo-home:ro,Z" "-v" "/var/lib/crater-agent-workspace/rustup-home:/opt/rustwide/rustup-home:ro,Z" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-w" "/opt/rustwide/workdir" "-m" "1610612736" "--user" "0:0" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:7ad1b28ee6f5f7f699f6cf7015098d6ccdd96d6f2d78dd06228f5b4c9faf309c" "/opt/rustwide/cargo-home/bin/cargo" "+c659ee110de67e82444e4b6c8407c1a9af9c2cf6" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }` [INFO] [stdout] c2f8cc19dc12a90de62da543a20055e62d6c85cc84ff8035ab2f3ca1b5c00f2f [INFO] running `Command { std: "docker" "start" "-a" "c2f8cc19dc12a90de62da543a20055e62d6c85cc84ff8035ab2f3ca1b5c00f2f", kill_on_drop: false }` [INFO] running `Command { std: "docker" "inspect" "c2f8cc19dc12a90de62da543a20055e62d6c85cc84ff8035ab2f3ca1b5c00f2f", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "c2f8cc19dc12a90de62da543a20055e62d6c85cc84ff8035ab2f3ca1b5c00f2f", kill_on_drop: false }` [INFO] [stdout] c2f8cc19dc12a90de62da543a20055e62d6c85cc84ff8035ab2f3ca1b5c00f2f [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-4-tc2/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-4-tc2/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/cargo-home:/opt/rustwide/cargo-home:ro,Z" "-v" "/var/lib/crater-agent-workspace/rustup-home:/opt/rustwide/rustup-home:ro,Z" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-e" "CARGO_INCREMENTAL=0" "-e" "RUST_BACKTRACE=full" "-e" "RUSTFLAGS=--cap-lints=forbid" "-e" "RUSTDOCFLAGS=--cap-lints=forbid" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-w" "/opt/rustwide/workdir" "-m" "1610612736" "--user" "0:0" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:7ad1b28ee6f5f7f699f6cf7015098d6ccdd96d6f2d78dd06228f5b4c9faf309c" "/opt/rustwide/cargo-home/bin/cargo" "+c659ee110de67e82444e4b6c8407c1a9af9c2cf6" "check" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }` [INFO] [stdout] 45f8e303c54527a9eb06d4bf5895b607cb6f4bf703b98c50206c940fcdb1c8cc [INFO] running `Command { std: "docker" "start" "-a" "45f8e303c54527a9eb06d4bf5895b607cb6f4bf703b98c50206c940fcdb1c8cc", kill_on_drop: false }` [INFO] [stderr] Compiling proc-macro2 v1.0.69 [INFO] [stderr] Compiling futures-util v0.3.29 [INFO] [stderr] Compiling proc-macro2 v0.4.30 [INFO] [stderr] Compiling unicode-xid v0.1.0 [INFO] [stderr] Compiling libm v0.2.8 [INFO] [stderr] Checking tracing-core v0.1.32 [INFO] [stderr] Compiling num-traits v0.2.17 [INFO] [stderr] Compiling syn v1.0.109 [INFO] [stderr] Compiling syn v0.15.44 [INFO] [stderr] Checking regex-automata v0.4.3 [INFO] [stderr] Checking wait-timeout v0.2.0 [INFO] [stderr] Checking rusty-fork v0.3.0 [INFO] [stderr] Checking rand_xorshift v0.3.0 [INFO] [stderr] Checking bimap v0.6.3 [INFO] [stderr] Checking array-init v2.1.0 [INFO] [stderr] Checking assert_matches v1.5.0 [INFO] [stderr] Checking tracing-log v0.2.0 [INFO] [stderr] Compiling quote v1.0.33 [INFO] [stderr] Compiling quote v0.6.13 [INFO] [stderr] Compiling syn v2.0.39 [INFO] [stderr] Checking proptest v1.4.0 [INFO] [stderr] Checking regex v1.10.2 [INFO] [stderr] Compiling proptest-attr-macro v1.0.0 [INFO] [stderr] Compiling proptest-derive v0.3.0 [INFO] [stderr] Compiling futures-macro v0.3.29 [INFO] [stderr] Compiling tracing-attributes v0.1.27 [INFO] [stderr] Compiling tokio-macros v2.2.0 [INFO] [stderr] Compiling bytemuck_derive v1.5.0 [INFO] [stderr] Compiling strum_macros v0.25.3 [INFO] [stderr] Compiling thiserror-impl v1.0.50 [INFO] [stderr] Compiling derive_more-impl v1.0.0-beta.2 [INFO] [stderr] Checking tokio v1.34.0 [INFO] [stderr] Checking bytemuck v1.14.0 [INFO] [stderr] Checking tracing v0.1.40 [INFO] [stderr] Checking thiserror v1.0.50 [INFO] [stderr] Checking derive_more v1.0.0-beta.2 [INFO] [stderr] Checking tracing-subscriber v0.3.18 [INFO] [stderr] Checking strum v0.25.0 [INFO] [stderr] Checking futures-executor v0.3.29 [INFO] [stderr] Checking futures v0.3.29 [INFO] [stderr] Checking juliet v0.3.0 (/opt/rustwide/workdir) [INFO] [stdout] warning: function `check` is never used [INFO] [stdout] --> src/header.rs:13:1 [INFO] [stdout] | [INFO] [stdout] 13 | / /// Header structure. [INFO] [stdout] 14 | | /// [INFO] [stdout] 15 | | /// Implements [`AsRef`], which will return a byte slice with the correct encoding of the header [INFO] [stdout] 16 | | /// that can be sent directly to a peer. [INFO] [stdout] 17 | | #[derive(Copy, Clone, Eq, PartialEq, Pod, Zeroable)] [INFO] [stdout] 18 | | #[repr(transparent)] [INFO] [stdout] 19 | | pub struct Header([u8; Header::SIZE]); [INFO] [stdout] | |______________________________________^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(dead_code)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: field `0` is never read [INFO] [stdout] --> src/protocol/outgoing_message.rs:170:8 [INFO] [stdout] | [INFO] [stdout] 170 | struct Preamble { [INFO] [stdout] | ^^^^^^^^ field in this struct [INFO] [stdout] | [INFO] [stdout] = help: consider removing this field [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function `check` is never used [INFO] [stdout] --> src/protocol/outgoing_message.rs:159:1 [INFO] [stdout] | [INFO] [stdout] 159 | / /// Combination of header and potential message payload length. [INFO] [stdout] 160 | | /// [INFO] [stdout] 161 | | /// A message with a payload always starts with an initial frame that has a header and a varint [INFO] [stdout] 162 | | /// encoded payload length. This type combines the two, and allows for the payload length to [INFO] [stdout] ... | [INFO] [stdout] 174 | | payload_length: Varint32, [INFO] [stdout] 175 | | } [INFO] [stdout] | |_^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function `check` is never used [INFO] [stdout] --> src/varint.rs:61:1 [INFO] [stdout] | [INFO] [stdout] 61 | / /// An encoded varint32. [INFO] [stdout] 62 | | /// [INFO] [stdout] 63 | | /// Internally these are stored as six byte arrays to make passing around convenient. Since the [INFO] [stdout] 64 | | /// maximum length a 32 bit varint can posses is 5 bytes, the 6th byte is used to record the [INFO] [stdout] ... | [INFO] [stdout] 67 | | #[derive(Copy, Clone, Pod, Zeroable)] [INFO] [stdout] 68 | | pub struct Varint32([u8; 6]); [INFO] [stdout] | |_____________________________^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing [INFO] [stdout] --> src/rpc.rs:235:27 [INFO] [stdout] | [INFO] [stdout] 235 | pub fn create_request(&self, channel: ChannelId) -> JulietRpcRequestBuilder { [INFO] [stdout] | ^^^^^ -------------------------- the same lifetime is hidden here [INFO] [stdout] | | [INFO] [stdout] | the lifetime is elided here [INFO] [stdout] | [INFO] [stdout] = help: the same lifetime is referred to in inconsistent ways, making the signature confusing [INFO] [stdout] = note: `#[warn(mismatched_lifetime_syntaxes)]` on by default [INFO] [stdout] help: use `'_` for type paths [INFO] [stdout] | [INFO] [stdout] 235 | pub fn create_request(&self, channel: ChannelId) -> JulietRpcRequestBuilder<'_, N> { [INFO] [stdout] | +++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function `check` is never used [INFO] [stdout] --> src/header.rs:13:1 [INFO] [stdout] | [INFO] [stdout] 13 | / /// Header structure. [INFO] [stdout] 14 | | /// [INFO] [stdout] 15 | | /// Implements [`AsRef`], which will return a byte slice with the correct encoding of the header [INFO] [stdout] 16 | | /// that can be sent directly to a peer. [INFO] [stdout] 17 | | #[derive(Copy, Clone, Eq, PartialEq, Pod, Zeroable)] [INFO] [stdout] 18 | | #[repr(transparent)] [INFO] [stdout] 19 | | pub struct Header([u8; Header::SIZE]); [INFO] [stdout] | |______________________________________^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(dead_code)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: field `0` is never read [INFO] [stdout] --> src/protocol/outgoing_message.rs:170:8 [INFO] [stdout] | [INFO] [stdout] 170 | struct Preamble { [INFO] [stdout] | ^^^^^^^^ field in this struct [INFO] [stdout] | [INFO] [stdout] = help: consider removing this field [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function `check` is never used [INFO] [stdout] --> src/protocol/outgoing_message.rs:159:1 [INFO] [stdout] | [INFO] [stdout] 159 | / /// Combination of header and potential message payload length. [INFO] [stdout] 160 | | /// [INFO] [stdout] 161 | | /// A message with a payload always starts with an initial frame that has a header and a varint [INFO] [stdout] 162 | | /// encoded payload length. This type combines the two, and allows for the payload length to [INFO] [stdout] ... | [INFO] [stdout] 174 | | payload_length: Varint32, [INFO] [stdout] 175 | | } [INFO] [stdout] | |_^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function `check` is never used [INFO] [stdout] --> src/varint.rs:61:1 [INFO] [stdout] | [INFO] [stdout] 61 | / /// An encoded varint32. [INFO] [stdout] 62 | | /// [INFO] [stdout] 63 | | /// Internally these are stored as six byte arrays to make passing around convenient. Since the [INFO] [stdout] 64 | | /// maximum length a 32 bit varint can posses is 5 bytes, the 6th byte is used to record the [INFO] [stdout] ... | [INFO] [stdout] 67 | | #[derive(Copy, Clone, Pod, Zeroable)] [INFO] [stdout] 68 | | pub struct Varint32([u8; 6]); [INFO] [stdout] | |_____________________________^ [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/header.rs:52:25 [INFO] [stdout] | [INFO] [stdout] 52 | #[cfg_attr(test, derive(proptest_derive::Arbitrary))] [INFO] [stdout] | ^------------------------- [INFO] [stdout] | | [INFO] [stdout] | `Arbitrary` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_IMPL_ARBITRARY_FOR_ErrorKind` [INFO] [stdout] 53 | #[repr(u8)] [INFO] [stdout] 54 | pub enum ErrorKind { [INFO] [stdout] | --------- `ErrorKind` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `proptest_derive::Arbitrary` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `proptest_derive::Arbitrary` may come from an old version of the `proptest_derive` crate, try updating your dependency with `cargo update -p proptest_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 `proptest_derive::Arbitrary` (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/header.rs:103:25 [INFO] [stdout] | [INFO] [stdout] 103 | #[cfg_attr(test, derive(proptest_derive::Arbitrary))] [INFO] [stdout] | ^------------------------- [INFO] [stdout] | | [INFO] [stdout] | `Arbitrary` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_IMPL_ARBITRARY_FOR_Kind` [INFO] [stdout] 104 | #[repr(u8)] [INFO] [stdout] 105 | pub enum Kind { [INFO] [stdout] | ---- `Kind` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `proptest_derive::Arbitrary` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `proptest_derive::Arbitrary` may come from an old version of the `proptest_derive` crate, try updating your dependency with `cargo update -p proptest_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 `proptest_derive::Arbitrary` (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/protocol/multiframe.rs:362:14 [INFO] [stdout] | [INFO] [stdout] 362 | #[derive(Arbitrary, derive_more::Debug)] [INFO] [stdout] | ^-------- [INFO] [stdout] | | [INFO] [stdout] | `Arbitrary` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_IMPL_ARBITRARY_FOR_Action` [INFO] [stdout] 363 | enum Action { [INFO] [stdout] | ------ `Action` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `Arbitrary` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `Arbitrary` may come from an old version of the `proptest_derive` crate, try updating your dependency with `cargo update -p proptest_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 `Arbitrary` (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/protocol.rs:1085:14 [INFO] [stdout] | [INFO] [stdout] 1085 | #[derive(Arbitrary, Clone, Copy, Debug, EnumIter)] [INFO] [stdout] | ^-------- [INFO] [stdout] | | [INFO] [stdout] | `Arbitrary` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_IMPL_ARBITRARY_FOR_VaryingPayload` [INFO] [stdout] 1086 | enum VaryingPayload { [INFO] [stdout] | -------------- `VaryingPayload` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `Arbitrary` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `Arbitrary` may come from an old version of the `proptest_derive` crate, try updating your dependency with `cargo update -p proptest_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 `Arbitrary` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing [INFO] [stdout] --> src/rpc.rs:235:27 [INFO] [stdout] | [INFO] [stdout] 235 | pub fn create_request(&self, channel: ChannelId) -> JulietRpcRequestBuilder { [INFO] [stdout] | ^^^^^ -------------------------- the same lifetime is hidden here [INFO] [stdout] | | [INFO] [stdout] | the lifetime is elided here [INFO] [stdout] | [INFO] [stdout] = help: the same lifetime is referred to in inconsistent ways, making the signature confusing [INFO] [stdout] = note: `#[warn(mismatched_lifetime_syntaxes)]` on by default [INFO] [stdout] help: use `'_` for type paths [INFO] [stdout] | [INFO] [stdout] 235 | pub fn create_request(&self, channel: ChannelId) -> JulietRpcRequestBuilder<'_, N> { [INFO] [stdout] | +++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused return value of `must_use` that must be used [INFO] [stdout] --> src/varint.rs:313:9 [INFO] [stdout] | [INFO] [stdout] 313 | format!("{:?}", Varint32::encode(value)); [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_must_use)]` on by default [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 313 | let _ = format!("{:?}", Varint32::encode(value)); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stderr] Finished `dev` profile [unoptimized + debuginfo] target(s) in 38.73s [INFO] running `Command { std: "docker" "inspect" "45f8e303c54527a9eb06d4bf5895b607cb6f4bf703b98c50206c940fcdb1c8cc", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "45f8e303c54527a9eb06d4bf5895b607cb6f4bf703b98c50206c940fcdb1c8cc", kill_on_drop: false }` [INFO] [stdout] 45f8e303c54527a9eb06d4bf5895b607cb6f4bf703b98c50206c940fcdb1c8cc