[INFO] fetching crate intercom-common 0.4.0...
[INFO] checking intercom-common-0.4.0 against 52995a223f2919dd3e2e427231db90a257350ded for pr-129249-4
[INFO] extracting crate intercom-common 0.4.0 into /workspace/builds/worker-5-tc2/source
[INFO] started tweaking crates.io crate intercom-common 0.4.0
[INFO] finished tweaking crates.io crate intercom-common 0.4.0
[INFO] tweaked toml for crates.io crate intercom-common 0.4.0 written to /workspace/builds/worker-5-tc2/source/Cargo.toml
[INFO] validating manifest of crates.io crate intercom-common 0.4.0 on toolchain 52995a223f2919dd3e2e427231db90a257350ded
[INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+52995a223f2919dd3e2e427231db90a257350ded" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }`
[INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+52995a223f2919dd3e2e427231db90a257350ded" "generate-lockfile" "--manifest-path" "Cargo.toml", kill_on_drop: false }`
[INFO] [stderr]     Updating crates.io index
[INFO] [stderr]      Locking 25 packages to highest compatible versions
[INFO] [stderr]       Adding difference v1.0.0 (available: v2.0.0)
[INFO] [stderr]       Adding indexmap v1.9.3 (available: v2.14.2)
[INFO] [stderr]       Adding sha1 v0.6.1 (available: v0.11.0)
[INFO] [stderr]       Adding syn v1.0.109 (available: v3.0.9)
[INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+52995a223f2919dd3e2e427231db90a257350ded" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-5-tc2/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-5-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:c0af3998e971868164375a7dfb646c9933ddd1a3bdb00fa2041266763dc1eeab" "sleep" "infinity", kill_on_drop: false }`
[INFO] [stdout] 5db6ebd0c486eb05a23e41751181c17f6a4caa9903345aaaa60e28f714dc003e
[INFO] running `Command { std: "docker" "start" "5db6ebd0c486eb05a23e41751181c17f6a4caa9903345aaaa60e28f714dc003e", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "5db6ebd0c486eb05a23e41751181c17f6a4caa9903345aaaa60e28f714dc003e", 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" "5db6ebd0c486eb05a23e41751181c17f6a4caa9903345aaaa60e28f714dc003e" "/opt/rustwide/cargo-home/bin/cargo" "+52995a223f2919dd3e2e427231db90a257350ded" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "5db6ebd0c486eb05a23e41751181c17f6a4caa9903345aaaa60e28f714dc003e", 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" "5db6ebd0c486eb05a23e41751181c17f6a4caa9903345aaaa60e28f714dc003e" "/opt/rustwide/cargo-home/bin/cargo" "+52995a223f2919dd3e2e427231db90a257350ded" "check" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }`
[INFO] [stderr]    Compiling syn v1.0.109
[INFO] [stderr]    Compiling proc-macro2 v1.0.107
[INFO] [stderr]    Compiling unicode-xid v0.2.6
[INFO] [stderr]     Checking unicode-ident v1.0.27
[INFO] [stderr]     Checking gimli v0.34.0
[INFO] [stderr]    Compiling failure_derive v0.1.8
[INFO] [stderr]     Checking memchr v2.8.3
[INFO] [stderr]     Checking rustc-demangle v0.1.28
[INFO] [stderr]     Checking sha1_smol v1.0.1
[INFO] [stderr]     Checking difference v1.0.0
[INFO] [stderr]     Checking miniz_oxide v0.9.1
[INFO] [stderr]     Checking sha1 v0.6.1
[INFO] [stderr]     Checking object v0.40.0
[INFO] [stderr]    Compiling quote v1.0.47
[INFO] [stderr]     Checking addr2line v0.27.1
[INFO] [stderr]    Compiling synstructure v0.12.6
[INFO] [stderr]     Checking backtrace v0.3.77
[INFO] [stderr]     Checking failure v0.1.8
[INFO] [stderr]     Checking intercom-common v0.4.0 (/opt/rustwide/workdir)
[INFO] [stdout] warning: variable does not need to be mutable
[INFO] [stdout]    --> src/ast_converters.rs:155:13
[INFO] [stdout]     |
[INFO] [stdout] 155 |         let mut last = result
[INFO] [stdout]     |             ----^^^^
[INFO] [stdout]     |             |
[INFO] [stdout]     |             help: remove this `mut`
[INFO] [stdout]     |
[INFO] [stdout]     = note: `#[warn(unused_mut)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variable does not need to be mutable
[INFO] [stdout]    --> src/ast_converters.rs:155:13
[INFO] [stdout]     |
[INFO] [stdout] 155 |         let mut last = result
[INFO] [stdout]     |             ----^^^^
[INFO] [stdout]     |             |
[INFO] [stdout]     |             help: remove this `mut`
[INFO] [stdout]     |
[INFO] [stdout]     = note: `#[warn(unused_mut)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variable does not need to be mutable
[INFO] [stdout]   --> src/idents.rs:39:17
[INFO] [stdout]    |
[INFO] [stdout] 39 |     if let Some(mut last) = clsid_path.segments.last_mut() {
[INFO] [stdout]    |                 ----^^^^
[INFO] [stdout]    |                 |
[INFO] [stdout]    |                 help: remove this `mut`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variable does not need to be mutable
[INFO] [stdout]   --> src/idents.rs:39:17
[INFO] [stdout]    |
[INFO] [stdout] 39 |     if let Some(mut last) = clsid_path.segments.last_mut() {
[INFO] [stdout]    |                 ----^^^^
[INFO] [stdout]    |                 |
[INFO] [stdout]    |                 help: remove this `mut`
[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/model/mod.rs:10:10
[INFO] [stdout]    |
[INFO] [stdout] 10 | #[derive(Fail, Debug)]
[INFO] [stdout]    |          ^---
[INFO] [stdout]    |          |
[INFO] [stdout]    |          `Fail` is not local
[INFO] [stdout]    |          move the `impl` block outside of this constant `_DERIVE_failure_Fail_FOR_ParseError`
[INFO] [stdout] 11 | #[non_exhaustive]
[INFO] [stdout] 12 | pub enum ParseError
[INFO] [stdout]    |          ---------- `ParseError` 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/model/mod.rs:10:10
[INFO] [stdout]    |
[INFO] [stdout] 10 | #[derive(Fail, Debug)]
[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_ParseError`
[INFO] [stdout] 11 | #[non_exhaustive]
[INFO] [stdout] 12 | pub enum ParseError
[INFO] [stdout]    |          ---------- `ParseError` 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: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]   --> src/utils.rs:22:32
[INFO] [stdout]    |
[INFO] [stdout] 22 | pub fn get_ident_and_fns(item: &Item) -> Option<InterfaceData>
[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] 22 | pub fn get_ident_and_fns(item: &Item) -> Option<InterfaceData<'_>>
[INFO] [stdout]    |                                                              ++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `&str`
[INFO] [stdout]    --> src/model/comlibrary.rs:109:13
[INFO] [stdout]     |
[INFO] [stdout] 109 |             "library_name".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] 109 -             "library_name".into(),
[INFO] [stdout] 109 +             "library_name",
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 109 -             "library_name".into(),
[INFO] [stdout] 109 +             <&str as Into<_>>::into("library_name"),
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `&str`
[INFO] [stdout]    --> src/model/comlibrary.rs:140:37
[INFO] [stdout]     |
[INFO] [stdout] 140 |         let lib = ComLibrary::parse("another_library".into(), quote!(class One, class Two))
[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] 140 -         let lib = ComLibrary::parse("another_library".into(), quote!(class One, class Two))
[INFO] [stdout] 140 +         let lib = ComLibrary::parse("another_library", quote!(class One, class Two))
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 140 -         let lib = ComLibrary::parse("another_library".into(), quote!(class One, class Two))
[INFO] [stdout] 140 +         let lib = ComLibrary::parse(<&str as Into<_>>::into("another_library"), quote!(class One, class Two))
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `&str`
[INFO] [stdout]    --> src/model/comlibrary.rs:156:37
[INFO] [stdout]     |
[INFO] [stdout] 156 |         let lib = ComLibrary::parse("lib".into(), quote!()).unwrap();
[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] 156 -         let lib = ComLibrary::parse("lib".into(), quote!()).unwrap();
[INFO] [stdout] 156 +         let lib = ComLibrary::parse("lib", quote!()).unwrap();
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 156 -         let lib = ComLibrary::parse("lib".into(), quote!()).unwrap();
[INFO] [stdout] 156 +         let lib = ComLibrary::parse(<&str as Into<_>>::into("lib"), quote!()).unwrap();
[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/model/mod.rs:10:10
[INFO] [stdout]    |
[INFO] [stdout] 10 | #[derive(Fail, Debug)]
[INFO] [stdout]    |          ^---
[INFO] [stdout]    |          |
[INFO] [stdout]    |          `Fail` is not local
[INFO] [stdout]    |          move the `impl` block outside of this constant `_DERIVE_failure_Fail_FOR_ParseError`
[INFO] [stdout] 11 | #[non_exhaustive]
[INFO] [stdout] 12 | pub enum ParseError
[INFO] [stdout]    |          ---------- `ParseError` 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/model/mod.rs:10:10
[INFO] [stdout]    |
[INFO] [stdout] 10 | #[derive(Fail, Debug)]
[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_ParseError`
[INFO] [stdout] 11 | #[non_exhaustive]
[INFO] [stdout] 12 | pub enum ParseError
[INFO] [stdout]    |          ---------- `ParseError` 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: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]   --> src/utils.rs:22:32
[INFO] [stdout]    |
[INFO] [stdout] 22 | pub fn get_ident_and_fns(item: &Item) -> Option<InterfaceData>
[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] 22 | pub fn get_ident_and_fns(item: &Item) -> Option<InterfaceData<'_>>
[INFO] [stdout]    |                                                              ++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr] error: could not compile `intercom-common` (lib test) due to 3 previous errors; 5 warnings emitted
[INFO] running `Command { std: "docker" "inspect" "5db6ebd0c486eb05a23e41751181c17f6a4caa9903345aaaa60e28f714dc003e", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "5db6ebd0c486eb05a23e41751181c17f6a4caa9903345aaaa60e28f714dc003e", kill_on_drop: false }`
[INFO] [stdout] 5db6ebd0c486eb05a23e41751181c17f6a4caa9903345aaaa60e28f714dc003e
