[INFO] cloning repository https://github.com/FlySkyPie/biomes-galois-shader
[INFO] running `Command { std: "git" "-c" "credential.helper=" "-c" "credential.helper=/workspace/cargo-home/bin/git-credential-null" "clone" "--bare" "https://github.com/FlySkyPie/biomes-galois-shader" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2FFlySkyPie%2Fbiomes-galois-shader", kill_on_drop: false }`
[INFO] [stderr] Cloning into bare repository '/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2FFlySkyPie%2Fbiomes-galois-shader'...
[INFO] running `Command { std: "git" "rev-parse" "HEAD", kill_on_drop: false }`
[INFO] [stdout] b00156adfc053c7896b1174b345aa0eacb3bfdce
[INFO] checking FlySkyPie/biomes-galois-shader against c60b92c03ca8a3b101eb9d57f51e2232690638c7 for pr-164148-crater-rollup
[INFO] running `Command { std: "git" "clone" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2FFlySkyPie%2Fbiomes-galois-shader" "/workspace/builds/worker-1-tc2/source", kill_on_drop: false }`
[INFO] [stderr] Cloning into '/workspace/builds/worker-1-tc2/source'...
[INFO] [stderr] done.
[INFO] started tweaking git repo https://github.com/FlySkyPie/biomes-galois-shader
[INFO] finished tweaking git repo https://github.com/FlySkyPie/biomes-galois-shader
[INFO] tweaked toml for git repo https://github.com/FlySkyPie/biomes-galois-shader written to /workspace/builds/worker-1-tc2/source/Cargo.toml
[INFO] validating manifest of git repo https://github.com/FlySkyPie/biomes-galois-shader on toolchain c60b92c03ca8a3b101eb9d57f51e2232690638c7
[INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+c60b92c03ca8a3b101eb9d57f51e2232690638c7" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }`
[INFO] crate git repo https://github.com/FlySkyPie/biomes-galois-shader 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" "+c60b92c03ca8a3b101eb9d57f51e2232690638c7" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-1-tc2/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-1-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] de9092c0617ae9b50b523927f741574e93c17eaeefd6fff0c5d30d99cad3d10a
[INFO] running `Command { std: "docker" "start" "de9092c0617ae9b50b523927f741574e93c17eaeefd6fff0c5d30d99cad3d10a", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "de9092c0617ae9b50b523927f741574e93c17eaeefd6fff0c5d30d99cad3d10a", 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" "de9092c0617ae9b50b523927f741574e93c17eaeefd6fff0c5d30d99cad3d10a" "/opt/rustwide/cargo-home/bin/cargo" "+c60b92c03ca8a3b101eb9d57f51e2232690638c7" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "de9092c0617ae9b50b523927f741574e93c17eaeefd6fff0c5d30d99cad3d10a", 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" "de9092c0617ae9b50b523927f741574e93c17eaeefd6fff0c5d30d99cad3d10a" "/opt/rustwide/cargo-home/bin/cargo" "+c60b92c03ca8a3b101eb9d57f51e2232690638c7" "check" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }`
[INFO] [stderr]     Checking nom v7.1.3
[INFO] [stderr]     Checking convert_case v0.10.0
[INFO] [stderr]    Compiling syn v2.0.111
[INFO] [stderr]     Checking glsl v7.0.0
[INFO] [stderr]    Compiling clap_derive v4.5.49
[INFO] [stderr]    Compiling serde_derive v1.0.228
[INFO] [stderr]     Checking clap v4.5.53
[INFO] [stderr]     Checking serde v1.0.228
[INFO] [stderr]     Checking generator v0.1.0 (/opt/rustwide/workdir)
[INFO] [stdout] warning: unused variable: `name`
[INFO] [stdout]    --> src/main.rs:293:38
[INFO] [stdout]     |
[INFO] [stdout] 293 | fn gen_hot_reload_init(galois: bool, name: &String) -> String {
[INFO] [stdout]     |                                      ^^^^ help: if this is intentional, prefix it with an underscore: `_name`
[INFO] [stdout]     |
[INFO] [stdout]     = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: field `in_varyings` is never read
[INFO] [stdout]    --> src/shader_link_info.rs:151:9
[INFO] [stdout]     |
[INFO] [stdout] 149 | pub struct ProgramLinkInfo {
[INFO] [stdout]     |            --------------- field in this struct
[INFO] [stdout] 150 |     pub uniforms: VarMap,
[INFO] [stdout] 151 |     pub in_varyings: VarMap,
[INFO] [stdout]     |         ^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default
[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/shader_link_info.rs:46:5
[INFO] [stdout]    |
[INFO] [stdout] 32 | / pub fn parse_shader_link_info(
[INFO] [stdout] 33 | |     source: &String,
[INFO] [stdout] 34 | |     shader_type: ShaderType,
[INFO] [stdout] 35 | | ) -> Result<ShaderLinkInfo, Box<dyn Error>> {
[INFO] [stdout]    | |___________________________________________- move the `impl` block outside of this function `parse_shader_link_info`
[INFO] [stdout] ...
[INFO] [stdout] 46 |       impl glsl::visitor::Visitor for ShaderLinkInfo {
[INFO] [stdout]    |       ^^^^^----------------------^^^^^--------------
[INFO] [stdout]    |            |                          |
[INFO] [stdout]    |            |                          `ShaderLinkInfo` is not local
[INFO] [stdout]    |            `Visitor` is not local
[INFO] [stdout]    |
[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: `#[warn(non_local_definitions)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `std::string::String` generator
[INFO] [stdout]    --> src/shader_link_info.rs:179:24
[INFO] [stdout]     |
[INFO] [stdout] 179 |               return Err(format!(
[INFO] [stdout]     |  ________________________^
[INFO] [stdout] 180 | |                 "Could not find fragment input varying '{}' as vertex output varying.",
[INFO] [stdout] 181 | |                 fs_in_varying.name
[INFO] [stdout] 182 | |             )
[INFO] [stdout] 183 | |             .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] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `std::string::String` generator
[INFO] [stdout]    --> src/shader_link_info.rs:186:24
[INFO] [stdout]     |
[INFO] [stdout] 186 |               return Err(format!(
[INFO] [stdout]     |  ________________________^
[INFO] [stdout] 187 | |                 "Type mismatch for varying '{}' between vertex and fragment shader.",
[INFO] [stdout] 188 | |                 fs_in_varying.name
[INFO] [stdout] 189 | |             )
[INFO] [stdout] 190 | |             .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] 
[INFO] [stdout] 
[INFO] [stderr] error: could not compile `generator` (bin "generator") due to 2 previous errors; 3 warnings emitted
[INFO] [stderr] warning: build failed, waiting for other jobs to finish...
[INFO] [stdout] warning: unused variable: `name`
[INFO] [stdout]    --> src/main.rs:293:38
[INFO] [stdout]     |
[INFO] [stdout] 293 | fn gen_hot_reload_init(galois: bool, name: &String) -> String {
[INFO] [stdout]     |                                      ^^^^ help: if this is intentional, prefix it with an underscore: `_name`
[INFO] [stdout]     |
[INFO] [stdout]     = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: field `in_varyings` is never read
[INFO] [stdout]    --> src/shader_link_info.rs:151:9
[INFO] [stdout]     |
[INFO] [stdout] 149 | pub struct ProgramLinkInfo {
[INFO] [stdout]     |            --------------- field in this struct
[INFO] [stdout] 150 |     pub uniforms: VarMap,
[INFO] [stdout] 151 |     pub in_varyings: VarMap,
[INFO] [stdout]     |         ^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default
[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/shader_link_info.rs:46:5
[INFO] [stdout]    |
[INFO] [stdout] 32 | / pub fn parse_shader_link_info(
[INFO] [stdout] 33 | |     source: &String,
[INFO] [stdout] 34 | |     shader_type: ShaderType,
[INFO] [stdout] 35 | | ) -> Result<ShaderLinkInfo, Box<dyn Error>> {
[INFO] [stdout]    | |___________________________________________- move the `impl` block outside of this function `parse_shader_link_info`
[INFO] [stdout] ...
[INFO] [stdout] 46 |       impl glsl::visitor::Visitor for ShaderLinkInfo {
[INFO] [stdout]    |       ^^^^^----------------------^^^^^--------------
[INFO] [stdout]    |            |                          |
[INFO] [stdout]    |            |                          `ShaderLinkInfo` is not local
[INFO] [stdout]    |            `Visitor` is not local
[INFO] [stdout]    |
[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: `#[warn(non_local_definitions)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `std::string::String` generator
[INFO] [stdout]    --> src/shader_link_info.rs:179:24
[INFO] [stdout]     |
[INFO] [stdout] 179 |               return Err(format!(
[INFO] [stdout]     |  ________________________^
[INFO] [stdout] 180 | |                 "Could not find fragment input varying '{}' as vertex output varying.",
[INFO] [stdout] 181 | |                 fs_in_varying.name
[INFO] [stdout] 182 | |             )
[INFO] [stdout] 183 | |             .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] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `std::string::String` generator
[INFO] [stdout]    --> src/shader_link_info.rs:186:24
[INFO] [stdout]     |
[INFO] [stdout] 186 |               return Err(format!(
[INFO] [stdout]     |  ________________________^
[INFO] [stdout] 187 | |                 "Type mismatch for varying '{}' between vertex and fragment shader.",
[INFO] [stdout] 188 | |                 fs_in_varying.name
[INFO] [stdout] 189 | |             )
[INFO] [stdout] 190 | |             .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] 
[INFO] [stdout] 
[INFO] [stderr] error: could not compile `generator` (bin "generator" test) due to 2 previous errors; 3 warnings emitted
[INFO] running `Command { std: "docker" "inspect" "de9092c0617ae9b50b523927f741574e93c17eaeefd6fff0c5d30d99cad3d10a", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "de9092c0617ae9b50b523927f741574e93c17eaeefd6fff0c5d30d99cad3d10a", kill_on_drop: false }`
[INFO] [stdout] de9092c0617ae9b50b523927f741574e93c17eaeefd6fff0c5d30d99cad3d10a
