[INFO] cloning repository https://github.com/renatoathaydes/wasmin-lang
[INFO] running `Command { std: "git" "-c" "credential.helper=" "-c" "credential.helper=/workspace/cargo-home/bin/git-credential-null" "clone" "--bare" "https://github.com/renatoathaydes/wasmin-lang" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Frenatoathaydes%2Fwasmin-lang", kill_on_drop: false }`
[INFO] [stderr] Cloning into bare repository '/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Frenatoathaydes%2Fwasmin-lang'...
[INFO] running `Command { std: "git" "rev-parse" "HEAD", kill_on_drop: false }`
[INFO] [stdout] 3856ffdd5959e0d3f397fc1334b7bdc2154ec9c0
[INFO] checking renatoathaydes/wasmin-lang 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%2Frenatoathaydes%2Fwasmin-lang" "/workspace/builds/worker-4-tc2/source", kill_on_drop: false }`
[INFO] [stderr] Cloning into '/workspace/builds/worker-4-tc2/source'...
[INFO] [stderr] done.
[INFO] started tweaking git repo https://github.com/renatoathaydes/wasmin-lang
[INFO] finished tweaking git repo https://github.com/renatoathaydes/wasmin-lang
[INFO] tweaked toml for git repo https://github.com/renatoathaydes/wasmin-lang written to /workspace/builds/worker-4-tc2/source/Cargo.toml
[INFO] validating manifest of git repo https://github.com/renatoathaydes/wasmin-lang 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/renatoathaydes/wasmin-lang 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] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-4-tc2/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-4-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] 045fc2af9ed6a24feffef780e75810cee43a31b64a7e6e5fedd73864119c52a2
[INFO] running `Command { std: "docker" "start" "-a" "045fc2af9ed6a24feffef780e75810cee43a31b64a7e6e5fedd73864119c52a2", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "045fc2af9ed6a24feffef780e75810cee43a31b64a7e6e5fedd73864119c52a2", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "045fc2af9ed6a24feffef780e75810cee43a31b64a7e6e5fedd73864119c52a2", kill_on_drop: false }`
[INFO] [stdout] 045fc2af9ed6a24feffef780e75810cee43a31b64a7e6e5fedd73864119c52a2
[INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-4-tc2/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-4-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] 58f9ae2d3666d21487e0c2c4b18cccd3d729b55a0a958b148f91d42be0259d17
[INFO] running `Command { std: "docker" "start" "-a" "58f9ae2d3666d21487e0c2c4b18cccd3d729b55a0a958b148f91d42be0259d17", kill_on_drop: false }`
[INFO] [stderr]    Compiling syn v1.0.55
[INFO] [stderr]    Compiling libc v0.2.81
[INFO] [stderr]    Compiling unicode-segmentation v1.7.1
[INFO] [stderr]     Checking unicode-width v0.1.8
[INFO] [stderr]     Checking leb128 v0.2.4
[INFO] [stderr]     Checking wasmparser v0.72.0
[INFO] [stderr]    Compiling proc-macro-error-attr v1.0.4
[INFO] [stderr]    Compiling proc-macro-error v1.0.4
[INFO] [stderr]    Compiling quote v1.0.8
[INFO] [stderr]     Checking textwrap v0.11.0
[INFO] [stderr]     Checking wasm-encoder v0.4.0
[INFO] [stderr]    Compiling heck v0.3.2
[INFO] [stderr]     Checking atty v0.2.14
[INFO] [stderr]     Checking clap v2.33.3
[INFO] [stderr]    Compiling structopt-derive v0.4.14
[INFO] [stderr]     Checking structopt v0.3.21
[INFO] [stderr]     Checking wasmin v0.1.0 (/opt/rustwide/workdir)
[INFO] [stdout] warning: variable does not need to be mutable
[INFO] [stdout]   --> src/parse/stack.rs:59:21
[INFO] [stdout]    |
[INFO] [stdout] 59 |         if let Some(mut entry) = self.get_entry(&id) {
[INFO] [stdout]    |                     ----^^^^^
[INFO] [stdout]    |                     |
[INFO] [stdout]    |                     help: remove this `mut`
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(unused_mut)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `fun_index`
[INFO] [stdout]    --> src/sink/wasm.rs:282:17
[INFO] [stdout]     |
[INFO] [stdout] 282 |                 fun_index,
[INFO] [stdout]     |                 ^^^^^^^^^ help: try ignoring the field: `fun_index: _`
[INFO] [stdout]     |
[INFO] [stdout]     = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `type_conversion`
[INFO] [stdout]   --> src/sink/wat.rs:53:28
[INFO] [stdout]    |
[INFO] [stdout] 53 |         let (names, value, type_conversion) = assignment;
[INFO] [stdout]    |                            ^^^^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_type_conversion`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `type_conversion`
[INFO] [stdout]   --> src/sink/wat.rs:95:28
[INFO] [stdout]    |
[INFO] [stdout] 95 |         let (names, value, type_conversion) = assignment;
[INFO] [stdout]    |                            ^^^^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_type_conversion`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]   --> src/sink/wasm_utils.rs:33:37
[INFO] [stdout]    |
[INFO] [stdout] 33 | pub fn to_const(typ: ValType, text: &str) -> Instruction {
[INFO] [stdout]    |                                     ^^^^     ^^^^^^^^^^^ the same lifetime is hidden here
[INFO] [stdout]    |                                     |
[INFO] [stdout]    |                                     the lifetime is elided here
[INFO] [stdout]    |
[INFO] [stdout]    = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
[INFO] [stdout]    = note: `#[warn(mismatched_lifetime_syntaxes)]` on by default
[INFO] [stdout] help: use `'_` for type paths
[INFO] [stdout]    |
[INFO] [stdout] 33 | pub fn to_const(typ: ValType, text: &str) -> Instruction<'_> {
[INFO] [stdout]    |                                                         ++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variable does not need to be mutable
[INFO] [stdout]   --> src/parse/stack.rs:59:21
[INFO] [stdout]    |
[INFO] [stdout] 59 |         if let Some(mut entry) = self.get_entry(&id) {
[INFO] [stdout]    |                     ----^^^^^
[INFO] [stdout]    |                     |
[INFO] [stdout]    |                     help: remove this `mut`
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(unused_mut)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused `Result` that must be used
[INFO] [stdout]    --> src/bin/wasmin.rs:115:5
[INFO] [stdout]     |
[INFO] [stdout] 115 | /     sink_to_receiver(sink, rcvr, file, writer).map_err(|e| match e {
[INFO] [stdout] 116 | |         WError::IO(e) => exit_with_error!(3, "{}", e),
[INFO] [stdout] 117 | |         WError::Wasmin(werr) => {
[INFO] [stdout] 118 | |             let relevant_text = werr.relevant_text(text);
[INFO] [stdout] ...   |
[INFO] [stdout] 121 | |         WError::Validation(e) => exit_with_error!(1, "{}", e),
[INFO] [stdout] 122 | |     });
[INFO] [stdout]     | |______^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this `Result` may be an `Err` variant, which should be handled
[INFO] [stdout]     = note: `#[warn(unused_must_use)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] help: use `let _ = ...` to ignore the resulting value
[INFO] [stdout]     |
[INFO] [stdout] 115 |     let _ = sink_to_receiver(sink, rcvr, file, writer).map_err(|e| match e {
[INFO] [stdout]     |     +++++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: this type will no longer be must used: Result<String, Infallible>
[INFO] [stdout]    --> src/bin/wasmin.rs:176:9
[INFO] [stdout]     |
[INFO] [stdout] 176 | /         /// Wasmin source file. If not given, the source is read from stdin.
[INFO] [stdout] 177 | |         input: Option<String>,
[INFO] [stdout]     | |_____________________________^
[INFO] [stdout]     |
[INFO] [stdout]     = note: requested on the command line with `-D unmustuse-in-always-ok`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused `Result` that must be used
[INFO] [stdout]    --> src/bin/wasmin.rs:115:5
[INFO] [stdout]     |
[INFO] [stdout] 115 | /     sink_to_receiver(sink, rcvr, file, writer).map_err(|e| match e {
[INFO] [stdout] 116 | |         WError::IO(e) => exit_with_error!(3, "{}", e),
[INFO] [stdout] 117 | |         WError::Wasmin(werr) => {
[INFO] [stdout] 118 | |             let relevant_text = werr.relevant_text(text);
[INFO] [stdout] ...   |
[INFO] [stdout] 121 | |         WError::Validation(e) => exit_with_error!(1, "{}", e),
[INFO] [stdout] 122 | |     });
[INFO] [stdout]     | |______^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this `Result` may be an `Err` variant, which should be handled
[INFO] [stdout]     = note: `#[warn(unused_must_use)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] help: use `let _ = ...` to ignore the resulting value
[INFO] [stdout]     |
[INFO] [stdout] 115 |     let _ = sink_to_receiver(sink, rcvr, file, writer).map_err(|e| match e {
[INFO] [stdout]     |     +++++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: this type will no longer be must used: Result<String, Infallible>
[INFO] [stdout]    --> src/bin/wasmin.rs:178:9
[INFO] [stdout]     |
[INFO] [stdout] 178 | /         #[structopt(
[INFO] [stdout] 179 | |         short = "o",
[INFO] [stdout] 180 | |         long = "output",
[INFO] [stdout] 181 | |         help = "output file. If not given, the output is sent to stdout."
[INFO] [stdout] 182 | |         )]
[INFO] [stdout] 183 | |         output: Option<String>,
[INFO] [stdout]     | |______________________________^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: this type will no longer be must used: Result<String, Infallible>
[INFO] [stdout]    --> src/bin/wasmin.rs:189:9
[INFO] [stdout]     |
[INFO] [stdout] 189 | /         /// WASM file.
[INFO] [stdout] 190 | |         file: String,
[INFO] [stdout]     | |____________________^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: this type will no longer be must used: Result<String, Infallible>
[INFO] [stdout]    --> src/bin/wasmin.rs:176:9
[INFO] [stdout]     |
[INFO] [stdout] 176 | /         /// Wasmin source file. If not given, the source is read from stdin.
[INFO] [stdout] 177 | |         input: Option<String>,
[INFO] [stdout]     | |_____________________________^
[INFO] [stdout]     |
[INFO] [stdout]     = note: requested on the command line with `-D unmustuse-in-always-ok`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: this type will no longer be must used: Result<String, Infallible>
[INFO] [stdout]    --> src/bin/wasmin.rs:178:9
[INFO] [stdout]     |
[INFO] [stdout] 178 | /         #[structopt(
[INFO] [stdout] 179 | |         short = "o",
[INFO] [stdout] 180 | |         long = "output",
[INFO] [stdout] 181 | |         help = "output file. If not given, the output is sent to stdout."
[INFO] [stdout] 182 | |         )]
[INFO] [stdout] 183 | |         output: Option<String>,
[INFO] [stdout]     | |______________________________^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: this type will no longer be must used: Result<String, Infallible>
[INFO] [stdout]    --> src/bin/wasmin.rs:189:9
[INFO] [stdout]     |
[INFO] [stdout] 189 | /         /// WASM file.
[INFO] [stdout] 190 | |         file: String,
[INFO] [stdout]     | |____________________^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr] error: could not compile `wasmin` (bin "wasmin") due to 3 previous errors; 1 warning emitted
[INFO] [stderr] warning: build failed, waiting for other jobs to finish...
[INFO] [stderr] error: could not compile `wasmin` (bin "wasmin" test) due to 3 previous errors; 1 warning emitted
[INFO] [stdout] warning: panic message is not a string literal
[INFO] [stdout]    --> tests/parser_tests.rs:217:24
[INFO] [stdout]     |
[INFO] [stdout] 217 |                 panic!(format!("Did not get Ext, got {:?}", e));
[INFO] [stdout]     |                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this usage of `panic!()` is deprecated; it will be a hard error in Rust 2021
[INFO] [stdout]     = note: for more information, see <https://doc.rust-lang.org/edition-guide/rust-2021/panic-macro-consistency.html>
[INFO] [stdout]     = note: the `panic!()` macro supports formatting, so there's no need for the `format!()` macro here
[INFO] [stdout]     = note: `#[warn(non_fmt_panics)]` (part of `#[warn(rust_2021_compatibility)]`) on by default
[INFO] [stdout] help: remove the `format!(..)` macro call
[INFO] [stdout]     |
[INFO] [stdout] 217 -                 panic!(format!("Did not get Ext, got {:?}", e));
[INFO] [stdout] 217 +                 panic!("Did not get Ext, got {:?}", e);
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `fun_index`
[INFO] [stdout]    --> src/sink/wasm.rs:282:17
[INFO] [stdout]     |
[INFO] [stdout] 282 |                 fun_index,
[INFO] [stdout]     |                 ^^^^^^^^^ help: try ignoring the field: `fun_index: _`
[INFO] [stdout]     |
[INFO] [stdout]     = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `type_conversion`
[INFO] [stdout]   --> src/sink/wat.rs:53:28
[INFO] [stdout]    |
[INFO] [stdout] 53 |         let (names, value, type_conversion) = assignment;
[INFO] [stdout]    |                            ^^^^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_type_conversion`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `type_conversion`
[INFO] [stdout]   --> src/sink/wat.rs:95:28
[INFO] [stdout]    |
[INFO] [stdout] 95 |         let (names, value, type_conversion) = assignment;
[INFO] [stdout]    |                            ^^^^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_type_conversion`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]   --> src/sink/wasm_utils.rs:33:37
[INFO] [stdout]    |
[INFO] [stdout] 33 | pub fn to_const(typ: ValType, text: &str) -> Instruction {
[INFO] [stdout]    |                                     ^^^^     ^^^^^^^^^^^ the same lifetime is hidden here
[INFO] [stdout]    |                                     |
[INFO] [stdout]    |                                     the lifetime is elided here
[INFO] [stdout]    |
[INFO] [stdout]    = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
[INFO] [stdout]    = note: `#[warn(mismatched_lifetime_syntaxes)]` on by default
[INFO] [stdout] help: use `'_` for type paths
[INFO] [stdout]    |
[INFO] [stdout] 33 | pub fn to_const(typ: ValType, text: &str) -> Instruction<'_> {
[INFO] [stdout]    |                                                         ++++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] running `Command { std: "docker" "inspect" "58f9ae2d3666d21487e0c2c4b18cccd3d729b55a0a958b148f91d42be0259d17", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "58f9ae2d3666d21487e0c2c4b18cccd3d729b55a0a958b148f91d42be0259d17", kill_on_drop: false }`
[INFO] [stdout] 58f9ae2d3666d21487e0c2c4b18cccd3d729b55a0a958b148f91d42be0259d17
