[INFO] cloning repository https://github.com/scrdest/covert-scheme
[INFO] running `Command { std: "git" "-c" "credential.helper=" "-c" "credential.helper=/workspace/cargo-home/bin/git-credential-null" "clone" "--bare" "https://github.com/scrdest/covert-scheme" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fscrdest%2Fcovert-scheme", kill_on_drop: false }`
[INFO] [stderr] Cloning into bare repository '/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fscrdest%2Fcovert-scheme'...
[INFO] running `Command { std: "git" "rev-parse" "HEAD", kill_on_drop: false }`
[INFO] [stdout] e3395a106a4039fc98e1544b58dd9d89dacbe839
[INFO] checking scrdest/covert-scheme against a5689b034ad287112f709693384a37f1377eb9cc for pr-129249-3
[INFO] running `Command { std: "git" "clone" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fscrdest%2Fcovert-scheme" "/workspace/builds/worker-5-tc2/source", kill_on_drop: false }`
[INFO] [stderr] Cloning into '/workspace/builds/worker-5-tc2/source'...
[INFO] [stderr] done.
[INFO] started tweaking git repo https://github.com/scrdest/covert-scheme
[INFO] finished tweaking git repo https://github.com/scrdest/covert-scheme
[INFO] tweaked toml for git repo https://github.com/scrdest/covert-scheme written to /workspace/builds/worker-5-tc2/source/Cargo.toml
[INFO] validating manifest of git repo https://github.com/scrdest/covert-scheme 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/scrdest/covert-scheme 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-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:77db811e55d90add9212f6832c23229e3de5f1c1f9905e8cb07319ea15df73ac" "sleep" "infinity", kill_on_drop: false }`
[INFO] [stdout] c878de9b357f99c9279be03124446dd68bfc0411ad76558f142ef1ef9a9f4c44
[INFO] running `Command { std: "docker" "start" "c878de9b357f99c9279be03124446dd68bfc0411ad76558f142ef1ef9a9f4c44", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "c878de9b357f99c9279be03124446dd68bfc0411ad76558f142ef1ef9a9f4c44", 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" "c878de9b357f99c9279be03124446dd68bfc0411ad76558f142ef1ef9a9f4c44" "/opt/rustwide/cargo-home/bin/cargo" "+a5689b034ad287112f709693384a37f1377eb9cc" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "c878de9b357f99c9279be03124446dd68bfc0411ad76558f142ef1ef9a9f4c44", 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" "c878de9b357f99c9279be03124446dd68bfc0411ad76558f142ef1ef9a9f4c44" "/opt/rustwide/cargo-home/bin/cargo" "+a5689b034ad287112f709693384a37f1377eb9cc" "check" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }`
[INFO] [stderr]    Compiling proc-macro2 v1.0.107
[INFO] [stderr]    Compiling quote v1.0.47
[INFO] [stderr]    Compiling zerocopy v0.8.55
[INFO] [stderr]     Checking chacha20 v0.10.1
[INFO] [stderr]     Checking getrandom v0.4.3
[INFO] [stderr]     Checking rand v0.10.2
[INFO] [stderr]    Compiling syn v3.0.3
[INFO] [stderr]    Compiling serde_derive v1.0.229
[INFO] [stderr]     Checking serde v1.0.229
[INFO] [stderr]     Checking ppv-lite86 v0.2.21
[INFO] [stderr]     Checking rand_chacha v0.10.0
[INFO] [stderr]     Checking covert-schemer v0.1.0 (/opt/rustwide/workdir)
[INFO] [stdout] error: useless conversion to the same type: `datapack::NamePool`
[INFO] [stdout]     --> src/datapack.rs:2056:20
[INFO] [stdout]      |
[INFO] [stdout] 2056 |             names: self.names.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] 2056 -             names: self.names.into(),
[INFO] [stdout] 2056 +             names: self.names,
[INFO] [stdout]      |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]      |
[INFO] [stdout] 2056 -             names: self.names.into(),
[INFO] [stdout] 2056 +             names: <datapack::NamePool as Into<_>>::into(self.names),
[INFO] [stdout]      |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Arc<str>`
[INFO] [stdout]    --> src/htn.rs:455:50
[INFO] [stdout]     |
[INFO] [stdout] 455 |                 self.spawned.insert(var.clone(), id.clone().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] 455 -                 self.spawned.insert(var.clone(), id.clone().into());
[INFO] [stdout] 455 +                 self.spawned.insert(var.clone(), id.clone());
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 455 -                 self.spawned.insert(var.clone(), id.clone().into());
[INFO] [stdout] 455 +                 self.spawned.insert(var.clone(), <Arc<str> as Into<_>>::into(id.clone()));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `&str`
[INFO] [stdout]    --> src/intel.rs:375:31
[INFO] [stdout]     |
[INFO] [stdout] 375 |               let placeholder = match v {
[INFO] [stdout]     |  _______________________________^
[INFO] [stdout] 376 | |                 _ if is_subject => "an unknown figure",
[INFO] [stdout] 377 | |                 DisplayToken::Person(_) => "an unknown contact",
[INFO] [stdout] 378 | |                 DisplayToken::Location(_) => "an unknown location",
[INFO] [stdout] 379 | |                 DisplayToken::Faction(_) => "affiliation unknown",
[INFO] [stdout] 380 | |                 _ => "",
[INFO] [stdout] 381 | |             }.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] 381 -             }.into();
[INFO] [stdout] 381 +             };
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 375 ~             let placeholder = <&str as Into<_>>::into(match v {
[INFO] [stdout] 376 |                 _ if is_subject => "an unknown figure",
[INFO] [stdout] ...
[INFO] [stdout] 380 |                 _ => "",
[INFO] [stdout] 381 ~             });
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Arc<str>`
[INFO] [stdout]    --> src/worldgen.rs:661:14
[INFO] [stdout]     |
[INFO] [stdout] 661 |     .map(|i| domain.personalities[i].tag.clone().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] 661 -     .map(|i| domain.personalities[i].tag.clone().into())
[INFO] [stdout] 661 +     .map(|i| domain.personalities[i].tag.clone())
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 661 -     .map(|i| domain.personalities[i].tag.clone().into())
[INFO] [stdout] 661 +     .map(|i| <Arc<str> as Into<_>>::into(domain.personalities[i].tag.clone()))
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr] error: could not compile `covert-schemer` (lib test) due to 4 previous errors
[INFO] [stderr] warning: build failed, waiting for other jobs to finish...
[INFO] [stdout] error: useless conversion to the same type: `datapack::NamePool`
[INFO] [stdout]     --> src/datapack.rs:2056:20
[INFO] [stdout]      |
[INFO] [stdout] 2056 |             names: self.names.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] 2056 -             names: self.names.into(),
[INFO] [stdout] 2056 +             names: self.names,
[INFO] [stdout]      |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]      |
[INFO] [stdout] 2056 -             names: self.names.into(),
[INFO] [stdout] 2056 +             names: <datapack::NamePool as Into<_>>::into(self.names),
[INFO] [stdout]      |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Arc<str>`
[INFO] [stdout]    --> src/htn.rs:455:50
[INFO] [stdout]     |
[INFO] [stdout] 455 |                 self.spawned.insert(var.clone(), id.clone().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] 455 -                 self.spawned.insert(var.clone(), id.clone().into());
[INFO] [stdout] 455 +                 self.spawned.insert(var.clone(), id.clone());
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 455 -                 self.spawned.insert(var.clone(), id.clone().into());
[INFO] [stdout] 455 +                 self.spawned.insert(var.clone(), <Arc<str> as Into<_>>::into(id.clone()));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `&str`
[INFO] [stdout]    --> src/intel.rs:375:31
[INFO] [stdout]     |
[INFO] [stdout] 375 |               let placeholder = match v {
[INFO] [stdout]     |  _______________________________^
[INFO] [stdout] 376 | |                 _ if is_subject => "an unknown figure",
[INFO] [stdout] 377 | |                 DisplayToken::Person(_) => "an unknown contact",
[INFO] [stdout] 378 | |                 DisplayToken::Location(_) => "an unknown location",
[INFO] [stdout] 379 | |                 DisplayToken::Faction(_) => "affiliation unknown",
[INFO] [stdout] 380 | |                 _ => "",
[INFO] [stdout] 381 | |             }.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] 381 -             }.into();
[INFO] [stdout] 381 +             };
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 375 ~             let placeholder = <&str as Into<_>>::into(match v {
[INFO] [stdout] 376 |                 _ if is_subject => "an unknown figure",
[INFO] [stdout] ...
[INFO] [stdout] 380 |                 _ => "",
[INFO] [stdout] 381 ~             });
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Arc<str>`
[INFO] [stdout]    --> src/worldgen.rs:661:14
[INFO] [stdout]     |
[INFO] [stdout] 661 |     .map(|i| domain.personalities[i].tag.clone().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] 661 -     .map(|i| domain.personalities[i].tag.clone().into())
[INFO] [stdout] 661 +     .map(|i| domain.personalities[i].tag.clone())
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 661 -     .map(|i| domain.personalities[i].tag.clone().into())
[INFO] [stdout] 661 +     .map(|i| <Arc<str> as Into<_>>::into(domain.personalities[i].tag.clone()))
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr] error: could not compile `covert-schemer` (lib) due to 4 previous errors
[INFO] running `Command { std: "docker" "inspect" "c878de9b357f99c9279be03124446dd68bfc0411ad76558f142ef1ef9a9f4c44", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "c878de9b357f99c9279be03124446dd68bfc0411ad76558f142ef1ef9a9f4c44", kill_on_drop: false }`
[INFO] [stdout] c878de9b357f99c9279be03124446dd68bfc0411ad76558f142ef1ef9a9f4c44
