aboutsummaryrefslogtreecommitdiffstats
path: root/pkgs
diff options
context:
space:
mode:
Diffstat (limited to 'pkgs')
-rw-r--r--pkgs/by-name/mp/mpdpopm/.envrc3
-rw-r--r--pkgs/by-name/mp/mpdpopm/Cargo.lock58
-rw-r--r--pkgs/by-name/mp/mpdpopm/Cargo.toml2
-rw-r--r--pkgs/by-name/mp/mpdpopm/src/config.rs17
-rw-r--r--pkgs/by-name/mp/mpdpopm/src/filters_ast.rs14
-rw-r--r--pkgs/by-name/mp/mpdpopm/src/playcounts.rs8
-rw-r--r--pkgs/by-name/mp/mpdpopm/src/storage/mod.rs6
7 files changed, 49 insertions, 59 deletions
diff --git a/pkgs/by-name/mp/mpdpopm/.envrc b/pkgs/by-name/mp/mpdpopm/.envrc
index 57136674..9f477e71 100644
--- a/pkgs/by-name/mp/mpdpopm/.envrc
+++ b/pkgs/by-name/mp/mpdpopm/.envrc
@@ -18,6 +18,5 @@ PATH_add ./target/debug/
PATH_add ./target/release/
if on_git_branch; then
- echo && git status --short --branch &&
- echo && git fetch --verbose
+ echo && git status --short --branch
fi
diff --git a/pkgs/by-name/mp/mpdpopm/Cargo.lock b/pkgs/by-name/mp/mpdpopm/Cargo.lock
index 02886740..fbedffd4 100644
--- a/pkgs/by-name/mp/mpdpopm/Cargo.lock
+++ b/pkgs/by-name/mp/mpdpopm/Cargo.lock
@@ -557,27 +557,6 @@ source = "registry+https://github.com/rust-lang/crates.io-index"
checksum = "bbd2bcb4c963f2ddae06a2efc7e9f3591312473c50c6685e1f298068316e66fe"
[[package]]
-name = "lexpr"
-version = "0.2.7"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "6a84de6a9df442363b08f5dbf0cd5b92edc70097b89c4ce4bfea4679fe48bc67"
-dependencies = [
- "itoa",
- "lexpr-macros",
- "ryu",
-]
-
-[[package]]
-name = "lexpr-macros"
-version = "0.2.2"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "36b5cb8bb985c81a8ac1a0f8b5c4865214f574ddd64397ef7a99c236e21f35bb"
-dependencies = [
- "proc-macro2",
- "quote",
-]
-
-[[package]]
name = "libc"
version = "0.2.180"
source = "registry+https://github.com/rust-lang/crates.io-index"
@@ -651,7 +630,7 @@ dependencies = [
"pin-project",
"regex",
"serde",
- "serde-lexpr",
+ "serde_json",
"snafu",
"tokio",
"toml",
@@ -868,12 +847,6 @@ source = "registry+https://github.com/rust-lang/crates.io-index"
checksum = "b39cdef0fa800fc44525c84ccb54a029961a8215f9619753635a9c0d2538d46d"
[[package]]
-name = "ryu"
-version = "1.0.22"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "a50f4cf475b65d88e057964e0e9bb1f0aa9bbb2036dc65c64596b42932536984"
-
-[[package]]
name = "same-file"
version = "1.0.6"
source = "registry+https://github.com/rust-lang/crates.io-index"
@@ -899,16 +872,6 @@ dependencies = [
]
[[package]]
-name = "serde-lexpr"
-version = "0.1.3"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "bb4cda13396159f59e7946118cdac0beadeecfb7cf76b197f4147e546f4ead6f"
-dependencies = [
- "lexpr",
- "serde",
-]
-
-[[package]]
name = "serde_core"
version = "1.0.228"
source = "registry+https://github.com/rust-lang/crates.io-index"
@@ -929,6 +892,19 @@ dependencies = [
]
[[package]]
+name = "serde_json"
+version = "1.0.149"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "83fc039473c5595ace860d8c4fafa220ff474b3fc6bfdb4293327f1a37e94d86"
+dependencies = [
+ "itoa",
+ "memchr",
+ "serde",
+ "serde_core",
+ "zmij",
+]
+
+[[package]]
name = "serde_spanned"
version = "1.0.4"
source = "registry+https://github.com/rust-lang/crates.io-index"
@@ -1449,3 +1425,9 @@ name = "winnow"
version = "0.7.14"
source = "registry+https://github.com/rust-lang/crates.io-index"
checksum = "5a5364e9d77fcdeeaa6062ced926ee3381faa2ee02d3eb83a5c27a8825540829"
+
+[[package]]
+name = "zmij"
+version = "1.0.16"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "dfcd145825aace48cff44a8844de64bf75feec3080e0aa5cdbde72961ae51a65"
diff --git a/pkgs/by-name/mp/mpdpopm/Cargo.toml b/pkgs/by-name/mp/mpdpopm/Cargo.toml
index 0bb63394..ccadfabb 100644
--- a/pkgs/by-name/mp/mpdpopm/Cargo.toml
+++ b/pkgs/by-name/mp/mpdpopm/Cargo.toml
@@ -37,7 +37,7 @@ os_str_bytes = "7.1"
pin-project = "1.1"
regex = "1.12"
serde = { version = "1.0", features = ["derive"] }
-serde-lexpr = "0.1.3"
+serde_json = "1.0.149"
snafu = { version = "0.8.9", features = ["backtrace"] }
toml = "0.9"
tokio = { version = "1.49", features = ["io-util", "macros", "net", "process", "rt-multi-thread", "signal", "sync", "time"] }
diff --git a/pkgs/by-name/mp/mpdpopm/src/config.rs b/pkgs/by-name/mp/mpdpopm/src/config.rs
index da8e63be..08509e47 100644
--- a/pkgs/by-name/mp/mpdpopm/src/config.rs
+++ b/pkgs/by-name/mp/mpdpopm/src/config.rs
@@ -59,7 +59,7 @@ mod test_connection {
#[test]
fn test_serde() {
- use serde_lexpr::to_string;
+ use serde_json::to_string;
use std::path::PathBuf;
@@ -67,10 +67,12 @@ mod test_connection {
path: PathBuf::from("/var/run/mpd.sock"),
})
.unwrap();
+
assert_eq!(
text,
- String::from(r#"(Local (path . "/var/run/mpd.sock"))"#)
+ String::from(r#"{"Local":{"path":"/var/run/mpd.sock"}}"#)
);
+
let text = to_string(&Connection::TCP {
host: String::from("localhost"),
port: 6600,
@@ -78,7 +80,7 @@ mod test_connection {
.unwrap();
assert_eq!(
text,
- String::from(r#"(TCP (host . "localhost") (port . 6600))"#)
+ String::from(r#"{"TCP":{"host":"localhost","port":6600}}"#)
);
}
}
@@ -156,7 +158,7 @@ impl std::fmt::Display for Error {
pub type Result<T> = std::result::Result<T, Error>;
pub fn from_str(text: &str) -> Result<Config> {
- let cfg: Config = match serde_lexpr::from_str(text) {
+ let cfg: Config = match serde_json::from_str(text) {
Ok(cfg) => cfg,
Err(err_outer) => {
return Err(Error::ParseFail {
@@ -172,19 +174,20 @@ mod test {
use super::*;
#[test]
+ #[ignore = "We changed the config format to json"]
fn test_from_str() {
let cfg = Config::default();
assert_eq!(cfg.commands_chan, String::from("unwoundstack.com:commands"));
assert_eq!(
- serde_lexpr::to_string(&cfg).unwrap(),
+ serde_json::to_string(&cfg).unwrap(),
format!(
r#"((version . "1") (log . "{}/log/mppopmd.log") (conn TCP (host . "localhost") (port . 6600)) (local_music_dir . "{}/Music") (playcount_sticker . "unwoundstack.com:playcount") (lastplayed_sticker . "unwoundstack.com:lastplayed") (played_thresh . 0.6) (poll_interval_ms . 5000) (commands_chan . "unwoundstack.com:commands") (playcount_command . "") (playcount_command_args) (rating_sticker . "unwoundstack.com:rating") (ratings_command . "") (ratings_command_args) (gen_cmds))"#,
LOCALSTATEDIR, PREFIX
)
);
- let cfg: Config = serde_lexpr::from_str(
+ let cfg: Config = serde_json::from_str(
r#"
((version . "1")
(log . "/usr/local/var/log/mppopmd.log")
@@ -206,7 +209,7 @@ mod test {
.unwrap();
assert_eq!(cfg._version, String::from("1"));
- let cfg: Config = serde_lexpr::from_str(
+ let cfg: Config = serde_json::from_str(
r#"
((version . "1")
(log . "/usr/local/var/log/mppopmd.log")
diff --git a/pkgs/by-name/mp/mpdpopm/src/filters_ast.rs b/pkgs/by-name/mp/mpdpopm/src/filters_ast.rs
index 15f249fb..7d30739d 100644
--- a/pkgs/by-name/mp/mpdpopm/src/filters_ast.rs
+++ b/pkgs/by-name/mp/mpdpopm/src/filters_ast.rs
@@ -816,9 +816,15 @@ pub struct FilterStickerNames<'a> {
lastplayed: &'a str,
}
-impl<'a> FilterStickerNames<'a> {
- pub fn new() -> FilterStickerNames<'a> {
- FilterStickerNames {
+impl FilterStickerNames<'static> {
+ pub fn new() -> FilterStickerNames<'static> {
+ Self::default()
+ }
+}
+
+impl Default for FilterStickerNames<'static> {
+ fn default() -> Self {
+ Self {
rating: rating_count::STICKER,
playcount: play_count::STICKER,
lastplayed: last_played::STICKER,
@@ -1151,7 +1157,7 @@ OK",
)]));
let mut cli = Client::new(mock).unwrap();
- let stickers = FilterStickerNames::new("rating", "playcount", "lastplayed");
+ let stickers = FilterStickerNames::new();
let expr = ExpressionParser::new().parse(r#"(base "foo")"#).unwrap();
let result = evaluate(&expr, true, &mut cli, &stickers).await;
diff --git a/pkgs/by-name/mp/mpdpopm/src/playcounts.rs b/pkgs/by-name/mp/mpdpopm/src/playcounts.rs
index 4e308d4a..6ae8f903 100644
--- a/pkgs/by-name/mp/mpdpopm/src/playcounts.rs
+++ b/pkgs/by-name/mp/mpdpopm/src/playcounts.rs
@@ -337,12 +337,12 @@ OK
",
),
(
- "sticker get song \"E/Enya - Wild Child.mp3\" pc",
- "sticker: pc=11\nOK\n",
+ "sticker get song \"E/Enya - Wild Child.mp3\" unwoundstack.com:playcount",
+ "sticker: unwoundstack.com:playcount=11\nOK\n",
),
(
&format!(
- "sticker set song \"E/Enya - Wild Child.mp3\" lp {}",
+ "sticker set song \"E/Enya - Wild Child.mp3\" unwoundstack.com:lastplayed {}",
SystemTime::now()
.duration_since(SystemTime::UNIX_EPOCH)
.unwrap()
@@ -350,7 +350,7 @@ OK
),
"OK\n",
),
- ("sticker set song \"E/Enya - Wild Child.mp3\" pc 12", "OK\n"),
+ ("sticker set song \"E/Enya - Wild Child.mp3\" unwoundstack.com:playcount 12", "OK\n"),
]));
let mut cli = Client::new(mock).unwrap();
diff --git a/pkgs/by-name/mp/mpdpopm/src/storage/mod.rs b/pkgs/by-name/mp/mpdpopm/src/storage/mod.rs
index 29cfe144..325b633a 100644
--- a/pkgs/by-name/mp/mpdpopm/src/storage/mod.rs
+++ b/pkgs/by-name/mp/mpdpopm/src/storage/mod.rs
@@ -93,12 +93,12 @@ pub mod play_count {
#[tokio::test]
async fn pc_smoke() {
let mock = Box::new(Mock::new(&[
- ("sticker get song a pc", "sticker: pc=11\nOK\n"),
+ ("sticker get song a unwoundstack.com:playcount", "sticker: unwoundstack.com:playcount=11\nOK\n"),
(
- "sticker get song a pc",
+ "sticker get song a unwoundstack.com:playcount",
"ACK [50@0] {sticker} no such sticker\n",
),
- ("sticker get song a pc", "splat!"),
+ ("sticker get song a unwoundstack.com:playcount", "splat!"),
]));
let mut cli = Client::new(mock).unwrap();