[INFO] cloning repository https://github.com/loxp/kvd [INFO] running `Command { std: "git" "-c" "credential.helper=" "-c" "credential.helper=/workspace/cargo-home/bin/git-credential-null" "clone" "--bare" "https://github.com/loxp/kvd" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Floxp%2Fkvd", kill_on_drop: false }` [INFO] [stderr] Cloning into bare repository '/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Floxp%2Fkvd'... [INFO] running `Command { std: "git" "rev-parse" "HEAD", kill_on_drop: false }` [INFO] [stdout] 16b1a5205420dc5f283a3050843a028806b6bf51 [INFO] testing loxp/kvd against master#d98a5da813da67eb189387b8ccfb73cf481275d8+rustflags=-Copt-level=3 for pr-138759-retry [INFO] running `Command { std: "git" "clone" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Floxp%2Fkvd" "/workspace/builds/worker-5-tc1/source", kill_on_drop: false }` [INFO] [stderr] Cloning into '/workspace/builds/worker-5-tc1/source'... [INFO] [stderr] done. [INFO] started tweaking git repo https://github.com/loxp/kvd [INFO] finished tweaking git repo https://github.com/loxp/kvd [INFO] tweaked toml for git repo https://github.com/loxp/kvd written to /workspace/builds/worker-5-tc1/source/Cargo.toml [INFO] validating manifest of git repo https://github.com/loxp/kvd on toolchain d98a5da813da67eb189387b8ccfb73cf481275d8 [INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+d98a5da813da67eb189387b8ccfb73cf481275d8" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }` [INFO] crate git repo https://github.com/loxp/kvd 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" "+d98a5da813da67eb189387b8ccfb73cf481275d8" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }` [INFO] [stderr] Updating crates.io index [INFO] [stderr] Downloading crates ... [INFO] [stderr] Downloaded failure v0.1.5 [INFO] [stderr] Downloaded arrayvec v0.4.11 [INFO] [stderr] Downloaded slog-term v2.4.1 [INFO] [stderr] Downloaded redox_users v0.3.1 [INFO] [stderr] Downloaded nodrop v0.1.13 [INFO] [stderr] Downloaded linked-hash-map v0.5.2 [INFO] [stderr] Downloaded crossbeam-utils v0.6.6 [INFO] [stderr] Downloaded assert_cmd v0.11.1 [INFO] [stderr] Downloaded serde_test v0.8.23 [INFO] [stderr] Downloaded serde v0.8.23 [INFO] [stderr] Downloaded linked-hash-map v0.3.0 [INFO] [stderr] Downloaded escargot v0.4.0 [INFO] [stderr] Downloaded crossbeam-epoch v0.7.2 [INFO] [stderr] Downloaded syn v1.0.5 [INFO] [stderr] Downloaded serde v1.0.99 [INFO] [stderr] Downloaded chrono v0.4.9 [INFO] [stderr] Downloaded crossbeam-channel v0.3.9 [INFO] [stderr] Downloaded slog v2.5.2 [INFO] [stderr] Downloaded rust-ini v0.13.0 [INFO] [stderr] Downloaded config v0.9.3 [INFO] [stderr] Downloaded crossbeam-queue v0.1.2 [INFO] [stderr] Downloaded slog-stdlog v4.0.0 [INFO] [stderr] Downloaded autocfg v0.1.6 [INFO] [stderr] Downloaded backtrace v0.3.35 [INFO] [stderr] Downloaded synstructure v0.10.2 [INFO] [stderr] Downloaded yaml-rust v0.4.3 [INFO] [stderr] Downloaded predicates v1.0.1 [INFO] [stderr] Downloaded backtrace-sys v0.1.31 [INFO] [stderr] Downloaded slog-async v2.3.0 [INFO] [stderr] Downloaded memoffset v0.5.1 [INFO] [stderr] Downloaded normalize-line-endings v0.2.2 [INFO] [stderr] Downloaded crossbeam v0.7.2 [INFO] [stderr] Downloaded slog-scope v4.1.2 [INFO] [stderr] Downloaded num-traits v0.2.8 [INFO] [stderr] Downloaded failure_derive v0.1.5 [INFO] [stderr] Downloaded rust-argon2 v0.5.1 [INFO] [stderr] Downloaded serde-hjson v0.8.2 [INFO] [stderr] Downloaded float-cmp v0.4.0 [INFO] [stderr] Downloaded proc-macro2 v1.0.3 [INFO] [stderr] Downloaded crossbeam-deque v0.7.1 [INFO] [stderr] Downloaded arc-swap v0.3.11 [INFO] [stderr] Downloaded blake2b_simd v0.5.8 [INFO] [stderr] Downloaded constant_time_eq v0.1.4 [INFO] [stderr] Downloaded serde_derive v1.0.100 [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-5-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-5-tc1/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:e90291280db7d1fac5b66fc6dad9f9662629e7365a55743daf9bdf73ebc4ea79" "/opt/rustwide/cargo-home/bin/cargo" "+d98a5da813da67eb189387b8ccfb73cf481275d8" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }` [INFO] [stdout] a213f2dac02e44b76795ef37b27fbbfcaaa0c696e3ae2730bd6f04960f30ee1d [INFO] running `Command { std: "docker" "start" "-a" "a213f2dac02e44b76795ef37b27fbbfcaaa0c696e3ae2730bd6f04960f30ee1d", kill_on_drop: false }` [INFO] running `Command { std: "docker" "inspect" "a213f2dac02e44b76795ef37b27fbbfcaaa0c696e3ae2730bd6f04960f30ee1d", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "a213f2dac02e44b76795ef37b27fbbfcaaa0c696e3ae2730bd6f04960f30ee1d", kill_on_drop: false }` [INFO] [stdout] a213f2dac02e44b76795ef37b27fbbfcaaa0c696e3ae2730bd6f04960f30ee1d [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-5-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-5-tc1/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 -Copt-level=3" "-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:e90291280db7d1fac5b66fc6dad9f9662629e7365a55743daf9bdf73ebc4ea79" "/opt/rustwide/cargo-home/bin/cargo" "+d98a5da813da67eb189387b8ccfb73cf481275d8" "build" "--frozen" "--message-format=json", kill_on_drop: false }` [INFO] [stdout] b7656df9354d358b65856c1b730b01cad206cf816ff72a9bba3c58c6ed5da996 [INFO] running `Command { std: "docker" "start" "-a" "b7656df9354d358b65856c1b730b01cad206cf816ff72a9bba3c58c6ed5da996", kill_on_drop: false }` [INFO] [stderr] Compiling libc v0.2.62 [INFO] [stderr] Compiling autocfg v0.1.6 [INFO] [stderr] Compiling cfg-if v0.1.9 [INFO] [stderr] Compiling proc-macro2 v1.0.3 [INFO] [stderr] Compiling syn v1.0.5 [INFO] [stderr] Compiling proc-macro2 v0.4.30 [INFO] [stderr] Compiling memchr v2.2.1 [INFO] [stderr] Compiling arrayvec v0.4.11 [INFO] [stderr] Compiling slog v2.5.2 [INFO] [stderr] Compiling unicode-xid v0.1.0 [INFO] [stderr] Compiling thread_local v0.3.6 [INFO] [stderr] Compiling nodrop v0.1.13 [INFO] [stderr] Compiling cc v1.0.41 [INFO] [stderr] Compiling syn v0.15.44 [INFO] [stderr] Compiling serde v1.0.99 [INFO] [stderr] Compiling memoffset v0.5.1 [INFO] [stderr] Compiling crossbeam-utils v0.6.6 [INFO] [stderr] Compiling ryu v1.0.0 [INFO] [stderr] Compiling version_check v0.1.5 [INFO] [stderr] Compiling serde v0.8.23 [INFO] [stderr] Compiling log v0.4.8 [INFO] [stderr] Compiling failure_derive v0.1.5 [INFO] [stderr] Compiling bitflags v1.1.0 [INFO] [stderr] Compiling regex-syntax v0.6.12 [INFO] [stderr] Compiling crossbeam-queue v0.1.2 [INFO] [stderr] Compiling num-traits v0.2.8 [INFO] [stderr] Compiling num-integer v0.1.41 [INFO] [stderr] Compiling crossbeam-epoch v0.7.2 [INFO] [stderr] Compiling quote v1.0.2 [INFO] [stderr] Compiling crossbeam-channel v0.3.9 [INFO] [stderr] Compiling time v0.1.42 [INFO] [stderr] Compiling dirs v1.0.5 [INFO] [stderr] Compiling quote v0.6.13 [INFO] [stderr] Compiling aho-corasick v0.7.6 [INFO] [stderr] Compiling nom v4.2.3 [INFO] [stderr] Compiling crossbeam-deque v0.7.1 [INFO] [stderr] Compiling atty v0.2.13 [INFO] [stderr] Compiling linked-hash-map v0.5.2 [INFO] [stderr] Compiling lazy_static v0.2.11 [INFO] [stderr] Compiling itoa v0.4.4 [INFO] [stderr] Compiling arc-swap v0.3.11 [INFO] [stderr] Compiling textwrap v0.11.0 [INFO] [stderr] Compiling yaml-rust v0.4.3 [INFO] [stderr] Compiling crossbeam v0.7.2 [INFO] [stderr] Compiling term v0.5.2 [INFO] [stderr] Compiling num-traits v0.1.43 [INFO] [stderr] Compiling slog-scope v4.1.2 [INFO] [stderr] Compiling strsim v0.8.0 [INFO] [stderr] Compiling rust-ini v0.13.0 [INFO] [stderr] Compiling chrono v0.4.9 [INFO] [stderr] Compiling take_mut v0.2.2 [INFO] [stderr] Compiling vec_map v0.8.1 [INFO] [stderr] Compiling ansi_term v0.11.0 [INFO] [stderr] Compiling regex v1.3.1 [INFO] [stderr] Compiling serde_test v0.8.23 [INFO] [stderr] Compiling clap v2.33.0 [INFO] [stderr] Compiling slog-async v2.3.0 [INFO] [stderr] Compiling linked-hash-map v0.3.0 [INFO] [stderr] Compiling slog-stdlog v4.0.0 [INFO] [stderr] Compiling slog-term v2.4.1 [INFO] [stderr] Compiling serde-hjson v0.8.2 [INFO] [stderr] Compiling backtrace-sys v0.1.31 [INFO] [stderr] Compiling synstructure v0.10.2 [INFO] [stderr] Compiling backtrace v0.3.35 [INFO] [stderr] Compiling serde_derive v1.0.100 [INFO] [stderr] Compiling failure v0.1.5 [INFO] [stderr] Compiling serde_json v1.0.40 [INFO] [stderr] Compiling toml v0.4.10 [INFO] [stderr] Compiling config v0.9.3 [INFO] [stderr] Compiling kvd v0.1.0 (/opt/rustwide/workdir) [INFO] [stdout] warning: unused import: `crate::model::KvdErrorKind::KeyNotFound` [INFO] [stdout] --> src/engine/bitcask.rs:2:5 [INFO] [stdout] | [INFO] [stdout] 2 | use crate::model::KvdErrorKind::KeyNotFound; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_imports)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unexpected `cfg` condition value: `cargo-clippy` [INFO] [stdout] --> src/engine/bitcask.rs:20:10 [INFO] [stdout] | [INFO] [stdout] 20 | #[derive(Serialize, Deserialize, Debug)] [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: no expected values for `feature` [INFO] [stdout] = note: using a cfg inside a derive macro will use the cfgs from the destination crate and not the ones from the defining crate [INFO] [stdout] = help: try referring to `Serialize` crate for guidance on how handle this unexpected cfg [INFO] [stdout] = help: the derive macro `Serialize` may come from an old version of the `serde_derive` crate, try updating your dependency with `cargo update -p serde_derive` [INFO] [stdout] = note: see for more information about checking conditional configuration [INFO] [stdout] = note: `#[warn(unexpected_cfgs)]` on by default [INFO] [stdout] = note: this warning originates in the derive macro `Serialize` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unexpected `cfg` condition value: `cargo-clippy` [INFO] [stdout] --> src/engine/bitcask.rs:20:21 [INFO] [stdout] | [INFO] [stdout] 20 | #[derive(Serialize, Deserialize, Debug)] [INFO] [stdout] | ^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: no expected values for `feature` [INFO] [stdout] = note: using a cfg inside a derive macro will use the cfgs from the destination crate and not the ones from the defining crate [INFO] [stdout] = help: try referring to `Deserialize` crate for guidance on how handle this unexpected cfg [INFO] [stdout] = help: the derive macro `Deserialize` may come from an old version of the `serde_derive` crate, try updating your dependency with `cargo update -p serde_derive` [INFO] [stdout] = note: see for more information about checking conditional configuration [INFO] [stdout] = note: this warning originates in the derive macro `Deserialize` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::string::FromUtf8Error` [INFO] [stdout] --> src/model.rs:7:5 [INFO] [stdout] | [INFO] [stdout] 7 | use std::string::FromUtf8Error; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `crate::engine::bitcask::BitcaskEngine` [INFO] [stdout] --> src/server.rs:1:5 [INFO] [stdout] | [INFO] [stdout] 1 | use crate::engine::bitcask::BitcaskEngine; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `crate::model::KvdErrorKind::KeyNotFound` [INFO] [stdout] --> src/server.rs:4:5 [INFO] [stdout] | [INFO] [stdout] 4 | use crate::model::KvdErrorKind::KeyNotFound; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `config::Config` [INFO] [stdout] --> src/server.rs:6:5 [INFO] [stdout] | [INFO] [stdout] 6 | use config::Config; [INFO] [stdout] | ^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::fs::File` [INFO] [stdout] --> src/server.rs:7:5 [INFO] [stdout] | [INFO] [stdout] 7 | use std::fs::File; [INFO] [stdout] | ^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `stdin` [INFO] [stdout] --> src/server.rs:8:15 [INFO] [stdout] | [INFO] [stdout] 8 | use std::io::{stdin, BufRead, BufReader, BufWriter, Write}; [INFO] [stdout] | ^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `Ipv4Addr` [INFO] [stdout] --> src/server.rs:9:16 [INFO] [stdout] | [INFO] [stdout] 9 | use std::net::{Ipv4Addr, TcpListener, TcpStream}; [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::str` [INFO] [stdout] --> src/server.rs:10:5 [INFO] [stdout] | [INFO] [stdout] 10 | use std::str; [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `thread` [INFO] [stdout] --> src/server.rs:11:15 [INFO] [stdout] | [INFO] [stdout] 11 | use std::{io, thread}; [INFO] [stdout] | ^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `#[macro_use]` import [INFO] [stdout] --> src/lib.rs:5:1 [INFO] [stdout] | [INFO] [stdout] 5 | #[macro_use] [INFO] [stdout] | ^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `#[macro_use]` import [INFO] [stdout] --> src/lib.rs:8:1 [INFO] [stdout] | [INFO] [stdout] 8 | #[macro_use] [INFO] [stdout] | ^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/engine/bitcask.rs:86:17 [INFO] [stdout] | [INFO] [stdout] 86 | let mut pos = reader.seek(SeekFrom::Start(0))?; [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: unused variable: `cmd_pos` [INFO] [stdout] --> src/engine/bitcask.rs:136:13 [INFO] [stdout] | [INFO] [stdout] 136 | let cmd_pos = self.file_store.write_command(cmd)?; [INFO] [stdout] | ^^^^^^^ help: if this is intentional, prefix it with an underscore: `_cmd_pos` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `read_size` [INFO] [stdout] --> src/engine/bitcask.rs:213:13 [INFO] [stdout] | [INFO] [stdout] 213 | let read_size = wal_reader.read(data.as_mut_slice())?; [INFO] [stdout] | ^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_read_size` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/engine/bitcask.rs:204:13 [INFO] [stdout] | [INFO] [stdout] 204 | let mut wal_reader = self [INFO] [stdout] | ----^^^^^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `e` [INFO] [stdout] --> src/model.rs:82:13 [INFO] [stdout] | [INFO] [stdout] 82 | fn from(e: io::Error) -> Self { [INFO] [stdout] | ^ help: if this is intentional, prefix it with an underscore: `_e` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `e` [INFO] [stdout] --> src/model.rs:90:13 [INFO] [stdout] | [INFO] [stdout] 90 | fn from(e: serde_json::error::Error) -> Self { [INFO] [stdout] | ^ help: if this is intentional, prefix it with an underscore: `_e` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `e` [INFO] [stdout] --> src/model.rs:98:13 [INFO] [stdout] | [INFO] [stdout] 98 | fn from(e: ConfigError) -> Self { [INFO] [stdout] | ^ help: if this is intentional, prefix it with an underscore: `_e` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `e` [INFO] [stdout] --> src/model.rs:106:13 [INFO] [stdout] | [INFO] [stdout] 106 | fn from(e: Utf8Error) -> Self { [INFO] [stdout] | ^ help: if this is intentional, prefix it with an underscore: `_e` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/server.rs:48:13 [INFO] [stdout] | [INFO] [stdout] 48 | let mut reader = BufReader::new(&conn); [INFO] [stdout] | ----^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: struct `StoreIndex` is never constructed [INFO] [stdout] --> src/engine/bitcask.rs:59:8 [INFO] [stdout] | [INFO] [stdout] 59 | struct StoreIndex { [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(dead_code)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: associated function `new` is never used [INFO] [stdout] --> src/engine/bitcask.rs:276:12 [INFO] [stdout] | [INFO] [stdout] 275 | impl StoreIndex { [INFO] [stdout] | --------------- associated function in this implementation [INFO] [stdout] 276 | pub fn new() -> StoreIndex { [INFO] [stdout] | ^^^ [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/engine/bitcask.rs:20:10 [INFO] [stdout] | [INFO] [stdout] 20 | #[derive(Serialize, Deserialize, Debug)] [INFO] [stdout] | ^-------- [INFO] [stdout] | | [INFO] [stdout] | `Serialize` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_IMPL_SERIALIZE_FOR_Command` [INFO] [stdout] 21 | pub enum Command { [INFO] [stdout] | ------- `Command` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `Serialize` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `Serialize` may come from an old version of the `serde_derive` crate, try updating your dependency with `cargo update -p serde_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 `Serialize` (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/engine/bitcask.rs:20:21 [INFO] [stdout] | [INFO] [stdout] 20 | #[derive(Serialize, Deserialize, Debug)] [INFO] [stdout] | ^---------- [INFO] [stdout] | | [INFO] [stdout] | `Deserialize` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_IMPL_DESERIALIZE_FOR_Command` [INFO] [stdout] 21 | pub enum Command { [INFO] [stdout] | ------- `Command` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `Deserialize` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `Deserialize` may come from an old version of the `serde_derive` crate, try updating your dependency with `cargo update -p serde_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 `Deserialize` (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/model.rs:11:45 [INFO] [stdout] | [INFO] [stdout] 11 | #[derive(Copy, Clone, Eq, PartialEq, Debug, Fail)] [INFO] [stdout] | ^--- [INFO] [stdout] | | [INFO] [stdout] | `Fail` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_DERIVE_failure_Fail_FOR_KvdErrorKind` [INFO] [stdout] 12 | pub enum KvdErrorKind { [INFO] [stdout] | ------------ `KvdErrorKind` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `Fail` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `Fail` may come from an old version of the `failure_derive` crate, try updating your dependency with `cargo update -p failure_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 `Fail` (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/model.rs:11:45 [INFO] [stdout] | [INFO] [stdout] 11 | #[derive(Copy, Clone, Eq, PartialEq, Debug, Fail)] [INFO] [stdout] | ^--- [INFO] [stdout] | | [INFO] [stdout] | `Display` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_DERIVE_failure_core_fmt_Display_FOR_KvdErrorKind` [INFO] [stdout] 12 | pub enum KvdErrorKind { [INFO] [stdout] | ------------ `KvdErrorKind` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `Fail` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `Fail` may come from an old version of the `failure_derive` crate, try updating your dependency with `cargo update -p failure_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 `Fail` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/server.rs:38:21 [INFO] [stdout] | [INFO] [stdout] 38 | self.handle_conn(stream); [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)]` on by default [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 38 | let _ = self.handle_conn(stream); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/server.rs:54:13 [INFO] [stdout] | [INFO] [stdout] 54 | writer.flush(); [INFO] [stdout] | ^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 54 | let _ = writer.flush(); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/bin/kvd.rs:73:9 [INFO] [stdout] | [INFO] [stdout] 73 | let mut server = Server::new(engine, server_port)?; [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: unused variable: `log_level` [INFO] [stdout] --> src/bin/kvd.rs:47:9 [INFO] [stdout] | [INFO] [stdout] 47 | let log_level = settings.get_str("log_level")?; // TODO: unused [INFO] [stdout] | ^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_log_level` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stderr] Finished `dev` profile [unoptimized + debuginfo] target(s) in 1m 48s [INFO] [stderr] warning: the following packages contain code that will be rejected by a future version of Rust: nom v4.2.3 [INFO] [stderr] note: to see what the problems were, use the option `--future-incompat-report`, or run `cargo report future-incompatibilities --id 1` [INFO] running `Command { std: "docker" "inspect" "b7656df9354d358b65856c1b730b01cad206cf816ff72a9bba3c58c6ed5da996", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "b7656df9354d358b65856c1b730b01cad206cf816ff72a9bba3c58c6ed5da996", kill_on_drop: false }` [INFO] [stdout] b7656df9354d358b65856c1b730b01cad206cf816ff72a9bba3c58c6ed5da996 [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-5-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-5-tc1/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 -Copt-level=3" "-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:e90291280db7d1fac5b66fc6dad9f9662629e7365a55743daf9bdf73ebc4ea79" "/opt/rustwide/cargo-home/bin/cargo" "+d98a5da813da67eb189387b8ccfb73cf481275d8" "test" "--frozen" "--no-run" "--message-format=json", kill_on_drop: false }` [INFO] [stdout] 351d341f48a3d1f2c552f53f112f3ab290a2cf40d8ccb55ceb657d71c0dc4a45 [INFO] running `Command { std: "docker" "start" "-a" "351d341f48a3d1f2c552f53f112f3ab290a2cf40d8ccb55ceb657d71c0dc4a45", kill_on_drop: false }` [INFO] [stderr] Compiling predicates-core v1.0.0 [INFO] [stderr] Compiling escargot v0.4.0 [INFO] [stderr] Compiling float-cmp v0.4.0 [INFO] [stderr] Compiling normalize-line-endings v0.2.2 [INFO] [stderr] Compiling difference v2.0.0 [INFO] [stderr] Compiling treeline v0.1.0 [INFO] [stdout] warning: unused import: `crate::model::KvdErrorKind::KeyNotFound` [INFO] [stdout] --> src/engine/bitcask.rs:2:5 [INFO] [stdout] | [INFO] [stdout] 2 | use crate::model::KvdErrorKind::KeyNotFound; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_imports)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unexpected `cfg` condition value: `cargo-clippy` [INFO] [stdout] --> src/engine/bitcask.rs:20:10 [INFO] [stdout] | [INFO] [stdout] 20 | #[derive(Serialize, Deserialize, Debug)] [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: no expected values for `feature` [INFO] [stdout] = note: using a cfg inside a derive macro will use the cfgs from the destination crate and not the ones from the defining crate [INFO] [stdout] = help: try referring to `Serialize` crate for guidance on how handle this unexpected cfg [INFO] [stdout] = help: the derive macro `Serialize` may come from an old version of the `serde_derive` crate, try updating your dependency with `cargo update -p serde_derive` [INFO] [stdout] = note: see for more information about checking conditional configuration [INFO] [stdout] = note: `#[warn(unexpected_cfgs)]` on by default [INFO] [stdout] = note: this warning originates in the derive macro `Serialize` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unexpected `cfg` condition value: `cargo-clippy` [INFO] [stdout] --> src/engine/bitcask.rs:20:21 [INFO] [stdout] | [INFO] [stdout] 20 | #[derive(Serialize, Deserialize, Debug)] [INFO] [stdout] | ^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: no expected values for `feature` [INFO] [stdout] = note: using a cfg inside a derive macro will use the cfgs from the destination crate and not the ones from the defining crate [INFO] [stdout] = help: try referring to `Deserialize` crate for guidance on how handle this unexpected cfg [INFO] [stdout] = help: the derive macro `Deserialize` may come from an old version of the `serde_derive` crate, try updating your dependency with `cargo update -p serde_derive` [INFO] [stdout] = note: see for more information about checking conditional configuration [INFO] [stdout] = note: this warning originates in the derive macro `Deserialize` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::string::FromUtf8Error` [INFO] [stdout] --> src/model.rs:7:5 [INFO] [stdout] | [INFO] [stdout] 7 | use std::string::FromUtf8Error; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `crate::engine::bitcask::BitcaskEngine` [INFO] [stdout] --> src/server.rs:1:5 [INFO] [stdout] | [INFO] [stdout] 1 | use crate::engine::bitcask::BitcaskEngine; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `crate::model::KvdErrorKind::KeyNotFound` [INFO] [stdout] --> src/server.rs:4:5 [INFO] [stdout] | [INFO] [stdout] 4 | use crate::model::KvdErrorKind::KeyNotFound; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `config::Config` [INFO] [stdout] --> src/server.rs:6:5 [INFO] [stdout] | [INFO] [stdout] 6 | use config::Config; [INFO] [stdout] | ^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::fs::File` [INFO] [stdout] --> src/server.rs:7:5 [INFO] [stdout] | [INFO] [stdout] 7 | use std::fs::File; [INFO] [stdout] | ^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `stdin` [INFO] [stdout] --> src/server.rs:8:15 [INFO] [stdout] | [INFO] [stdout] 8 | use std::io::{stdin, BufRead, BufReader, BufWriter, Write}; [INFO] [stdout] | ^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `Ipv4Addr` [INFO] [stdout] --> src/server.rs:9:16 [INFO] [stdout] | [INFO] [stdout] 9 | use std::net::{Ipv4Addr, TcpListener, TcpStream}; [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::str` [INFO] [stdout] --> src/server.rs:10:5 [INFO] [stdout] | [INFO] [stdout] 10 | use std::str; [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `thread` [INFO] [stdout] --> src/server.rs:11:15 [INFO] [stdout] | [INFO] [stdout] 11 | use std::{io, thread}; [INFO] [stdout] | ^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `#[macro_use]` import [INFO] [stdout] --> src/lib.rs:5:1 [INFO] [stdout] | [INFO] [stdout] 5 | #[macro_use] [INFO] [stdout] | ^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `#[macro_use]` import [INFO] [stdout] --> src/lib.rs:8:1 [INFO] [stdout] | [INFO] [stdout] 8 | #[macro_use] [INFO] [stdout] | ^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/engine/bitcask.rs:86:17 [INFO] [stdout] | [INFO] [stdout] 86 | let mut pos = reader.seek(SeekFrom::Start(0))?; [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: unused variable: `cmd_pos` [INFO] [stdout] --> src/engine/bitcask.rs:136:13 [INFO] [stdout] | [INFO] [stdout] 136 | let cmd_pos = self.file_store.write_command(cmd)?; [INFO] [stdout] | ^^^^^^^ help: if this is intentional, prefix it with an underscore: `_cmd_pos` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `read_size` [INFO] [stdout] --> src/engine/bitcask.rs:213:13 [INFO] [stdout] | [INFO] [stdout] 213 | let read_size = wal_reader.read(data.as_mut_slice())?; [INFO] [stdout] | ^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_read_size` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/engine/bitcask.rs:204:13 [INFO] [stdout] | [INFO] [stdout] 204 | let mut wal_reader = self [INFO] [stdout] | ----^^^^^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `e` [INFO] [stdout] --> src/model.rs:82:13 [INFO] [stdout] | [INFO] [stdout] 82 | fn from(e: io::Error) -> Self { [INFO] [stdout] | ^ help: if this is intentional, prefix it with an underscore: `_e` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `e` [INFO] [stdout] --> src/model.rs:90:13 [INFO] [stdout] | [INFO] [stdout] 90 | fn from(e: serde_json::error::Error) -> Self { [INFO] [stdout] | ^ help: if this is intentional, prefix it with an underscore: `_e` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `e` [INFO] [stdout] --> src/model.rs:98:13 [INFO] [stdout] | [INFO] [stdout] 98 | fn from(e: ConfigError) -> Self { [INFO] [stdout] | ^ help: if this is intentional, prefix it with an underscore: `_e` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `e` [INFO] [stdout] --> src/model.rs:106:13 [INFO] [stdout] | [INFO] [stdout] 106 | fn from(e: Utf8Error) -> Self { [INFO] [stdout] | ^ help: if this is intentional, prefix it with an underscore: `_e` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/server.rs:48:13 [INFO] [stdout] | [INFO] [stdout] 48 | let mut reader = BufReader::new(&conn); [INFO] [stdout] | ----^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: struct `StoreIndex` is never constructed [INFO] [stdout] --> src/engine/bitcask.rs:59:8 [INFO] [stdout] | [INFO] [stdout] 59 | struct StoreIndex { [INFO] [stdout] | ^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(dead_code)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: associated function `new` is never used [INFO] [stdout] --> src/engine/bitcask.rs:276:12 [INFO] [stdout] | [INFO] [stdout] 275 | impl StoreIndex { [INFO] [stdout] | --------------- associated function in this implementation [INFO] [stdout] 276 | pub fn new() -> StoreIndex { [INFO] [stdout] | ^^^ [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/engine/bitcask.rs:20:10 [INFO] [stdout] | [INFO] [stdout] 20 | #[derive(Serialize, Deserialize, Debug)] [INFO] [stdout] | ^-------- [INFO] [stdout] | | [INFO] [stdout] | `Serialize` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_IMPL_SERIALIZE_FOR_Command` [INFO] [stdout] 21 | pub enum Command { [INFO] [stdout] | ------- `Command` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `Serialize` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `Serialize` may come from an old version of the `serde_derive` crate, try updating your dependency with `cargo update -p serde_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 `Serialize` (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/engine/bitcask.rs:20:21 [INFO] [stdout] | [INFO] [stdout] 20 | #[derive(Serialize, Deserialize, Debug)] [INFO] [stdout] | ^---------- [INFO] [stdout] | | [INFO] [stdout] | `Deserialize` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_IMPL_DESERIALIZE_FOR_Command` [INFO] [stdout] 21 | pub enum Command { [INFO] [stdout] | ------- `Command` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `Deserialize` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `Deserialize` may come from an old version of the `serde_derive` crate, try updating your dependency with `cargo update -p serde_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 `Deserialize` (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/model.rs:11:45 [INFO] [stdout] | [INFO] [stdout] 11 | #[derive(Copy, Clone, Eq, PartialEq, Debug, Fail)] [INFO] [stdout] | ^--- [INFO] [stdout] | | [INFO] [stdout] | `Fail` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_DERIVE_failure_Fail_FOR_KvdErrorKind` [INFO] [stdout] 12 | pub enum KvdErrorKind { [INFO] [stdout] | ------------ `KvdErrorKind` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `Fail` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `Fail` may come from an old version of the `failure_derive` crate, try updating your dependency with `cargo update -p failure_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 `Fail` (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/model.rs:11:45 [INFO] [stdout] | [INFO] [stdout] 11 | #[derive(Copy, Clone, Eq, PartialEq, Debug, Fail)] [INFO] [stdout] | ^--- [INFO] [stdout] | | [INFO] [stdout] | `Display` is not local [INFO] [stdout] | move the `impl` block outside of this constant `_DERIVE_failure_core_fmt_Display_FOR_KvdErrorKind` [INFO] [stdout] 12 | pub enum KvdErrorKind { [INFO] [stdout] | ------------ `KvdErrorKind` is not local [INFO] [stdout] | [INFO] [stdout] = note: the derive macro `Fail` defines the non-local `impl`, and may need to be changed [INFO] [stdout] = note: the derive macro `Fail` may come from an old version of the `failure_derive` crate, try updating your dependency with `cargo update -p failure_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 `Fail` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/server.rs:38:21 [INFO] [stdout] | [INFO] [stdout] 38 | self.handle_conn(stream); [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)]` on by default [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 38 | let _ = self.handle_conn(stream); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `Result` that must be used [INFO] [stdout] --> src/server.rs:54:13 [INFO] [stdout] | [INFO] [stdout] 54 | writer.flush(); [INFO] [stdout] | ^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: this `Result` may be an `Err` variant, which should be handled [INFO] [stdout] help: use `let _ = ...` to ignore the resulting value [INFO] [stdout] | [INFO] [stdout] 54 | let _ = writer.flush(); [INFO] [stdout] | +++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/bin/kvd.rs:73:9 [INFO] [stdout] | [INFO] [stdout] 73 | let mut server = Server::new(engine, server_port)?; [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: unused variable: `log_level` [INFO] [stdout] --> src/bin/kvd.rs:47:9 [INFO] [stdout] | [INFO] [stdout] 47 | let log_level = settings.get_str("log_level")?; // TODO: unused [INFO] [stdout] | ^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_log_level` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stderr] Compiling predicates v1.0.1 [INFO] [stderr] Compiling predicates-tree v1.0.0 [INFO] [stderr] Compiling assert_cmd v0.11.1 [INFO] [stderr] Compiling kvd v0.1.0 (/opt/rustwide/workdir) [INFO] [stdout] warning: unused import: `crate::model::KvdErrorKind::KeyNotFound` [INFO] [stdout] --> src/engine/bitcask.rs:2:5 [INFO] [stdout] | [INFO] [stdout] 2 | use crate::model::KvdErrorKind::KeyNotFound; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_imports)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unexpected `cfg` condition value: `cargo-clippy` [INFO] [stdout] --> src/engine/bitcask.rs:20:10 [INFO] [stdout] | [INFO] [stdout] 20 | #[derive(Serialize, Deserialize, Debug)] [INFO] [stdout] | ^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: no expected values for `feature` [INFO] [stdout] = note: using a cfg inside a derive macro will use the cfgs from the destination crate and not the ones from the defining crate [INFO] [stdout] = help: try referring to `Serialize` crate for guidance on how handle this unexpected cfg [INFO] [stdout] = help: the derive macro `Serialize` may come from an old version of the `serde_derive` crate, try updating your dependency with `cargo update -p serde_derive` [INFO] [stdout] = note: see for more information about checking conditional configuration [INFO] [stdout] = note: `#[warn(unexpected_cfgs)]` on by default [INFO] [stdout] = note: this warning originates in the derive macro `Serialize` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unexpected `cfg` condition value: `cargo-clippy` [INFO] [stdout] --> src/engine/bitcask.rs:20:21 [INFO] [stdout] | [INFO] [stdout] 20 | #[derive(Serialize, Deserialize, Debug)] [INFO] [stdout] | ^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: no expected values for `feature` [INFO] [stdout] = note: using a cfg inside a derive macro will use the cfgs from the destination crate and not the ones from the defining crate [INFO] [stdout] = help: try referring to `Deserialize` crate for guidance on how handle this unexpected cfg [INFO] [stdout] = help: the derive macro `Deserialize` may come from an old version of the `serde_derive` crate, try updating your dependency with `cargo update -p serde_derive` [INFO] [stdout] = note: see for more information about checking conditional configuration [INFO] [stdout] = note: this warning originates in the derive macro `Deserialize` (in Nightly builds, run with -Z macro-backtrace for more info) [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::string::FromUtf8Error` [INFO] [stdout] --> src/model.rs:7:5 [INFO] [stdout] | [INFO] [stdout] 7 | use std::string::FromUtf8Error; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `crate::engine::bitcask::BitcaskEngine` [INFO] [stdout] --> src/server.rs:1:5 [INFO] [stdout] | [INFO] [stdout] 1 | use crate::engine::bitcask::BitcaskEngine; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `crate::model::KvdErrorKind::KeyNotFound` [INFO] [stdout] --> src/server.rs:4:5 [INFO] [stdout] | [INFO] [stdout] 4 | use crate::model::KvdErrorKind::KeyNotFound; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `config::Config` [INFO] [stdout] --> src/server.rs:6:5 [INFO] [stdout] | [INFO] [stdout] 6 | use config::Config; [INFO] [stdout] | ^^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::fs::File` [INFO] [stdout] --> src/server.rs:7:5 [INFO] [stdout] | [INFO] [stdout] 7 | use std::fs::File; [INFO] [stdout] | ^^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `stdin` [INFO] [stdout] --> src/server.rs:8:15 [INFO] [stdout] | [INFO] [stdout] 8 | use std::io::{stdin, BufRead, BufReader, BufWriter, Write}; [INFO] [stdout] | ^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `Ipv4Addr` [INFO] [stdout] --> src/server.rs:9:16 [INFO] [stdout] | [INFO] [stdout] 9 | use std::net::{Ipv4Addr, TcpListener, TcpStream}; [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `std::str` [INFO] [stdout] --> src/server.rs:10:5 [INFO] [stdout] | [INFO] [stdout] 10 | use std::str; [INFO] [stdout] | ^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `thread` [INFO] [stdout] --> src/server.rs:11:15 [INFO] [stdout] | [INFO] [stdout] 11 | use std::{io, thread}; [INFO] [stdout] | ^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `#[macro_use]` import [INFO] [stdout] --> src/lib.rs:5:1 [INFO] [stdout] | [INFO] [stdout] 5 | #[macro_use] [INFO] [stdout] | ^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused `#[macro_use]` import [INFO] [stdout] --> src/lib.rs:8:1 [INFO] [stdout] | [INFO] [stdout] 8 | #[macro_use] [INFO] [stdout] | ^^^^^^^^^^^^ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/bin/kvd.rs:73:9 [INFO] [stdout] | [INFO] [stdout] 73 | let mut server = Server::new(engine, server_port)?; [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: unused variable: `log_level` [INFO] [stdout] --> src/bin/kvd.rs:47:9 [INFO] [stdout] | [INFO] [stdout] 47 | let log_level = settings.get_str("log_level")?; // TODO: unused [INFO] [stdout] | ^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_log_level` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] error[E0061]: this function takes 2 arguments but 1 argument was supplied [INFO] [stdout] --> src/server.rs:120:22 [INFO] [stdout] | [INFO] [stdout] 120 | let server = Server::new(engine).unwrap(); [INFO] [stdout] | ^^^^^^^^^^^-------- argument #2 of type `u16` is missing [INFO] [stdout] | [INFO] [stdout] note: associated function defined here [INFO] [stdout] --> src/server.rs:19:12 [INFO] [stdout] | [INFO] [stdout] 19 | pub fn new(engine: T, port: u16) -> KvdResult> { [INFO] [stdout] | ^^^ --------- [INFO] [stdout] help: provide the argument [INFO] [stdout] | [INFO] [stdout] 120 | let server = Server::new(engine, /* u16 */).unwrap(); [INFO] [stdout] | +++++++++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/engine/bitcask.rs:86:17 [INFO] [stdout] | [INFO] [stdout] 86 | let mut pos = reader.seek(SeekFrom::Start(0))?; [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: unused variable: `cmd_pos` [INFO] [stdout] --> src/engine/bitcask.rs:136:13 [INFO] [stdout] | [INFO] [stdout] 136 | let cmd_pos = self.file_store.write_command(cmd)?; [INFO] [stdout] | ^^^^^^^ help: if this is intentional, prefix it with an underscore: `_cmd_pos` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `read_size` [INFO] [stdout] --> src/engine/bitcask.rs:213:13 [INFO] [stdout] | [INFO] [stdout] 213 | let read_size = wal_reader.read(data.as_mut_slice())?; [INFO] [stdout] | ^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_read_size` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/engine/bitcask.rs:204:13 [INFO] [stdout] | [INFO] [stdout] 204 | let mut wal_reader = self [INFO] [stdout] | ----^^^^^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `e` [INFO] [stdout] --> src/model.rs:82:13 [INFO] [stdout] | [INFO] [stdout] 82 | fn from(e: io::Error) -> Self { [INFO] [stdout] | ^ help: if this is intentional, prefix it with an underscore: `_e` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `e` [INFO] [stdout] --> src/model.rs:90:13 [INFO] [stdout] | [INFO] [stdout] 90 | fn from(e: serde_json::error::Error) -> Self { [INFO] [stdout] | ^ help: if this is intentional, prefix it with an underscore: `_e` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `e` [INFO] [stdout] --> src/model.rs:98:13 [INFO] [stdout] | [INFO] [stdout] 98 | fn from(e: ConfigError) -> Self { [INFO] [stdout] | ^ help: if this is intentional, prefix it with an underscore: `_e` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `e` [INFO] [stdout] --> src/model.rs:106:13 [INFO] [stdout] | [INFO] [stdout] 106 | fn from(e: Utf8Error) -> Self { [INFO] [stdout] | ^ help: if this is intentional, prefix it with an underscore: `_e` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: variable does not need to be mutable [INFO] [stdout] --> src/server.rs:48:13 [INFO] [stdout] | [INFO] [stdout] 48 | let mut reader = BufReader::new(&conn); [INFO] [stdout] | ----^^^^^^ [INFO] [stdout] | | [INFO] [stdout] | help: remove this `mut` [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] For more information about this error, try `rustc --explain E0061`. [INFO] [stdout] [INFO] [stderr] error: could not compile `kvd` (lib test) due to 1 previous error; 23 warnings emitted [INFO] [stderr] warning: build failed, waiting for other jobs to finish... [INFO] running `Command { std: "docker" "inspect" "351d341f48a3d1f2c552f53f112f3ab290a2cf40d8ccb55ceb657d71c0dc4a45", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "351d341f48a3d1f2c552f53f112f3ab290a2cf40d8ccb55ceb657d71c0dc4a45", kill_on_drop: false }` [INFO] [stdout] 351d341f48a3d1f2c552f53f112f3ab290a2cf40d8ccb55ceb657d71c0dc4a45