[INFO] cloning repository https://github.com/mkilgore/rshell [INFO] running `Command { std: "git" "-c" "credential.helper=" "-c" "credential.helper=/workspace/cargo-home/bin/git-credential-null" "clone" "--bare" "https://github.com/mkilgore/rshell" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fmkilgore%2Frshell", kill_on_drop: false }` [INFO] [stderr] Cloning into bare repository '/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fmkilgore%2Frshell'... [INFO] running `Command { std: "git" "rev-parse" "HEAD", kill_on_drop: false }` [INFO] [stdout] 4553cf1c2ecfc193a1fcbdf1044f5e9a6e54d131 [INFO] testing mkilgore/rshell against beta-2022-02-22 for beta-1.60-1 [INFO] running `Command { std: "git" "clone" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Fmkilgore%2Frshell" "/workspace/builds/worker-16/source", kill_on_drop: false }` [INFO] [stderr] Cloning into '/workspace/builds/worker-16/source'... [INFO] [stderr] done. [INFO] validating manifest of git repo https://github.com/mkilgore/rshell on toolchain beta-2022-02-22 [INFO] running `Command { std: "/workspace/cargo-home/bin/cargo" "+beta-2022-02-22" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }` [INFO] started tweaking git repo https://github.com/mkilgore/rshell [INFO] finished tweaking git repo https://github.com/mkilgore/rshell [INFO] tweaked toml for git repo https://github.com/mkilgore/rshell written to /workspace/builds/worker-16/source/Cargo.toml [INFO] crate git repo https://github.com/mkilgore/rshell already has a lockfile, it will not be regenerated [INFO] running `Command { std: "/workspace/cargo-home/bin/cargo" "+beta-2022-02-22" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }` [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-16/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-16/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:0cd99ca24d8e8c98e67c542213511d985b8778b5bdcbb160e038429496686047" "/opt/rustwide/cargo-home/bin/cargo" "+beta-2022-02-22" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }` [INFO] [stdout] 7a4566928670a2da4643523cdc8825e32b0f3f57fd93f58888193274b7c75ef1 [INFO] running `Command { std: "docker" "start" "-a" "7a4566928670a2da4643523cdc8825e32b0f3f57fd93f58888193274b7c75ef1", kill_on_drop: false }` [INFO] running `Command { std: "docker" "inspect" "7a4566928670a2da4643523cdc8825e32b0f3f57fd93f58888193274b7c75ef1", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "7a4566928670a2da4643523cdc8825e32b0f3f57fd93f58888193274b7c75ef1", kill_on_drop: false }` [INFO] [stdout] 7a4566928670a2da4643523cdc8825e32b0f3f57fd93f58888193274b7c75ef1 [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-16/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-16/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=warn" "-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:0cd99ca24d8e8c98e67c542213511d985b8778b5bdcbb160e038429496686047" "/opt/rustwide/cargo-home/bin/cargo" "+beta-2022-02-22" "build" "--frozen" "--message-format=json", kill_on_drop: false }` [INFO] [stdout] 4c0619bfbe79c459e62f97255e8f5becc767ad26bc20748af694a918c8bd44af [INFO] running `Command { std: "docker" "start" "-a" "4c0619bfbe79c459e62f97255e8f5becc767ad26bc20748af694a918c8bd44af", kill_on_drop: false }` [INFO] [stderr] Compiling lazy_static v0.2.8 [INFO] [stderr] Compiling libc v0.2.21 [INFO] [stderr] Compiling rshell v0.1.0 (/opt/rustwide/workdir) [INFO] [stdout] warning: unnecessary trailing semicolon [INFO] [stdout] --> src/job.rs:91:23 [INFO] [stdout] | [INFO] [stdout] 91 | { ; } [INFO] [stdout] | ^ help: remove this semicolon [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(redundant_semicolons)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary trailing semicolon [INFO] [stdout] --> src/builtin.rs:157:17 [INFO] [stdout] | [INFO] [stdout] 157 | ; [INFO] [stdout] | ^ help: remove this semicolon [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary parentheses around function argument [INFO] [stdout] --> src/job_list.rs:38:44 [INFO] [stdout] | [INFO] [stdout] 38 | self.list.push(Arc::new(Mutex::new((job)))); [INFO] [stdout] | ^ ^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_parens)]` on by default [INFO] [stdout] help: remove these parentheses [INFO] [stdout] | [INFO] [stdout] 38 - self.list.push(Arc::new(Mutex::new((job)))); [INFO] [stdout] 38 + self.list.push(Arc::new(Mutex::new(job))); [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: use of deprecated constant `std::sync::ONCE_INIT`: the `new` function is now preferred [INFO] [stdout] --> src/main.rs:22:1 [INFO] [stdout] | [INFO] [stdout] 22 | / lazy_static! { [INFO] [stdout] 23 | | pub static ref JOB_LIST: Mutex = Mutex::new(JobList::new()); [INFO] [stdout] 24 | | pub static ref CURRENT_DIR: Mutex = Mutex::new(String::new()); [INFO] [stdout] 25 | | } [INFO] [stdout] | |_^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(deprecated)]` on by default [INFO] [stdout] = note: this warning originates in the macro `__lazy_static_create` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: use of deprecated constant `std::sync::ONCE_INIT`: the `new` function is now preferred [INFO] [stdout] --> src/main.rs:22:1 [INFO] [stdout] | [INFO] [stdout] 22 | / lazy_static! { [INFO] [stdout] 23 | | pub static ref JOB_LIST: Mutex = Mutex::new(JobList::new()); [INFO] [stdout] 24 | | pub static ref CURRENT_DIR: Mutex = Mutex::new(String::new()); [INFO] [stdout] 25 | | } [INFO] [stdout] | |_^ [INFO] [stdout] | [INFO] [stdout] = note: this warning originates in the macro `__lazy_static_create` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: use of deprecated constant `std::sync::ONCE_INIT`: the `new` function is now preferred [INFO] [stdout] --> src/main.rs:22:1 [INFO] [stdout] | [INFO] [stdout] 22 | / lazy_static! { [INFO] [stdout] 23 | | pub static ref JOB_LIST: Mutex = Mutex::new(JobList::new()); [INFO] [stdout] 24 | | pub static ref CURRENT_DIR: Mutex = Mutex::new(String::new()); [INFO] [stdout] 25 | | } [INFO] [stdout] | |_^ [INFO] [stdout] | [INFO] [stdout] = note: this warning originates in the macro `__lazy_static_create` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: use of deprecated constant `std::sync::ONCE_INIT`: the `new` function is now preferred [INFO] [stdout] --> src/main.rs:22:1 [INFO] [stdout] | [INFO] [stdout] 22 | / lazy_static! { [INFO] [stdout] 23 | | pub static ref JOB_LIST: Mutex = Mutex::new(JobList::new()); [INFO] [stdout] 24 | | pub static ref CURRENT_DIR: Mutex = Mutex::new(String::new()); [INFO] [stdout] 25 | | } [INFO] [stdout] | |_^ [INFO] [stdout] | [INFO] [stdout] = note: this warning originates in the macro `__lazy_static_create` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: use of deprecated constant `std::sync::ONCE_INIT`: the `new` function is now preferred [INFO] [stdout] --> src/builtin.rs:26:1 [INFO] [stdout] | [INFO] [stdout] 26 | / lazy_static! { [INFO] [stdout] 27 | | static ref BUILTIN_MAP: HashMap<&'static str, ShellBuiltin> = { [INFO] [stdout] 28 | | let mut m = HashMap::new(); [INFO] [stdout] 29 | | m.insert("pwd", ShellBuiltin::new(builtin_pwd)); [INFO] [stdout] ... | [INFO] [stdout] 36 | | }; [INFO] [stdout] 37 | | } [INFO] [stdout] | |_^ [INFO] [stdout] | [INFO] [stdout] = note: this warning originates in the macro `__lazy_static_create` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: use of deprecated constant `std::sync::ONCE_INIT`: the `new` function is now preferred [INFO] [stdout] --> src/builtin.rs:26:1 [INFO] [stdout] | [INFO] [stdout] 26 | / lazy_static! { [INFO] [stdout] 27 | | static ref BUILTIN_MAP: HashMap<&'static str, ShellBuiltin> = { [INFO] [stdout] 28 | | let mut m = HashMap::new(); [INFO] [stdout] 29 | | m.insert("pwd", ShellBuiltin::new(builtin_pwd)); [INFO] [stdout] ... | [INFO] [stdout] 36 | | }; [INFO] [stdout] 37 | | } [INFO] [stdout] | |_^ [INFO] [stdout] | [INFO] [stdout] = note: this warning originates in the macro `__lazy_static_create` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/job.rs:80:22 [INFO] [stdout] | [INFO] [stdout] 80 | pub fn parse_job(mut lex: &mut InputLexer) -> Option { [INFO] [stdout] | ----^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_mut)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: getting the inner pointer of a temporary `CString` [INFO] [stdout] --> src/job.rs:110:96 [INFO] [stdout] | [INFO] [stdout] 110 | ... cur_prog.stdout = unsafe { libc::open(CString::new(s).unwrap().as_ptr(), [INFO] [stdout] | ------------------------ ^^^^^^ this pointer will be invalid [INFO] [stdout] | | [INFO] [stdout] | this `CString` is deallocated at the end of the statement, bind it to a variable to extend its lifetime [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(temporary_cstring_as_ptr)]` on by default [INFO] [stdout] = note: pointers do not have a lifetime; when calling `as_ptr` the `CString` will be deallocated at the end of the statement because nothing is referencing it as far as the type system is concerned [INFO] [stdout] = help: for more information, see https://doc.rust-lang.org/reference/destructors.html [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: getting the inner pointer of a temporary `CString` [INFO] [stdout] --> src/job.rs:113:96 [INFO] [stdout] | [INFO] [stdout] 113 | ... cur_prog.stdout = unsafe { libc::open(CString::new(s).unwrap().as_ptr(), [INFO] [stdout] | ------------------------ ^^^^^^ this pointer will be invalid [INFO] [stdout] | | [INFO] [stdout] | this `CString` is deallocated at the end of the statement, bind it to a variable to extend its lifetime [INFO] [stdout] | [INFO] [stdout] = note: pointers do not have a lifetime; when calling `as_ptr` the `CString` will be deallocated at the end of the statement because nothing is referencing it as far as the type system is concerned [INFO] [stdout] = help: for more information, see https://doc.rust-lang.org/reference/destructors.html [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: getting the inner pointer of a temporary `CString` [INFO] [stdout] --> src/job.rs:116:95 [INFO] [stdout] | [INFO] [stdout] 116 | ... cur_prog.stdin = unsafe { libc::open(CString::new(s).unwrap().as_ptr(), libc::O_RDONLY) }; [INFO] [stdout] | ------------------------ ^^^^^^ this pointer will be invalid [INFO] [stdout] | | [INFO] [stdout] | this `CString` is deallocated at the end of the statement, bind it to a variable to extend its lifetime [INFO] [stdout] | [INFO] [stdout] = note: pointers do not have a lifetime; when calling `as_ptr` the `CString` will be deallocated at the end of the statement because nothing is referencing it as far as the type system is concerned [INFO] [stdout] = help: for more information, see https://doc.rust-lang.org/reference/destructors.html [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: getting the inner pointer of a temporary `CString` [INFO] [stdout] --> src/prog.rs:89:63 [INFO] [stdout] | [INFO] [stdout] 89 | libc::execvp(CString::new(self.file.clone()).unwrap().as_ptr() as *const libc::c_char, c_char_vec.as_ptr() as *const *const libc:... [INFO] [stdout] | ---------------------------------------- ^^^^^^ this pointer will be invalid [INFO] [stdout] | | [INFO] [stdout] | this `CString` is deallocated at the end of the statement, bind it to a variable to extend its lifetime [INFO] [stdout] | [INFO] [stdout] = note: pointers do not have a lifetime; when calling `as_ptr` the `CString` will be deallocated at the end of the statement because nothing is referencing it as far as the type system is concerned [INFO] [stdout] = help: for more information, see https://doc.rust-lang.org/reference/destructors.html [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: 14 warnings emitted [INFO] [stdout] [INFO] [stdout] [INFO] [stderr] Finished dev [unoptimized + debuginfo] target(s) in 2.03s [INFO] running `Command { std: "docker" "inspect" "4c0619bfbe79c459e62f97255e8f5becc767ad26bc20748af694a918c8bd44af", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "4c0619bfbe79c459e62f97255e8f5becc767ad26bc20748af694a918c8bd44af", kill_on_drop: false }` [INFO] [stdout] 4c0619bfbe79c459e62f97255e8f5becc767ad26bc20748af694a918c8bd44af [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-16/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-16/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=warn" "-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:0cd99ca24d8e8c98e67c542213511d985b8778b5bdcbb160e038429496686047" "/opt/rustwide/cargo-home/bin/cargo" "+beta-2022-02-22" "test" "--frozen" "--no-run" "--message-format=json", kill_on_drop: false }` [INFO] [stdout] 4ed474bae2e2f69067957fcfac2b4052e71fe12e102ea085b53738c1ec1db702 [INFO] running `Command { std: "docker" "start" "-a" "4ed474bae2e2f69067957fcfac2b4052e71fe12e102ea085b53738c1ec1db702", kill_on_drop: false }` [INFO] [stderr] Compiling rshell v0.1.0 (/opt/rustwide/workdir) [INFO] [stdout] warning: unnecessary trailing semicolon [INFO] [stdout] --> src/job.rs:91:23 [INFO] [stdout] | [INFO] [stdout] 91 | { ; } [INFO] [stdout] | ^ help: remove this semicolon [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(redundant_semicolons)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary trailing semicolon [INFO] [stdout] --> src/builtin.rs:157:17 [INFO] [stdout] | [INFO] [stdout] 157 | ; [INFO] [stdout] | ^ help: remove this semicolon [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unnecessary parentheses around function argument [INFO] [stdout] --> src/job_list.rs:38:44 [INFO] [stdout] | [INFO] [stdout] 38 | self.list.push(Arc::new(Mutex::new((job)))); [INFO] [stdout] | ^ ^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_parens)]` on by default [INFO] [stdout] help: remove these parentheses [INFO] [stdout] | [INFO] [stdout] 38 - self.list.push(Arc::new(Mutex::new((job)))); [INFO] [stdout] 38 + self.list.push(Arc::new(Mutex::new(job))); [INFO] [stdout] | [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: use of deprecated constant `std::sync::ONCE_INIT`: the `new` function is now preferred [INFO] [stdout] --> src/main.rs:22:1 [INFO] [stdout] | [INFO] [stdout] 22 | / lazy_static! { [INFO] [stdout] 23 | | pub static ref JOB_LIST: Mutex = Mutex::new(JobList::new()); [INFO] [stdout] 24 | | pub static ref CURRENT_DIR: Mutex = Mutex::new(String::new()); [INFO] [stdout] 25 | | } [INFO] [stdout] | |_^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(deprecated)]` on by default [INFO] [stdout] = note: this warning originates in the macro `__lazy_static_create` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: use of deprecated constant `std::sync::ONCE_INIT`: the `new` function is now preferred [INFO] [stdout] --> src/main.rs:22:1 [INFO] [stdout] | [INFO] [stdout] 22 | / lazy_static! { [INFO] [stdout] 23 | | pub static ref JOB_LIST: Mutex = Mutex::new(JobList::new()); [INFO] [stdout] 24 | | pub static ref CURRENT_DIR: Mutex = Mutex::new(String::new()); [INFO] [stdout] 25 | | } [INFO] [stdout] | |_^ [INFO] [stdout] | [INFO] [stdout] = note: this warning originates in the macro `__lazy_static_create` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: use of deprecated constant `std::sync::ONCE_INIT`: the `new` function is now preferred [INFO] [stdout] --> src/main.rs:22:1 [INFO] [stdout] | [INFO] [stdout] 22 | / lazy_static! { [INFO] [stdout] 23 | | pub static ref JOB_LIST: Mutex = Mutex::new(JobList::new()); [INFO] [stdout] 24 | | pub static ref CURRENT_DIR: Mutex = Mutex::new(String::new()); [INFO] [stdout] 25 | | } [INFO] [stdout] | |_^ [INFO] [stdout] | [INFO] [stdout] = note: this warning originates in the macro `__lazy_static_create` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: use of deprecated constant `std::sync::ONCE_INIT`: the `new` function is now preferred [INFO] [stdout] --> src/main.rs:22:1 [INFO] [stdout] | [INFO] [stdout] 22 | / lazy_static! { [INFO] [stdout] 23 | | pub static ref JOB_LIST: Mutex = Mutex::new(JobList::new()); [INFO] [stdout] 24 | | pub static ref CURRENT_DIR: Mutex = Mutex::new(String::new()); [INFO] [stdout] 25 | | } [INFO] [stdout] | |_^ [INFO] [stdout] | [INFO] [stdout] = note: this warning originates in the macro `__lazy_static_create` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: use of deprecated constant `std::sync::ONCE_INIT`: the `new` function is now preferred [INFO] [stdout] --> src/builtin.rs:26:1 [INFO] [stdout] | [INFO] [stdout] 26 | / lazy_static! { [INFO] [stdout] 27 | | static ref BUILTIN_MAP: HashMap<&'static str, ShellBuiltin> = { [INFO] [stdout] 28 | | let mut m = HashMap::new(); [INFO] [stdout] 29 | | m.insert("pwd", ShellBuiltin::new(builtin_pwd)); [INFO] [stdout] ... | [INFO] [stdout] 36 | | }; [INFO] [stdout] 37 | | } [INFO] [stdout] | |_^ [INFO] [stdout] | [INFO] [stdout] = note: this warning originates in the macro `__lazy_static_create` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: use of deprecated constant `std::sync::ONCE_INIT`: the `new` function is now preferred [INFO] [stdout] --> src/builtin.rs:26:1 [INFO] [stdout] | [INFO] [stdout] 26 | / lazy_static! { [INFO] [stdout] 27 | | static ref BUILTIN_MAP: HashMap<&'static str, ShellBuiltin> = { [INFO] [stdout] 28 | | let mut m = HashMap::new(); [INFO] [stdout] 29 | | m.insert("pwd", ShellBuiltin::new(builtin_pwd)); [INFO] [stdout] ... | [INFO] [stdout] 36 | | }; [INFO] [stdout] 37 | | } [INFO] [stdout] | |_^ [INFO] [stdout] | [INFO] [stdout] = note: this warning originates in the macro `__lazy_static_create` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/job.rs:80:22 [INFO] [stdout] | [INFO] [stdout] 80 | pub fn parse_job(mut lex: &mut InputLexer) -> Option { [INFO] [stdout] | ----^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_mut)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: getting the inner pointer of a temporary `CString` [INFO] [stdout] --> src/job.rs:110:96 [INFO] [stdout] | [INFO] [stdout] 110 | ... cur_prog.stdout = unsafe { libc::open(CString::new(s).unwrap().as_ptr(), [INFO] [stdout] | ------------------------ ^^^^^^ this pointer will be invalid [INFO] [stdout] | | [INFO] [stdout] | this `CString` is deallocated at the end of the statement, bind it to a variable to extend its lifetime [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(temporary_cstring_as_ptr)]` on by default [INFO] [stdout] = note: pointers do not have a lifetime; when calling `as_ptr` the `CString` will be deallocated at the end of the statement because nothing is referencing it as far as the type system is concerned [INFO] [stdout] = help: for more information, see https://doc.rust-lang.org/reference/destructors.html [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: getting the inner pointer of a temporary `CString` [INFO] [stdout] --> src/job.rs:113:96 [INFO] [stdout] | [INFO] [stdout] 113 | ... cur_prog.stdout = unsafe { libc::open(CString::new(s).unwrap().as_ptr(), [INFO] [stdout] | ------------------------ ^^^^^^ this pointer will be invalid [INFO] [stdout] | | [INFO] [stdout] | this `CString` is deallocated at the end of the statement, bind it to a variable to extend its lifetime [INFO] [stdout] | [INFO] [stdout] = note: pointers do not have a lifetime; when calling `as_ptr` the `CString` will be deallocated at the end of the statement because nothing is referencing it as far as the type system is concerned [INFO] [stdout] = help: for more information, see https://doc.rust-lang.org/reference/destructors.html [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: getting the inner pointer of a temporary `CString` [INFO] [stdout] --> src/job.rs:116:95 [INFO] [stdout] | [INFO] [stdout] 116 | ... cur_prog.stdin = unsafe { libc::open(CString::new(s).unwrap().as_ptr(), libc::O_RDONLY) }; [INFO] [stdout] | ------------------------ ^^^^^^ this pointer will be invalid [INFO] [stdout] | | [INFO] [stdout] | this `CString` is deallocated at the end of the statement, bind it to a variable to extend its lifetime [INFO] [stdout] | [INFO] [stdout] = note: pointers do not have a lifetime; when calling `as_ptr` the `CString` will be deallocated at the end of the statement because nothing is referencing it as far as the type system is concerned [INFO] [stdout] = help: for more information, see https://doc.rust-lang.org/reference/destructors.html [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: getting the inner pointer of a temporary `CString` [INFO] [stdout] --> src/prog.rs:89:63 [INFO] [stdout] | [INFO] [stdout] 89 | libc::execvp(CString::new(self.file.clone()).unwrap().as_ptr() as *const libc::c_char, c_char_vec.as_ptr() as *const *const libc:... [INFO] [stdout] | ---------------------------------------- ^^^^^^ this pointer will be invalid [INFO] [stdout] | | [INFO] [stdout] | this `CString` is deallocated at the end of the statement, bind it to a variable to extend its lifetime [INFO] [stdout] | [INFO] [stdout] = note: pointers do not have a lifetime; when calling `as_ptr` the `CString` will be deallocated at the end of the statement because nothing is referencing it as far as the type system is concerned [INFO] [stdout] = help: for more information, see https://doc.rust-lang.org/reference/destructors.html [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: 14 warnings emitted [INFO] [stdout] [INFO] [stdout] [INFO] [stderr] Finished test [unoptimized + debuginfo] target(s) in 0.38s [INFO] running `Command { std: "docker" "inspect" "4ed474bae2e2f69067957fcfac2b4052e71fe12e102ea085b53738c1ec1db702", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "4ed474bae2e2f69067957fcfac2b4052e71fe12e102ea085b53738c1ec1db702", kill_on_drop: false }` [INFO] [stdout] 4ed474bae2e2f69067957fcfac2b4052e71fe12e102ea085b53738c1ec1db702 [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-16/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-16/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=warn" "-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:0cd99ca24d8e8c98e67c542213511d985b8778b5bdcbb160e038429496686047" "/opt/rustwide/cargo-home/bin/cargo" "+beta-2022-02-22" "test" "--frozen", kill_on_drop: false }` [INFO] [stdout] 7390ee01c72f72854848eb4e69ed95a0b722fa7b835d2b85dd7906cfcf639ebf [INFO] running `Command { std: "docker" "start" "-a" "7390ee01c72f72854848eb4e69ed95a0b722fa7b835d2b85dd7906cfcf639ebf", kill_on_drop: false }` [INFO] [stderr] warning: unnecessary trailing semicolon [INFO] [stderr] --> src/job.rs:91:23 [INFO] [stderr] | [INFO] [stderr] 91 | { ; } [INFO] [stderr] | ^ help: remove this semicolon [INFO] [stderr] | [INFO] [stderr] = note: `#[warn(redundant_semicolons)]` on by default [INFO] [stderr] [INFO] [stderr] warning: unnecessary trailing semicolon [INFO] [stderr] --> src/builtin.rs:157:17 [INFO] [stderr] | [INFO] [stderr] 157 | ; [INFO] [stderr] | ^ help: remove this semicolon [INFO] [stderr] [INFO] [stderr] warning: unnecessary parentheses around function argument [INFO] [stderr] --> src/job_list.rs:38:44 [INFO] [stderr] | [INFO] [stderr] 38 | self.list.push(Arc::new(Mutex::new((job)))); [INFO] [stderr] | ^ ^ [INFO] [stderr] | [INFO] [stderr] = note: `#[warn(unused_parens)]` on by default [INFO] [stderr] help: remove these parentheses [INFO] [stderr] | [INFO] [stderr] 38 - self.list.push(Arc::new(Mutex::new((job)))); [INFO] [stderr] 38 + self.list.push(Arc::new(Mutex::new(job))); [INFO] [stderr] | [INFO] [stderr] [INFO] [stderr] warning: use of deprecated constant `std::sync::ONCE_INIT`: the `new` function is now preferred [INFO] [stderr] --> src/main.rs:22:1 [INFO] [stderr] | [INFO] [stderr] 22 | / lazy_static! { [INFO] [stderr] 23 | | pub static ref JOB_LIST: Mutex = Mutex::new(JobList::new()); [INFO] [stderr] 24 | | pub static ref CURRENT_DIR: Mutex = Mutex::new(String::new()); [INFO] [stderr] 25 | | } [INFO] [stderr] | |_^ [INFO] [stderr] | [INFO] [stderr] = note: `#[warn(deprecated)]` on by default [INFO] [stderr] = note: this warning originates in the macro `__lazy_static_create` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stderr] [INFO] [stderr] warning: use of deprecated constant `std::sync::ONCE_INIT`: the `new` function is now preferred [INFO] [stderr] --> src/main.rs:22:1 [INFO] [stderr] | [INFO] [stderr] 22 | / lazy_static! { [INFO] [stderr] 23 | | pub static ref JOB_LIST: Mutex = Mutex::new(JobList::new()); [INFO] [stderr] 24 | | pub static ref CURRENT_DIR: Mutex = Mutex::new(String::new()); [INFO] [stderr] 25 | | } [INFO] [stderr] | |_^ [INFO] [stderr] | [INFO] [stderr] = note: this warning originates in the macro `__lazy_static_create` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stderr] [INFO] [stderr] warning: use of deprecated constant `std::sync::ONCE_INIT`: the `new` function is now preferred [INFO] [stderr] --> src/builtin.rs:26:1 [INFO] [stderr] | [INFO] [stderr] 26 | / lazy_static! { [INFO] [stderr] 27 | | static ref BUILTIN_MAP: HashMap<&'static str, ShellBuiltin> = { [INFO] [stderr] 28 | | let mut m = HashMap::new(); [INFO] [stderr] 29 | | m.insert("pwd", ShellBuiltin::new(builtin_pwd)); [INFO] [stderr] ... | [INFO] [stderr] 36 | | }; [INFO] [stderr] 37 | | } [INFO] [stderr] | |_^ [INFO] [stderr] | [INFO] [stderr] = note: this warning originates in the macro `__lazy_static_create` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stderr] [INFO] [stderr] warning: variable does not need to be mutable [INFO] [stderr] --> src/job.rs:80:22 [INFO] [stderr] | [INFO] [stderr] 80 | pub fn parse_job(mut lex: &mut InputLexer) -> Option { [INFO] [stderr] | ----^^^ [INFO] [stderr] | | [INFO] [stderr] | help: remove this `mut` [INFO] [stderr] | [INFO] [stderr] = note: `#[warn(unused_mut)]` on by default [INFO] [stderr] [INFO] [stderr] warning: getting the inner pointer of a temporary `CString` [INFO] [stderr] --> src/job.rs:110:96 [INFO] [stderr] | [INFO] [stderr] 110 | ... cur_prog.stdout = unsafe { libc::open(CString::new(s).unwrap().as_ptr(), [INFO] [stderr] | ------------------------ ^^^^^^ this pointer will be invalid [INFO] [stderr] | | [INFO] [stderr] | this `CString` is deallocated at the end of the statement, bind it to a variable to extend its lifetime [INFO] [stderr] | [INFO] [stderr] = note: `#[warn(temporary_cstring_as_ptr)]` on by default [INFO] [stderr] = note: pointers do not have a lifetime; when calling `as_ptr` the `CString` will be deallocated at the end of the statement because nothing is referencing it as far as the type system is concerned [INFO] [stderr] = help: for more information, see https://doc.rust-lang.org/reference/destructors.html [INFO] [stderr] [INFO] [stderr] warning: getting the inner pointer of a temporary `CString` [INFO] [stderr] --> src/job.rs:113:96 [INFO] [stderr] | [INFO] [stderr] 113 | ... cur_prog.stdout = unsafe { libc::open(CString::new(s).unwrap().as_ptr(), [INFO] [stderr] | ------------------------ ^^^^^^ this pointer will be invalid [INFO] [stderr] | | [INFO] [stderr] | this `CString` is deallocated at the end of the statement, bind it to a variable to extend its lifetime [INFO] [stderr] | [INFO] [stderr] = note: pointers do not have a lifetime; when calling `as_ptr` the `CString` will be deallocated at the end of the statement because nothing is referencing it as far as the type system is concerned [INFO] [stderr] = help: for more information, see https://doc.rust-lang.org/reference/destructors.html [INFO] [stderr] [INFO] [stderr] warning: getting the inner pointer of a temporary `CString` [INFO] [stderr] --> src/job.rs:116:95 [INFO] [stderr] | [INFO] [stderr] 116 | ... cur_prog.stdin = unsafe { libc::open(CString::new(s).unwrap().as_ptr(), libc::O_RDONLY) }; [INFO] [stderr] | ------------------------ ^^^^^^ this pointer will be invalid [INFO] [stderr] | | [INFO] [stderr] | this `CString` is deallocated at the end of the statement, bind it to a variable to extend its lifetime [INFO] [stderr] | [INFO] [stderr] = note: pointers do not have a lifetime; when calling `as_ptr` the `CString` will be deallocated at the end of the statement because nothing is referencing it as far as the type system is concerned [INFO] [stderr] = help: for more information, see https://doc.rust-lang.org/reference/destructors.html [INFO] [stderr] [INFO] [stderr] warning: getting the inner pointer of a temporary `CString` [INFO] [stderr] --> src/prog.rs:89:63 [INFO] [stderr] | [INFO] [stderr] 89 | libc::execvp(CString::new(self.file.clone()).unwrap().as_ptr() as *const libc::c_char, c_char_vec.as_ptr() as *const *const libc:... [INFO] [stderr] | ---------------------------------------- ^^^^^^ this pointer will be invalid [INFO] [stderr] | | [INFO] [stderr] | this `CString` is deallocated at the end of the statement, bind it to a variable to extend its lifetime [INFO] [stderr] | [INFO] [stderr] = note: pointers do not have a lifetime; when calling `as_ptr` the `CString` will be deallocated at the end of the statement because nothing is referencing it as far as the type system is concerned [INFO] [stderr] = help: for more information, see https://doc.rust-lang.org/reference/destructors.html [INFO] [stderr] [INFO] [stderr] warning: `rshell` (bin "rshell" test) generated 14 warnings (3 duplicates) [INFO] [stderr] Finished test [unoptimized + debuginfo] target(s) in 0.02s [INFO] [stderr] Running unittests (/opt/rustwide/target/debug/deps/rshell-27d32a502029aab7) [INFO] [stdout] [INFO] [stdout] running 0 tests [INFO] [stdout] [INFO] [stdout] test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.00s [INFO] [stdout] [INFO] running `Command { std: "docker" "inspect" "7390ee01c72f72854848eb4e69ed95a0b722fa7b835d2b85dd7906cfcf639ebf", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "7390ee01c72f72854848eb4e69ed95a0b722fa7b835d2b85dd7906cfcf639ebf", kill_on_drop: false }` [INFO] [stdout] 7390ee01c72f72854848eb4e69ed95a0b722fa7b835d2b85dd7906cfcf639ebf