[INFO] cloning repository https://github.com/ueokande/sagume
[INFO] running `Command { std: "git" "-c" "credential.helper=" "-c" "credential.helper=/workspace/cargo-home/bin/git-credential-null" "clone" "--bare" "https://github.com/ueokande/sagume" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fueokande%2Fsagume", kill_on_drop: false }`
[INFO] [stderr] Cloning into bare repository '/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fueokande%2Fsagume'...
[INFO] running `Command { std: "git" "rev-parse" "HEAD", kill_on_drop: false }`
[INFO] [stdout] 72994f354a1c1028b25fd17901b1c663a2ee1647
[INFO] checking ueokande/sagume against a5689b034ad287112f709693384a37f1377eb9cc for pr-129249-3
[INFO] running `Command { std: "git" "clone" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fueokande%2Fsagume" "/workspace/builds/worker-3-tc2/source", kill_on_drop: false }`
[INFO] [stderr] Cloning into '/workspace/builds/worker-3-tc2/source'...
[INFO] [stderr] done.
[INFO] started tweaking git repo https://github.com/ueokande/sagume
[INFO] finished tweaking git repo https://github.com/ueokande/sagume
[INFO] tweaked toml for git repo https://github.com/ueokande/sagume written to /workspace/builds/worker-3-tc2/source/Cargo.toml
[INFO] validating manifest of git repo https://github.com/ueokande/sagume 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/ueokande/sagume 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-3-tc2/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-3-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] eaee57eaf175b35a1a769cc830daf8b48cc2fc4fb2249342345eb7b022fb3686
[INFO] running `Command { std: "docker" "start" "eaee57eaf175b35a1a769cc830daf8b48cc2fc4fb2249342345eb7b022fb3686", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "eaee57eaf175b35a1a769cc830daf8b48cc2fc4fb2249342345eb7b022fb3686", 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" "eaee57eaf175b35a1a769cc830daf8b48cc2fc4fb2249342345eb7b022fb3686" "/opt/rustwide/cargo-home/bin/cargo" "+a5689b034ad287112f709693384a37f1377eb9cc" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "eaee57eaf175b35a1a769cc830daf8b48cc2fc4fb2249342345eb7b022fb3686", 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" "eaee57eaf175b35a1a769cc830daf8b48cc2fc4fb2249342345eb7b022fb3686" "/opt/rustwide/cargo-home/bin/cargo" "+a5689b034ad287112f709693384a37f1377eb9cc" "check" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }`
[INFO] [stderr]     Checking sagume v0.1.0 (/opt/rustwide/workdir)
[INFO] [stdout] warning: unnecessary trailing semicolon
[INFO] [stdout]   --> src/index.rs:74:86
[INFO] [stdout]    |
[INFO] [stdout] 74 |                 .unwrap_or(self.field_names.iter().map(|f| f.to_string()).collect());;
[INFO] [stdout]    |                                                                                      ^ help: remove this semicolon
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(redundant_semicolons)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: trait objects without an explicit `dyn` are deprecated
[INFO] [stdout]  --> src/pipeline.rs:3:25
[INFO] [stdout]   |
[INFO] [stdout] 3 | type PipelineFunction = Fn(i32) -> i32;
[INFO] [stdout]   |                         ^^^^^^^^^^^^^^
[INFO] [stdout]   |
[INFO] [stdout]   = warning: this is accepted in the current edition (Rust 2018) but is a hard error in Rust 2021!
[INFO] [stdout]   = note: for more information, see <https://doc.rust-lang.org/edition-guide/rust-2021/warnings-promoted-to-error.html>
[INFO] [stdout]   = note: `#[warn(bare_trait_objects)]` (part of `#[warn(rust_2021_compatibility)]`) on by default
[INFO] [stdout] help: if this is a dyn-compatible trait, use `dyn`
[INFO] [stdout]   |
[INFO] [stdout] 3 | type PipelineFunction = dyn Fn(i32) -> i32;
[INFO] [stdout]   |                         +++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unnecessary trailing semicolon
[INFO] [stdout]   --> src/index.rs:74:86
[INFO] [stdout]    |
[INFO] [stdout] 74 |                 .unwrap_or(self.field_names.iter().map(|f| f.to_string()).collect());;
[INFO] [stdout]    |                                                                                      ^ help: remove this semicolon
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(redundant_semicolons)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: trait objects without an explicit `dyn` are deprecated
[INFO] [stdout]  --> src/pipeline.rs:3:25
[INFO] [stdout]   |
[INFO] [stdout] 3 | type PipelineFunction = Fn(i32) -> i32;
[INFO] [stdout]   |                         ^^^^^^^^^^^^^^
[INFO] [stdout]   |
[INFO] [stdout]   = warning: this is accepted in the current edition (Rust 2018) but is a hard error in Rust 2021!
[INFO] [stdout]   = note: for more information, see <https://doc.rust-lang.org/edition-guide/rust-2021/warnings-promoted-to-error.html>
[INFO] [stdout]   = note: `#[warn(bare_trait_objects)]` (part of `#[warn(rust_2021_compatibility)]`) on by default
[INFO] [stdout] help: if this is a dyn-compatible trait, use `dyn`
[INFO] [stdout]   |
[INFO] [stdout] 3 | type PipelineFunction = dyn Fn(i32) -> i32;
[INFO] [stdout]   |                         +++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `&String`
[INFO] [stdout]   --> src/index.rs:90:58
[INFO] [stdout]    |
[INFO] [stdout] 90 |                     let matching_docs = ri.documents.get(field.into()).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]    = note: `#[deny(self_type_conversion)]` on by default
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]    |
[INFO] [stdout] 90 -                     let matching_docs = ri.documents.get(field.into()).unwrap();
[INFO] [stdout] 90 +                     let matching_docs = ri.documents.get(field).unwrap();
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 90 -                     let matching_docs = ri.documents.get(field.into()).unwrap();
[INFO] [stdout] 90 +                     let matching_docs = ri.documents.get(<&String as Into<_>>::into(field)).unwrap();
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr] error: could not compile `sagume` (lib) due to 1 previous error; 2 warnings emitted
[INFO] [stderr] warning: build failed, waiting for other jobs to finish...
[INFO] [stdout] error: useless conversion to the same type: `&String`
[INFO] [stdout]   --> src/index.rs:90:58
[INFO] [stdout]    |
[INFO] [stdout] 90 |                     let matching_docs = ri.documents.get(field.into()).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]    = note: `#[deny(self_type_conversion)]` on by default
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]    |
[INFO] [stdout] 90 -                     let matching_docs = ri.documents.get(field.into()).unwrap();
[INFO] [stdout] 90 +                     let matching_docs = ri.documents.get(field).unwrap();
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 90 -                     let matching_docs = ri.documents.get(field.into()).unwrap();
[INFO] [stdout] 90 +                     let matching_docs = ri.documents.get(<&String as Into<_>>::into(field)).unwrap();
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `&str`
[INFO] [stdout]    --> src/builder.rs:202:44
[INFO] [stdout]     |
[INFO] [stdout] 202 |     assert!(!b.inverted_index.contains_key("missing".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] 202 -     assert!(!b.inverted_index.contains_key("missing".into()));
[INFO] [stdout] 202 +     assert!(!b.inverted_index.contains_key("missing"));
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 202 -     assert!(!b.inverted_index.contains_key("missing".into()));
[INFO] [stdout] 202 +     assert!(!b.inverted_index.contains_key(<&str as Into<_>>::into("missing")));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `&str`
[INFO] [stdout]    --> src/builder.rs:207:18
[INFO] [stdout]     |
[INFO] [stdout] 207 |             .get("constructor".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] 207 -             .get("constructor".into())
[INFO] [stdout] 207 +             .get("constructor")
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 207 -             .get("constructor".into())
[INFO] [stdout] 207 +             .get(<&str as Into<_>>::into("constructor"))
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `&str`
[INFO] [stdout]    --> src/builder.rs:220:36
[INFO] [stdout]     |
[INFO] [stdout] 220 |     assert!(b.field_names.contains("title".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] 220 -     assert!(b.field_names.contains("title".into()));
[INFO] [stdout] 220 +     assert!(b.field_names.contains("title"));
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 220 -     assert!(b.field_names.contains("title".into()));
[INFO] [stdout] 220 +     assert!(b.field_names.contains(<&str as Into<_>>::into("title")));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `&str`
[INFO] [stdout]    --> src/builder.rs:221:36
[INFO] [stdout]     |
[INFO] [stdout] 221 |     assert!(b.field_names.contains("isbn".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] 221 -     assert!(b.field_names.contains("isbn".into()));
[INFO] [stdout] 221 +     assert!(b.field_names.contains("isbn"));
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 221 -     assert!(b.field_names.contains("isbn".into()));
[INFO] [stdout] 221 +     assert!(b.field_names.contains(<&str as Into<_>>::into("isbn")));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `&str`
[INFO] [stdout]    --> src/builder.rs:222:36
[INFO] [stdout]     |
[INFO] [stdout] 222 |     assert!(b.field_names.contains("author/age".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] 222 -     assert!(b.field_names.contains("author/age".into()));
[INFO] [stdout] 222 +     assert!(b.field_names.contains("author/age"));
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 222 -     assert!(b.field_names.contains("author/age".into()));
[INFO] [stdout] 222 +     assert!(b.field_names.contains(<&str as Into<_>>::into("author/age")));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `&str`
[INFO] [stdout]    --> src/token.rs:368:38
[INFO] [stdout]     |
[INFO] [stdout] 368 |     let root = TokenSet::from_string("a".into()).root;
[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] 368 -     let root = TokenSet::from_string("a".into()).root;
[INFO] [stdout] 368 +     let root = TokenSet::from_string("a").root;
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 368 -     let root = TokenSet::from_string("a".into()).root;
[INFO] [stdout] 368 +     let root = TokenSet::from_string(<&str as Into<_>>::into("a")).root;
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `&str`
[INFO] [stdout]    --> src/token.rs:372:38
[INFO] [stdout]     |
[INFO] [stdout] 372 |     let root = TokenSet::from_string("a*".into()).root;
[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] 372 -     let root = TokenSet::from_string("a*".into()).root;
[INFO] [stdout] 372 +     let root = TokenSet::from_string("a*").root;
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 372 -     let root = TokenSet::from_string("a*".into()).root;
[INFO] [stdout] 372 +     let root = TokenSet::from_string(<&str as Into<_>>::into("a*")).root;
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr] error: could not compile `sagume` (lib test) due to 8 previous errors; 2 warnings emitted
[INFO] running `Command { std: "docker" "inspect" "eaee57eaf175b35a1a769cc830daf8b48cc2fc4fb2249342345eb7b022fb3686", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "eaee57eaf175b35a1a769cc830daf8b48cc2fc4fb2249342345eb7b022fb3686", kill_on_drop: false }`
[INFO] [stdout] eaee57eaf175b35a1a769cc830daf8b48cc2fc4fb2249342345eb7b022fb3686
