[INFO] cloning repository https://github.com/ConradKhakhria/woah
[INFO] running `Command { std: "git" "-c" "credential.helper=" "-c" "credential.helper=/workspace/cargo-home/bin/git-credential-null" "clone" "--bare" "https://github.com/ConradKhakhria/woah" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2FConradKhakhria%2Fwoah", kill_on_drop: false }`
[INFO] [stderr] Cloning into bare repository '/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2FConradKhakhria%2Fwoah'...
[INFO] running `Command { std: "git" "rev-parse" "HEAD", kill_on_drop: false }`
[INFO] [stdout] 250c4fd462854567624d51793f357ecfc18a18f0
[INFO] checking ConradKhakhria/woah against a5689b034ad287112f709693384a37f1377eb9cc for pr-129249-3
[INFO] running `Command { std: "git" "clone" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2FConradKhakhria%2Fwoah" "/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/ConradKhakhria/woah
[INFO] finished tweaking git repo https://github.com/ConradKhakhria/woah
[INFO] tweaked toml for git repo https://github.com/ConradKhakhria/woah written to /workspace/builds/worker-1-tc2/source/Cargo.toml
[INFO] validating manifest of git repo https://github.com/ConradKhakhria/woah 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/ConradKhakhria/woah 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-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] b17ab3bf49cc18f066e36af36a5c12859eec4e0d15d1e5bd7f5e1a4ab76aba2b
[INFO] running `Command { std: "docker" "start" "b17ab3bf49cc18f066e36af36a5c12859eec4e0d15d1e5bd7f5e1a4ab76aba2b", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "b17ab3bf49cc18f066e36af36a5c12859eec4e0d15d1e5bd7f5e1a4ab76aba2b", 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" "b17ab3bf49cc18f066e36af36a5c12859eec4e0d15d1e5bd7f5e1a4ab76aba2b" "/opt/rustwide/cargo-home/bin/cargo" "+a5689b034ad287112f709693384a37f1377eb9cc" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "b17ab3bf49cc18f066e36af36a5c12859eec4e0d15d1e5bd7f5e1a4ab76aba2b", 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" "b17ab3bf49cc18f066e36af36a5c12859eec4e0d15d1e5bd7f5e1a4ab76aba2b" "/opt/rustwide/cargo-home/bin/cargo" "+a5689b034ad287112f709693384a37f1377eb9cc" "check" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }`
[INFO] [stderr]    Compiling proc-macro2 v1.0.26
[INFO] [stderr]    Compiling unicode-xid v0.2.1
[INFO] [stderr]    Compiling syn v1.0.70
[INFO] [stderr]    Compiling memchr v2.3.4
[INFO] [stderr]    Compiling doc-comment v0.3.3
[INFO] [stderr]     Checking regex-syntax v0.6.23
[INFO] [stderr]     Checking aho-corasick v0.7.15
[INFO] [stderr]    Compiling quote v1.0.9
[INFO] [stderr]     Checking regex v1.4.6
[INFO] [stderr]    Compiling snafu-derive v0.6.10
[INFO] [stderr]    Compiling derive-getters v0.2.0
[INFO] [stderr]     Checking snafu v0.6.10
[INFO] [stderr]     Checking woah v0.1.0 (/opt/rustwide/workdir)
[INFO] [stdout] warning: unused import: `analyser::StaticAnalysisResults as StaticAnalysisResults`
[INFO] [stdout]  --> src/analysis/mod.rs:5:9
[INFO] [stdout]   |
[INFO] [stdout] 5 | pub use analyser::StaticAnalysisResults as StaticAnalysisResults;
[INFO] [stdout]   |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]   |
[INFO] [stdout]   = note: `#[warn(unused_imports)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `function::Argument as Argument`
[INFO] [stdout]   --> src/parse/mod.rs:11:9
[INFO] [stdout]    |
[INFO] [stdout] 11 | pub use function::Argument as Argument;
[INFO] [stdout]    |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `analyser::StaticAnalysisResults as StaticAnalysisResults`
[INFO] [stdout]  --> src/analysis/mod.rs:5:9
[INFO] [stdout]   |
[INFO] [stdout] 5 | pub use analyser::StaticAnalysisResults as StaticAnalysisResults;
[INFO] [stdout]   |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]   |
[INFO] [stdout]   = note: `#[warn(unused_imports)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `function::Argument as Argument`
[INFO] [stdout]   --> src/parse/mod.rs:11:9
[INFO] [stdout]    |
[INFO] [stdout] 11 | pub use function::Argument as Argument;
[INFO] [stdout]    |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `root_path`
[INFO] [stdout]   --> src/interface.rs:18:9
[INFO] [stdout]    |
[INFO] [stdout] 18 |     let root_path = src_path.parent().unwrap().to_path_buf();
[INFO] [stdout]    |         ^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_root_path`
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `static_analysis_results`
[INFO] [stdout]   --> src/interface.rs:23:9
[INFO] [stdout]    |
[INFO] [stdout] 23 |     let static_analysis_results = analyse_program(&mut modules)?;
[INFO] [stdout]    |         ^^^^^^^^^^^^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_static_analysis_results`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: method `get_attr_res_type` is never used
[INFO] [stdout]    --> src/analysis/type_checker.rs:593:8
[INFO] [stdout]     |
[INFO] [stdout]  21 | impl<'a> TypeChecker<'a> {
[INFO] [stdout]     | ------------------------ method in this implementation
[INFO] [stdout] ...
[INFO] [stdout] 593 |     fn get_attr_res_type(&self, parent: &Box<Expr>, attr_name: &String) -> Result<Rc<TypeKind>, Vec<Error>> {
[INFO] [stdout]     |        ^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: methods `instance_methods_mut`, `module_methods_mut`, `module_path`, and `module_type` are never used
[INFO] [stdout]   --> src/parse/module.rs:49:12
[INFO] [stdout]    |
[INFO] [stdout] 30 | impl Module {
[INFO] [stdout]    | ----------- methods in this implementation
[INFO] [stdout] ...
[INFO] [stdout] 49 |     pub fn instance_methods_mut(&mut self) -> &mut HashMap<String, Function> {
[INFO] [stdout]    |            ^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] ...
[INFO] [stdout] 65 |     pub fn module_methods_mut(&mut self) -> &mut HashMap<String, Function> {
[INFO] [stdout]    |            ^^^^^^^^^^^^^^^^^^
[INFO] [stdout] ...
[INFO] [stdout] 75 |     pub fn module_path(&self) -> &Vec<String> {
[INFO] [stdout]    |            ^^^^^^^^^^^
[INFO] [stdout] ...
[INFO] [stdout] 80 |     pub fn module_type(&self) -> &ModuleType {
[INFO] [stdout]    |            ^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `error::Error`
[INFO] [stdout]    --> src/parse/function.rs:125:17
[INFO] [stdout]     |
[INFO] [stdout] 125 | /                 Error::new(ErrorKind::SyntaxError)
[INFO] [stdout] 126 | |                     .set_position(tokens[index].position())
[INFO] [stdout] 127 | |                     .set_message("expected a (lower-case) function name")
[INFO] [stdout] 128 | |                     .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] 127 -                     .set_message("expected a (lower-case) function name")
[INFO] [stdout] 128 -                     .into()
[INFO] [stdout] 127 +                     .set_message("expected a (lower-case) function name")
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 125 ~                 <error::Error as Into<_>>::into(Error::new(ErrorKind::SyntaxError)
[INFO] [stdout] 126 |                     .set_position(tokens[index].position())
[INFO] [stdout] 127 ~                     .set_message("expected a (lower-case) function name"))
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr] error: could not compile `woah` (bin "woah" test) due to 1 previous error; 6 warnings emitted
[INFO] [stderr] warning: build failed, waiting for other jobs to finish...
[INFO] [stdout] warning: unused variable: `root_path`
[INFO] [stdout]   --> src/interface.rs:18:9
[INFO] [stdout]    |
[INFO] [stdout] 18 |     let root_path = src_path.parent().unwrap().to_path_buf();
[INFO] [stdout]    |         ^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_root_path`
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `static_analysis_results`
[INFO] [stdout]   --> src/interface.rs:23:9
[INFO] [stdout]    |
[INFO] [stdout] 23 |     let static_analysis_results = analyse_program(&mut modules)?;
[INFO] [stdout]    |         ^^^^^^^^^^^^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_static_analysis_results`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: method `get_attr_res_type` is never used
[INFO] [stdout]    --> src/analysis/type_checker.rs:593:8
[INFO] [stdout]     |
[INFO] [stdout]  21 | impl<'a> TypeChecker<'a> {
[INFO] [stdout]     | ------------------------ method in this implementation
[INFO] [stdout] ...
[INFO] [stdout] 593 |     fn get_attr_res_type(&self, parent: &Box<Expr>, attr_name: &String) -> Result<Rc<TypeKind>, Vec<Error>> {
[INFO] [stdout]     |        ^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: methods `instance_methods_mut`, `module_methods_mut`, `module_path`, and `module_type` are never used
[INFO] [stdout]   --> src/parse/module.rs:49:12
[INFO] [stdout]    |
[INFO] [stdout] 30 | impl Module {
[INFO] [stdout]    | ----------- methods in this implementation
[INFO] [stdout] ...
[INFO] [stdout] 49 |     pub fn instance_methods_mut(&mut self) -> &mut HashMap<String, Function> {
[INFO] [stdout]    |            ^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] ...
[INFO] [stdout] 65 |     pub fn module_methods_mut(&mut self) -> &mut HashMap<String, Function> {
[INFO] [stdout]    |            ^^^^^^^^^^^^^^^^^^
[INFO] [stdout] ...
[INFO] [stdout] 75 |     pub fn module_path(&self) -> &Vec<String> {
[INFO] [stdout]    |            ^^^^^^^^^^^
[INFO] [stdout] ...
[INFO] [stdout] 80 |     pub fn module_type(&self) -> &ModuleType {
[INFO] [stdout]    |            ^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `error::Error`
[INFO] [stdout]    --> src/parse/function.rs:125:17
[INFO] [stdout]     |
[INFO] [stdout] 125 | /                 Error::new(ErrorKind::SyntaxError)
[INFO] [stdout] 126 | |                     .set_position(tokens[index].position())
[INFO] [stdout] 127 | |                     .set_message("expected a (lower-case) function name")
[INFO] [stdout] 128 | |                     .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] 127 -                     .set_message("expected a (lower-case) function name")
[INFO] [stdout] 128 -                     .into()
[INFO] [stdout] 127 +                     .set_message("expected a (lower-case) function name")
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 125 ~                 <error::Error as Into<_>>::into(Error::new(ErrorKind::SyntaxError)
[INFO] [stdout] 126 |                     .set_position(tokens[index].position())
[INFO] [stdout] 127 ~                     .set_message("expected a (lower-case) function name"))
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr] error: could not compile `woah` (bin "woah") due to 1 previous error; 6 warnings emitted
[INFO] [stderr] warning: unused dependency `derive-getters`
[INFO] [stderr]  --> Cargo.toml:2:1
[INFO] [stderr]   |
[INFO] [stderr] 2 | derive-getters = "0.2.0"
[INFO] [stderr]   | ^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stderr]   |
[INFO] [stderr]   = note: `cargo::unused_dependencies` is set to `warn` by default
[INFO] [stderr] help: consider removing the dependency on `derive-getters`
[INFO] [stderr] warning: unused dependency `snafu`
[INFO] [stderr]  --> Cargo.toml:5:1
[INFO] [stderr]   |
[INFO] [stderr] 5 | snafu = "0.6.10"
[INFO] [stderr]   | ^^^^^^^^^^^^^^^^
[INFO] [stderr]   |
[INFO] [stderr] help: consider removing the dependency on `snafu`
[INFO] [stderr] warning: `woah` (manifest) generated 2 warnings
[INFO] running `Command { std: "docker" "inspect" "b17ab3bf49cc18f066e36af36a5c12859eec4e0d15d1e5bd7f5e1a4ab76aba2b", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "b17ab3bf49cc18f066e36af36a5c12859eec4e0d15d1e5bd7f5e1a4ab76aba2b", kill_on_drop: false }`
[INFO] [stdout] b17ab3bf49cc18f066e36af36a5c12859eec4e0d15d1e5bd7f5e1a4ab76aba2b
