rmenu/src/cache.rs

@@ -3,11 +3,15 @@ use std::fs;
 use std::path::PathBuf;
 use std::time::{Duration, SystemTime};
 
+use once_cell::sync::Lazy;
 use rmenu_plugin::Entry;
 use thiserror::Error;
 
 use crate::config::{CacheSetting, PluginConfig};
-use crate::XDG_PREFIX;
+use crate::CONFIG_DIR;
+
+static CONFIG_PATH: Lazy<PathBuf> =
+    Lazy::new(|| PathBuf::from(shellexpand::tilde(CONFIG_DIR).to_string()));
 
 #[derive(Debug, Error)]
 pub enum CacheError {
@@ -25,10 +29,7 @@ pub enum CacheError {
 
 #[inline]
 fn cache_file(name: &str) -> PathBuf {
-    xdg::BaseDirectories::with_prefix(XDG_PREFIX)
-        .expect("Failed to read xdg base dirs")
-        .place_cache_file(format!("{name}.cache"))
-        .expect("Failed to write xdg cache dirs")
+    CONFIG_PATH.join(format!("{name}.cache"))
 }
 
 /// Read Entries from Cache (if Valid and Available)

rmenu/src/cli.rs

@@ -10,7 +10,7 @@ use rmenu_plugin::{Entry, Message};
 use thiserror::Error;
 
 use crate::config::{cfg_replace, Config, Keybind};
-use crate::{DEFAULT_CONFIG, DEFAULT_THEME, XDG_PREFIX};
+use crate::{DEFAULT_CONFIG, DEFAULT_THEME};
 
 /// Allowed Formats for Entry Ingestion
 #[derive(Debug, Clone)]
@@ -179,26 +179,20 @@ pub enum RMenuError {
 pub type Result<T> = std::result::Result<T, RMenuError>;
 
 impl Args {
-    /// Find a specifically named file across xdg config paths
-    fn find_xdg_file(&self, name: &str, base: &Option<PathBuf>) -> Option<String> {
-        return base
-            .clone()
-            .or_else(|| {
-                xdg::BaseDirectories::with_prefix(XDG_PREFIX)
+    /// Find Configuration Path
+    pub fn find_config(&self) -> PathBuf {
+        self.config.clone().unwrap_or_else(|| {
+            xdg::BaseDirectories::with_prefix("rmenu")
                 .expect("Failed to read xdg base dirs")
-                    .find_config_file(name)
+                .find_config_file(DEFAULT_CONFIG)
+                .unwrap_or_else(PathBuf::new)
         })
-            .map(|f| {
-                let f = f.to_string_lossy().to_string();
-                shellexpand::tilde(&f).to_string()
-            });
     }
 
     /// Load Configuration File
-    pub fn get_config(&self) -> Result<Config> {
-        let config = self.find_xdg_file(DEFAULT_CONFIG, &self.config);
-
-        if let Some(path) = config {
+    pub fn get_config(&self, path: &PathBuf) -> Result<Config> {
+        let path = path.to_string_lossy().to_string();
+        let path = shellexpand::tilde(&path).to_string();
         let config: Config = match read_to_string(path) {
             Ok(content) => serde_yaml::from_str(&content),
             Err(err) => {
@@ -206,10 +200,7 @@ impl Args {
                 Ok(Config::default())
             }
         }?;
-            return Ok(config);
-        }
-        log::error!("Failed to Load Config: no file found in xdg config paths");
-        Ok(Config::default())
+        Ok(config)
     }
 
     /// Update Configuration w/ CLI Specified Settings
@@ -251,16 +242,17 @@ impl Args {
     }
 
     /// Load CSS Theme or Default
-    pub fn get_theme(&self) -> String {
-        self.find_xdg_file(DEFAULT_THEME, &self.theme)
-            .map(read_to_string)
-            .map(|f| {
-                f.unwrap_or_else(|err| {
-                    log::error!("Failed to load CSS: {err:?}");
+    pub fn get_theme(&self, cfgdir: &PathBuf) -> String {
+        let theme = self.theme.clone().or(Some(cfgdir.join(DEFAULT_THEME)));
+        if let Some(theme) = theme {
+            let path = theme.to_string_lossy().to_string();
+            let path = shellexpand::tilde(&path).to_string();
+            match read_to_string(&path) {
+                Ok(css) => return css,
+                Err(err) => log::error!("Failed to load CSS: {err:?}"),
+            }
+        }
         String::new()
-                })
-            })
-            .unwrap_or_else(String::new)
     }
 
     /// Load Additional CSS or Default
@@ -268,9 +260,9 @@ impl Args {
         let css = self
             .css
             .clone()
-            .map(|s| s.to_string_lossy().to_string())
-            .or(c.css.clone());
-        if let Some(path) = css {
+            .or(c.css.as_ref().map(|s| PathBuf::from(s)));
+        if let Some(css) = css {
+            let path = css.to_string_lossy().to_string();
             let path = shellexpand::tilde(&path).to_string();
             match read_to_string(&path) {
                 Ok(css) => return css,

rmenu/src/main.rs

@@ -10,9 +10,9 @@ mod state;
 use clap::Parser;
 use rmenu_plugin::{self_exe, Entry};
 
+static CONFIG_DIR: &'static str = "~/.config/rmenu/";
 static DEFAULT_THEME: &'static str = "style.css";
 static DEFAULT_CONFIG: &'static str = "config.yaml";
-static XDG_PREFIX: &'static str = "rmenu";
 static DEFAULT_CSS_CONTENT: &'static str = include_str!("../public/default.css");
 
 /// Application State for GUI
@@ -43,7 +43,8 @@ fn main() -> cli::Result<()> {
 
     // parse cli and retrieve values for app
     let mut cli = cli::Args::parse();
-    let mut config = cli.get_config()?;
+    let mut cfgpath = cli.find_config();
+    let mut config = cli.get_config(&cfgpath)?;
     let entries = cli.get_entries(&mut config)?;
 
     // update config based on cli-settings and entries
@@ -54,7 +55,10 @@ fn main() -> cli::Result<()> {
             .any(|e| e.icon.is_some() || e.icon_alt.is_some());
     config.use_comments = config.use_comments && entries.iter().any(|e| e.comment.is_some());
 
-    let theme = cli.get_theme();
+    // retrieve cfgdir and get theme/css
+    cfgpath.pop();
+    let cfgdir = cfgpath;
+    let theme = cli.get_theme(&cfgdir);
     let css = cli.get_css(&config);
 
     // genrate app context and run gui