[INFO] cloning repository https://github.com/est31/cargo-ulint [INFO] running `Command { std: "git" "-c" "credential.helper=" "-c" "credential.helper=/workspace/cargo-home/bin/git-credential-null" "clone" "--bare" "https://github.com/est31/cargo-ulint" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fest31%2Fcargo-ulint", kill_on_drop: false }` [INFO] [stderr] Cloning into bare repository '/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fest31%2Fcargo-ulint'... [INFO] running `Command { std: "git" "rev-parse" "HEAD", kill_on_drop: false }` [INFO] [stdout] ffc088ecb632a539e6e66eff7c69f0fb6b7e969b [INFO] checking est31/cargo-ulint against try#4f260f0f20b3133d20cfb50353c0221943af5796+rustflags=-Dunmustuse_in_always_ok for pr-148577 [INFO] running `Command { std: "git" "clone" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fest31%2Fcargo-ulint" "/workspace/builds/worker-0-tc2/source", kill_on_drop: false }` [INFO] [stderr] Cloning into '/workspace/builds/worker-0-tc2/source'... [INFO] [stderr] done. [INFO] started tweaking git repo https://github.com/est31/cargo-ulint [INFO] finished tweaking git repo https://github.com/est31/cargo-ulint [INFO] tweaked toml for git repo https://github.com/est31/cargo-ulint written to /workspace/builds/worker-0-tc2/source/Cargo.toml [INFO] validating manifest of git repo https://github.com/est31/cargo-ulint on toolchain 4f260f0f20b3133d20cfb50353c0221943af5796 [INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+4f260f0f20b3133d20cfb50353c0221943af5796" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }` [INFO] crate git repo https://github.com/est31/cargo-ulint 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" "+4f260f0f20b3133d20cfb50353c0221943af5796" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }` [INFO] [stderr] warning: no edition set: defaulting to the 2015 edition while the latest is 2024 [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-0-tc2/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-0-tc2/source:/opt/rustwide/workdir:ro,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" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-w" "/opt/rustwide/workdir" "-m" "1610612736" "--user" "0:0" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:4848fb76d95f26979359cc7e45710b1dbc8f3acb7aeedee7c460d7702230f228" "/opt/rustwide/cargo-home/bin/cargo" "+4f260f0f20b3133d20cfb50353c0221943af5796" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }` [INFO] [stdout] cd925a59755e10bc5d79b337ce74eb8d293c59ff62ae851d5e9aa8cfbd89ab42 [INFO] running `Command { std: "docker" "start" "-a" "cd925a59755e10bc5d79b337ce74eb8d293c59ff62ae851d5e9aa8cfbd89ab42", kill_on_drop: false }` [INFO] running `Command { std: "docker" "inspect" "cd925a59755e10bc5d79b337ce74eb8d293c59ff62ae851d5e9aa8cfbd89ab42", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "cd925a59755e10bc5d79b337ce74eb8d293c59ff62ae851d5e9aa8cfbd89ab42", kill_on_drop: false }` [INFO] [stdout] cd925a59755e10bc5d79b337ce74eb8d293c59ff62ae851d5e9aa8cfbd89ab42 [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-0-tc2/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-0-tc2/source:/opt/rustwide/workdir:ro,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" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-e" "CARGO_INCREMENTAL=0" "-e" "RUST_BACKTRACE=full" "-e" "RUSTFLAGS=--cap-lints=forbid -Dunmustuse_in_always_ok" "-e" "RUSTDOCFLAGS=--cap-lints=forbid" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-w" "/opt/rustwide/workdir" "-m" "1610612736" "--user" "0:0" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:4848fb76d95f26979359cc7e45710b1dbc8f3acb7aeedee7c460d7702230f228" "/opt/rustwide/cargo-home/bin/cargo" "+4f260f0f20b3133d20cfb50353c0221943af5796" "check" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }` [INFO] [stdout] f9da1a29f84310f1607a6660b68513b2938dd8375a4d550b3bb1b2cca05b83fe [INFO] running `Command { std: "docker" "start" "-a" "f9da1a29f84310f1607a6660b68513b2938dd8375a4d550b3bb1b2cca05b83fe", kill_on_drop: false }` [INFO] [stderr] warning: no edition set: defaulting to the 2015 edition while the latest is 2024 [INFO] [stderr] Checking libc v0.2.34 [INFO] [stderr] Checking strsim v0.6.0 [INFO] [stderr] Compiling structopt-derive v0.1.6 [INFO] [stderr] Checking atty v0.2.6 [INFO] [stderr] Checking clap v2.29.0 [INFO] [stderr] Checking structopt v0.1.6 [INFO] [stderr] Checking cargo-ulint v0.1.0 (/opt/rustwide/workdir) [INFO] [stdout] warning: unused variable: `Options` [INFO] [stdout] --> src/main.rs:16:6 [INFO] [stdout] | [INFO] [stdout] 16 | let Options = Options::from_args(); [INFO] [stdout] | ^^^^^^^ help: if this is intentional, prefix it with an underscore: `_Options` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: field `toml` is never read [INFO] [stdout] --> src/main.rs:12:2 [INFO] [stdout] | [INFO] [stdout] 9 | struct Options { [INFO] [stdout] | ------- field in this struct [INFO] [stdout] ... [INFO] [stdout] 12 | toml: Option, [INFO] [stdout] | ^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `Options` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis [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/main.rs:7:10 [INFO] [stdout] | [INFO] [stdout] 7 | #[derive(StructOpt, Debug)] [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | `Options` is not local [INFO] [stdout] | `StructOpt` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_IMPL_STRUCTOPT_FOR_Options` [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `StructOpt` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `StructOpt` may come from an old version of the `structopt_derive` crate, try updating your dependency with `cargo update -p structopt_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 `StructOpt` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: this type will no longer be must used: Result [INFO] [stdout] --> src/main.rs:7:10 [INFO] [stdout] | [INFO] [stdout] 7 | #[derive(StructOpt, Debug)] [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: requested on the command line with `-D unmustuse-in-always-ok` [INFO] [stdout] = note: this error originates in the derive macro `StructOpt` (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/main.rs:7:10 [INFO] [stdout] | [INFO] [stdout] 7 | #[derive(StructOpt, Debug)] [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | `Options` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_IMPL_STRUCTOPT_FOR_Options` [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `StructOpt` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `StructOpt` may come from an old version of the `structopt_derive` crate, try updating your dependency with `cargo update -p structopt_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 `StructOpt` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable `Options` should have a snake case name [INFO] [stdout] --> src/main.rs:16:6 [INFO] [stdout] | [INFO] [stdout] 16 | let Options = Options::from_args(); [INFO] [stdout] | ^^^^^^^ help: convert the identifier to snake case (notice the capitalization): `options` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(non_snake_case)]` (part of `#[warn(nonstandard_style)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stderr] error: could not compile `cargo-ulint` (bin "cargo-ulint" test) due to 1 previous error; 5 warnings emitted [INFO] [stderr] warning: build failed, waiting for other jobs to finish... [INFO] [stdout] warning: unused variable: `Options` [INFO] [stdout] --> src/main.rs:16:6 [INFO] [stdout] | [INFO] [stdout] 16 | let Options = Options::from_args(); [INFO] [stdout] | ^^^^^^^ help: if this is intentional, prefix it with an underscore: `_Options` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: field `toml` is never read [INFO] [stdout] --> src/main.rs:12:2 [INFO] [stdout] | [INFO] [stdout] 9 | struct Options { [INFO] [stdout] | ------- field in this struct [INFO] [stdout] ... [INFO] [stdout] 12 | toml: Option, [INFO] [stdout] | ^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `Options` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis [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/main.rs:7:10 [INFO] [stdout] | [INFO] [stdout] 7 | #[derive(StructOpt, Debug)] [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | `Options` is not local [INFO] [stdout] | `StructOpt` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_IMPL_STRUCTOPT_FOR_Options` [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `StructOpt` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `StructOpt` may come from an old version of the `structopt_derive` crate, try updating your dependency with `cargo update -p structopt_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 `StructOpt` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error: this type will no longer be must used: Result [INFO] [stdout] --> src/main.rs:7:10 [INFO] [stdout] | [INFO] [stdout] 7 | #[derive(StructOpt, Debug)] [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: requested on the command line with `-D unmustuse-in-always-ok` [INFO] [stdout] = note: this error originates in the derive macro `StructOpt` (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/main.rs:7:10 [INFO] [stdout] | [INFO] [stdout] 7 | #[derive(StructOpt, Debug)] [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | `Options` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_IMPL_STRUCTOPT_FOR_Options` [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `StructOpt` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `StructOpt` may come from an old version of the `structopt_derive` crate, try updating your dependency with `cargo update -p structopt_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 `StructOpt` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable `Options` should have a snake case name [INFO] [stdout] --> src/main.rs:16:6 [INFO] [stdout] | [INFO] [stdout] 16 | let Options = Options::from_args(); [INFO] [stdout] | ^^^^^^^ help: convert the identifier to snake case (notice the capitalization): `options` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(non_snake_case)]` (part of `#[warn(nonstandard_style)]`) on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stderr] error: could not compile `cargo-ulint` (bin "cargo-ulint") due to 1 previous error; 5 warnings emitted [INFO] running `Command { std: "docker" "inspect" "f9da1a29f84310f1607a6660b68513b2938dd8375a4d550b3bb1b2cca05b83fe", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "f9da1a29f84310f1607a6660b68513b2938dd8375a4d550b3bb1b2cca05b83fe", kill_on_drop: false }` [INFO] [stdout] f9da1a29f84310f1607a6660b68513b2938dd8375a4d550b3bb1b2cca05b83fe