[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 1.98.0-beta.8+cargoflags=--release for beta-release-1.99-2
[INFO] running `Command { std: "git" "clone" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Floxp%2Fkvd" "/workspace/builds/worker-0-tc1/source", kill_on_drop: false }`
[INFO] [stderr] Cloning into '/workspace/builds/worker-0-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-0-tc1/source/Cargo.toml
[INFO] validating manifest of git repo https://github.com/loxp/kvd on toolchain 1.98.0-beta.8
[INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+1.98.0-beta.8" "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" "+1.98.0-beta.8" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }`
[INFO] [stderr]     Blocking waiting for file lock on package cache
[INFO] [stderr]     Updating crates.io index
[INFO] [stderr]     Blocking waiting for file lock on package cache
[INFO] [stderr]  Downloading crates ...
[INFO] [stderr]   Downloaded thread_local v0.3.6
[INFO] [stderr]   Downloaded redox_users v0.3.1
[INFO] [stderr]   Downloaded slog-term v2.4.1
[INFO] [stderr]   Downloaded dirs v1.0.5
[INFO] [stderr]   Downloaded term v0.5.2
[INFO] [stderr]   Downloaded unicode-xid v0.2.0
[INFO] [stderr]   Downloaded crossbeam-deque v0.7.1
[INFO] [stderr]   Downloaded slog-scope v4.1.2
[INFO] [stderr]   Downloaded predicates-core v1.0.0
[INFO] [stderr]   Downloaded slog-async v2.3.0
[INFO] [stderr]   Downloaded memoffset v0.5.1
[INFO] [stderr]   Downloaded arc-swap v0.3.11
[INFO] [stderr]   Downloaded ryu v1.0.0
[INFO] [stderr]   Downloaded normalize-line-endings v0.2.2
[INFO] [stderr]   Downloaded arrayref v0.3.5
[INFO] [stderr]   Downloaded failure_derive v0.1.5
[INFO] [stderr]   Downloaded crossbeam v0.7.2
[INFO] [stderr]   Downloaded scopeguard v1.0.0
[INFO] [stderr]   Downloaded linked-hash-map v0.5.2
[INFO] [stderr]   Downloaded num-traits v0.1.43
[INFO] [stderr]   Downloaded slog-stdlog v4.0.0
[INFO] [stderr]   Downloaded atty v0.2.13
[INFO] [stderr]   Downloaded rust-ini v0.13.0
[INFO] [stderr]   Downloaded version_check v0.1.5
[INFO] [stderr]   Downloaded crossbeam-queue v0.1.2
[INFO] [stderr]   Downloaded itoa v0.4.4
[INFO] [stderr]   Downloaded predicates-tree v1.0.0
[INFO] [stderr]   Downloaded treeline v0.1.0
[INFO] [stderr]   Downloaded constant_time_eq v0.1.4
[INFO] [stderr]   Downloaded float-cmp v0.4.0
[INFO] [stderr]   Downloaded linked-hash-map v0.3.0
[INFO] [stderr]   Downloaded lazy_static v0.2.11
[INFO] [stderr]   Downloaded cfg-if v0.1.9
[INFO] [stderr]   Downloaded autocfg v0.1.6
[INFO] [stderr]   Downloaded serde_test v0.8.23
[INFO] [stderr]   Downloaded serde-hjson v0.8.2
[INFO] [stderr]   Downloaded nodrop v0.1.13
[INFO] [stderr]   Downloaded arrayvec v0.4.11
[INFO] [stderr]   Downloaded synstructure v0.10.2
[INFO] [stderr]   Downloaded assert_cmd v0.11.1
[INFO] [stderr]   Downloaded quote v1.0.2
[INFO] [stderr]   Downloaded predicates v1.0.1
[INFO] [stderr]   Downloaded memchr v2.2.1
[INFO] [stderr]   Downloaded config v0.9.3
[INFO] [stderr]   Downloaded bitflags v1.1.0
[INFO] [stderr]   Downloaded serde v0.8.23
[INFO] [stderr]   Downloaded escargot v0.4.0
[INFO] [stderr]   Downloaded failure v0.1.5
[INFO] [stderr]   Downloaded num-traits v0.2.8
[INFO] [stderr]   Downloaded num-integer v0.1.41
[INFO] [stderr]   Downloaded crossbeam-epoch v0.7.2
[INFO] [stderr]   Downloaded proc-macro2 v1.0.3
[INFO] [stderr]   Downloaded yaml-rust v0.4.3
[INFO] [stderr]   Downloaded toml v0.4.10
[INFO] [stderr]   Downloaded unicode-width v0.1.6
[INFO] [stderr]   Downloaded rust-argon2 v0.5.1
[INFO] [stderr]   Downloaded byteorder v1.3.2
[INFO] [stderr]   Downloaded slog v2.5.2
[INFO] [stderr]   Downloaded crossbeam-utils v0.6.6
[INFO] [stderr]   Downloaded crossbeam-channel v0.3.9
[INFO] [stderr]   Downloaded serde_derive v1.0.100
[INFO] [stderr]   Downloaded blake2b_simd v0.5.8
[INFO] [stderr]   Downloaded cc v1.0.41
[INFO] [stderr]   Downloaded serde v1.0.99
[INFO] [stderr]   Downloaded backtrace v0.3.35
[INFO] [stderr]   Downloaded serde_json v1.0.40
[INFO] [stderr]   Downloaded nom v4.2.3
[INFO] [stderr]   Downloaded chrono v0.4.9
[INFO] [stderr]   Downloaded aho-corasick v0.7.6
[INFO] [stderr]   Downloaded difference v2.0.0
[INFO] [stderr]   Downloaded syn v1.0.5
[INFO] [stderr]   Downloaded clap v2.33.0
[INFO] [stderr]   Downloaded regex v1.3.1
[INFO] [stderr]   Downloaded regex-syntax v0.6.12
[INFO] [stderr]   Downloaded libc v0.2.62
[INFO] [stderr]   Downloaded backtrace-sys v0.1.31
[INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-0-tc1/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-0-tc1/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:8683fc1fc2eb5c9ac98e0d076ab094b2ffac7f99da555d2b6a2e27f346de2ec7" "sleep" "infinity", kill_on_drop: false }`
[INFO] [stdout] 525fe1b6fa1fad4699e4375f55f0758cd33858209d16dd0fb2f36317eeda8b32
[INFO] running `Command { std: "docker" "start" "525fe1b6fa1fad4699e4375f55f0758cd33858209d16dd0fb2f36317eeda8b32", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "525fe1b6fa1fad4699e4375f55f0758cd33858209d16dd0fb2f36317eeda8b32", 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" "525fe1b6fa1fad4699e4375f55f0758cd33858209d16dd0fb2f36317eeda8b32" "/opt/rustwide/cargo-home/bin/cargo" "+1.98.0-beta.8" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "525fe1b6fa1fad4699e4375f55f0758cd33858209d16dd0fb2f36317eeda8b32", 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=warn" "-e" "RUSTDOCFLAGS=--cap-lints=warn" "-w" "/opt/rustwide/workdir" "--user" "0:0" "525fe1b6fa1fad4699e4375f55f0758cd33858209d16dd0fb2f36317eeda8b32" "/opt/rustwide/cargo-home/bin/cargo" "+1.98.0-beta.8" "build" "--frozen" "--message-format=json" "--release", 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 semver-parser v0.7.0
[INFO] [stderr]    Compiling unicode-xid v0.2.0
[INFO] [stderr]    Compiling memchr v2.2.1
[INFO] [stderr]    Compiling syn v1.0.5
[INFO] [stderr]    Compiling arrayvec v0.4.11
[INFO] [stderr]    Compiling slog v2.5.2
[INFO] [stderr]    Compiling thread_local v0.3.6
[INFO] [stderr]    Compiling serde v1.0.99
[INFO] [stderr]    Compiling syn v0.15.44
[INFO] [stderr]    Compiling nodrop v0.1.13
[INFO] [stderr]    Compiling cc v1.0.41
[INFO] [stderr]    Compiling ryu v1.0.0
[INFO] [stderr]    Compiling crossbeam-utils v0.6.6
[INFO] [stderr]    Compiling byteorder v1.3.2
[INFO] [stderr]    Compiling semver v0.9.0
[INFO] [stderr]    Compiling serde v0.8.23
[INFO] [stderr]    Compiling version_check v0.1.5
[INFO] [stderr]    Compiling scopeguard v1.0.0
[INFO] [stderr]    Compiling failure_derive v0.1.5
[INFO] [stderr]    Compiling bitflags v1.1.0
[INFO] [stderr]    Compiling rustc_version v0.2.3
[INFO] [stderr]    Compiling num-traits v0.2.8
[INFO] [stderr]    Compiling num-integer v0.1.41
[INFO] [stderr]    Compiling regex-syntax v0.6.12
[INFO] [stderr]    Compiling log v0.4.8
[INFO] [stderr]    Compiling memoffset v0.5.1
[INFO] [stderr]    Compiling itoa v0.4.4
[INFO] [stderr]    Compiling nom v4.2.3
[INFO] [stderr]    Compiling arc-swap v0.3.11
[INFO] [stderr]    Compiling linked-hash-map v0.5.2
[INFO] [stderr]    Compiling lazy_static v0.2.11
[INFO] [stderr]    Compiling unicode-width v0.1.6
[INFO] [stderr]    Compiling crossbeam-epoch v0.7.2
[INFO] [stderr]    Compiling crossbeam-queue v0.1.2
[INFO] [stderr]    Compiling crossbeam-channel v0.3.9
[INFO] [stderr]    Compiling aho-corasick v0.7.6
[INFO] [stderr]    Compiling dirs v1.0.5
[INFO] [stderr]    Compiling time v0.1.42
[INFO] [stderr]    Compiling atty v0.2.13
[INFO] [stderr]    Compiling slog-scope v4.1.2
[INFO] [stderr]    Compiling term v0.5.2
[INFO] [stderr]    Compiling yaml-rust v0.4.3
[INFO] [stderr]    Compiling quote v1.0.2
[INFO] [stderr]    Compiling textwrap v0.11.0
[INFO] [stderr]    Compiling backtrace-sys v0.1.31
[INFO] [stderr]    Compiling crossbeam-deque v0.7.1
[INFO] [stderr]    Compiling vec_map v0.8.1
[INFO] [stderr]    Compiling rust-ini v0.13.0
[INFO] [stderr]    Compiling strsim v0.8.0
[INFO] [stderr]    Compiling crossbeam v0.7.2
[INFO] [stderr]    Compiling ansi_term v0.11.0
[INFO] [stderr]    Compiling slog-stdlog v4.0.0
[INFO] [stderr]    Compiling slog-async v2.3.0
[INFO] [stderr]    Compiling num-traits v0.1.43
[INFO] [stderr]    Compiling clap v2.33.0
[INFO] [stderr]    Compiling synstructure v0.10.2
[INFO] [stderr]    Compiling chrono v0.4.9
[INFO] [stderr]    Compiling serde_test v0.8.23
[INFO] [stderr]    Compiling linked-hash-map v0.3.0
[INFO] [stderr]    Compiling regex v1.3.1
[INFO] [stderr]    Compiling slog-term v2.4.1
[INFO] [stderr]    Compiling serde_derive v1.0.100
[INFO] [stderr]    Compiling serde-hjson v0.8.2
[INFO] [stderr]    Compiling backtrace v0.3.35
[INFO] [stderr]    Compiling failure v0.1.5
[INFO] [stderr]    Compiling toml v0.4.10
[INFO] [stderr]    Compiling serde_json v1.0.40
[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)]` (part of `#[warn(unused)]`) 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]    = note: see <https://doc.rust-lang.org/nightly/rustc/check-cfg/cargo-specifics.html> 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]    = note: see <https://doc.rust-lang.org/nightly/rustc/check-cfg/cargo-specifics.html> 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)]` (part of `#[warn(unused)]`) 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)]` (part of `#[warn(unused)]`) on by default
[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: `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: 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)]` (part of `#[warn(unused)]`) 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)]` (part of `#[warn(unused)]`) 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)]` (part of `#[warn(unused)]`) 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)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr]     Finished `release` profile [optimized] target(s) in 1m 03s
[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" "525fe1b6fa1fad4699e4375f55f0758cd33858209d16dd0fb2f36317eeda8b32", 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=warn" "-e" "RUSTDOCFLAGS=--cap-lints=warn" "-w" "/opt/rustwide/workdir" "--user" "0:0" "525fe1b6fa1fad4699e4375f55f0758cd33858209d16dd0fb2f36317eeda8b32" "/opt/rustwide/cargo-home/bin/cargo" "+1.98.0-beta.8" "test" "--frozen" "--no-run" "--message-format=json" "--release", 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 treeline v0.1.0
[INFO] [stderr]    Compiling difference v2.0.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)]` (part of `#[warn(unused)]`) 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]    = note: see <https://doc.rust-lang.org/nightly/rustc/check-cfg/cargo-specifics.html> 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]    = note: see <https://doc.rust-lang.org/nightly/rustc/check-cfg/cargo-specifics.html> 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)]` (part of `#[warn(unused)]`) 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)]` (part of `#[warn(unused)]`) on by default
[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: `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: 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)]` (part of `#[warn(unused)]`) 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)]` (part of `#[warn(unused)]`) 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)]` (part of `#[warn(unused)]`) 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)]` (part of `#[warn(unused)]`) 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: 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)]` (part of `#[warn(unused)]`) 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)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[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)]` (part of `#[warn(unused)]`) 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]    = note: see <https://doc.rust-lang.org/nightly/rustc/check-cfg/cargo-specifics.html> 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]    = note: see <https://doc.rust-lang.org/nightly/rustc/check-cfg/cargo-specifics.html> 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] 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<Server<T>> {
[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)]` (part of `#[warn(unused)]`) 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)]` (part of `#[warn(unused)]`) on by default
[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: `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: 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] running `Command { std: "docker" "inspect" "525fe1b6fa1fad4699e4375f55f0758cd33858209d16dd0fb2f36317eeda8b32", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "525fe1b6fa1fad4699e4375f55f0758cd33858209d16dd0fb2f36317eeda8b32", kill_on_drop: false }`
[INFO] [stdout] 525fe1b6fa1fad4699e4375f55f0758cd33858209d16dd0fb2f36317eeda8b32
