[INFO] fetching crate sato 0.1.10...
[INFO] checking sato-0.1.10 against 52995a223f2919dd3e2e427231db90a257350ded for pr-129249-4
[INFO] extracting crate sato 0.1.10 into /workspace/builds/worker-7-tc2/source
[INFO] started tweaking crates.io crate sato 0.1.10
[INFO] finished tweaking crates.io crate sato 0.1.10
[INFO] tweaked toml for crates.io crate sato 0.1.10 written to /workspace/builds/worker-7-tc2/source/Cargo.toml
[INFO] validating manifest of crates.io crate sato 0.1.10 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 7 packages to highest compatible versions
[INFO] [stderr]       Adding thiserror v1.0.69 (available: v2.0.21)
[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-7-tc2/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-7-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] 55b9b63dbdd671ac733237b090d2931aafd792d71bf9696667ae8153960e2093
[INFO] running `Command { std: "docker" "start" "55b9b63dbdd671ac733237b090d2931aafd792d71bf9696667ae8153960e2093", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "55b9b63dbdd671ac733237b090d2931aafd792d71bf9696667ae8153960e2093", 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" "55b9b63dbdd671ac733237b090d2931aafd792d71bf9696667ae8153960e2093" "/opt/rustwide/cargo-home/bin/cargo" "+52995a223f2919dd3e2e427231db90a257350ded" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "55b9b63dbdd671ac733237b090d2931aafd792d71bf9696667ae8153960e2093", 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" "55b9b63dbdd671ac733237b090d2931aafd792d71bf9696667ae8153960e2093" "/opt/rustwide/cargo-home/bin/cargo" "+52995a223f2919dd3e2e427231db90a257350ded" "check" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }`
[INFO] [stderr]     Checking sexp v1.1.4
[INFO] [stderr]    Compiling syn v2.0.119
[INFO] [stderr]    Compiling thiserror-impl v1.0.69
[INFO] [stderr]     Checking thiserror v1.0.69
[INFO] [stderr]     Checking sato v0.1.10 (/opt/rustwide/workdir)
[INFO] [stdout] error: useless conversion to the same type: `RenderValue`
[INFO] [stdout]  --> src/builtins.rs:8:12
[INFO] [stdout]   |
[INFO] [stdout] 8 |     v.push(basic_html_tag("html".into(), &attrs, &expr, renderer, context)?.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] 8 -     v.push(basic_html_tag("html".into(), &attrs, &expr, renderer, context)?.into());
[INFO] [stdout] 8 +     v.push(basic_html_tag("html".into(), &attrs, &expr, renderer, context)?);
[INFO] [stdout]   |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]   |
[INFO] [stdout] 8 -     v.push(basic_html_tag("html".into(), &attrs, &expr, renderer, context)?.into());
[INFO] [stdout] 8 +     v.push(<RenderValue as Into<_>>::into(basic_html_tag("html".into(), &attrs, &expr, renderer, context)?));
[INFO] [stdout]   |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `RenderValue`
[INFO] [stdout]    --> src/renderer.rs:272:16
[INFO] [stdout]     |
[INFO] [stdout] 272 |         l.push(renderer.evaluate_multiple(expr, context)?.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] 272 -         l.push(renderer.evaluate_multiple(expr, context)?.into());
[INFO] [stdout] 272 +         l.push(renderer.evaluate_multiple(expr, context)?);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 272 -         l.push(renderer.evaluate_multiple(expr, context)?.into());
[INFO] [stdout] 272 +         l.push(<RenderValue as Into<_>>::into(renderer.evaluate_multiple(expr, context)?));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `String`
[INFO] [stdout]   --> src/template.rs:36:89
[INFO] [stdout]    |
[INFO] [stdout] 36 |         Ok(parse_expr(&sexp::parse(&other).map_err(|err| TemplateError::ParseError(err, other.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] 36 -         Ok(parse_expr(&sexp::parse(&other).map_err(|err| TemplateError::ParseError(err, other.into()))?)?)
[INFO] [stdout] 36 +         Ok(parse_expr(&sexp::parse(&other).map_err(|err| TemplateError::ParseError(err, other))?)?)
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 36 -         Ok(parse_expr(&sexp::parse(&other).map_err(|err| TemplateError::ParseError(err, other.into()))?)?)
[INFO] [stdout] 36 +         Ok(parse_expr(&sexp::parse(&other).map_err(|err| TemplateError::ParseError(err, <String as Into<_>>::into(other)))?)?)
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr] error: could not compile `sato` (lib) due to 3 previous errors
[INFO] [stderr] warning: build failed, waiting for other jobs to finish...
[INFO] [stdout] error: useless conversion to the same type: `RenderValue`
[INFO] [stdout]  --> src/builtins.rs:8:12
[INFO] [stdout]   |
[INFO] [stdout] 8 |     v.push(basic_html_tag("html".into(), &attrs, &expr, renderer, context)?.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] 8 -     v.push(basic_html_tag("html".into(), &attrs, &expr, renderer, context)?.into());
[INFO] [stdout] 8 +     v.push(basic_html_tag("html".into(), &attrs, &expr, renderer, context)?);
[INFO] [stdout]   |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]   |
[INFO] [stdout] 8 -     v.push(basic_html_tag("html".into(), &attrs, &expr, renderer, context)?.into());
[INFO] [stdout] 8 +     v.push(<RenderValue as Into<_>>::into(basic_html_tag("html".into(), &attrs, &expr, renderer, context)?));
[INFO] [stdout]   |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `RenderValue`
[INFO] [stdout]    --> src/renderer.rs:272:16
[INFO] [stdout]     |
[INFO] [stdout] 272 |         l.push(renderer.evaluate_multiple(expr, context)?.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] 272 -         l.push(renderer.evaluate_multiple(expr, context)?.into());
[INFO] [stdout] 272 +         l.push(renderer.evaluate_multiple(expr, context)?);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 272 -         l.push(renderer.evaluate_multiple(expr, context)?.into());
[INFO] [stdout] 272 +         l.push(<RenderValue as Into<_>>::into(renderer.evaluate_multiple(expr, context)?));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `String`
[INFO] [stdout]   --> src/template.rs:36:89
[INFO] [stdout]    |
[INFO] [stdout] 36 |         Ok(parse_expr(&sexp::parse(&other).map_err(|err| TemplateError::ParseError(err, other.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] 36 -         Ok(parse_expr(&sexp::parse(&other).map_err(|err| TemplateError::ParseError(err, other.into()))?)?)
[INFO] [stdout] 36 +         Ok(parse_expr(&sexp::parse(&other).map_err(|err| TemplateError::ParseError(err, other))?)?)
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 36 -         Ok(parse_expr(&sexp::parse(&other).map_err(|err| TemplateError::ParseError(err, other.into()))?)?)
[INFO] [stdout] 36 +         Ok(parse_expr(&sexp::parse(&other).map_err(|err| TemplateError::ParseError(err, <String as Into<_>>::into(other)))?)?)
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `RenderValue`
[INFO] [stdout]    --> src/lib.rs:491:29
[INFO] [stdout]     |
[INFO] [stdout] 491 |                 output.push(renderer.evaluate_multiple(expr, context)?.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] 491 -                 output.push(renderer.evaluate_multiple(expr, context)?.into());
[INFO] [stdout] 491 +                 output.push(renderer.evaluate_multiple(expr, context)?);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 491 -                 output.push(renderer.evaluate_multiple(expr, context)?.into());
[INFO] [stdout] 491 +                 output.push(<RenderValue as Into<_>>::into(renderer.evaluate_multiple(expr, context)?));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `RenderValue`
[INFO] [stdout]    --> src/lib.rs:562:29
[INFO] [stdout]     |
[INFO] [stdout] 562 |                 output.push(renderer.evaluate_multiple(expr, context)?.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] 562 -                 output.push(renderer.evaluate_multiple(expr, context)?.into());
[INFO] [stdout] 562 +                 output.push(renderer.evaluate_multiple(expr, context)?);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 562 -                 output.push(renderer.evaluate_multiple(expr, context)?.into());
[INFO] [stdout] 562 +                 output.push(<RenderValue as Into<_>>::into(renderer.evaluate_multiple(expr, context)?));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr] error: could not compile `sato` (lib test) due to 5 previous errors
[INFO] running `Command { std: "docker" "inspect" "55b9b63dbdd671ac733237b090d2931aafd792d71bf9696667ae8153960e2093", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "55b9b63dbdd671ac733237b090d2931aafd792d71bf9696667ae8153960e2093", kill_on_drop: false }`
[INFO] [stdout] 55b9b63dbdd671ac733237b090d2931aafd792d71bf9696667ae8153960e2093
