changed default log file to ~/.odilia.log

common/Cargo.toml

@@ -21,3 +21,4 @@ thiserror = "1.0.37"
 zbus.workspace = true
 serde_plain.workspace = true
 figment = "0.10.15"
+xdg = "2.4.1"

common/src/settings/log.rs

@@ -1,6 +1,6 @@
+use serde::{Deserialize, Serialize};
 use std::path::PathBuf;
 
-use serde::{Deserialize, Serialize};
 ///structure used for all the configurable options related to logging
 #[derive(Debug, Serialize, Deserialize)]
 #[allow(clippy::module_name_repetitions)]
@@ -16,10 +16,14 @@ pub struct LogSettings {
 }
 impl Default for LogSettings {
 	fn default() -> Self {
-		Self {
-			level: "info".to_owned(),
-			logger: LoggingKind::File("/var/log/odilia.log".into()),
-		}
+        let xdg_dirs = xdg::BaseDirectories::with_prefix("odilia").expect(
+			"unable to find the odilia config directory according to the xdg dirs specification",
+		);
+        let log_path = xdg_dirs
+            .place_data_file("odilia.log")
+            .expect("unable to place log file");
+
+		Self { level: "info".to_owned(), logger: LoggingKind::File(log_path) }
 	}
 }
 

odilia/src/logging.rs

@@ -27,14 +27,14 @@ pub fn init(config: &ApplicationConfig) -> eyre::Result<()> {
 	//this requires boxing because the types returned by this match block would be incompatible otherwise, since we return different layers depending on what we get from the configuration. It is possible to do it otherwise, hopefully, but for now this and a forced dereference at the end would do
 	let output_layer = match &config.log.logger {
 		LoggingKind::File(path) => {
-			let file = std::fs::File::options()
+            let file = std::fs::File::options()
 				.create_new(true)
 				.write(true)
 				.open(path)
 				.with_context(|| {
 					format!("creating log file '{}'", path.display())
 				})?;
-			let fmt =
+            let fmt =
 				tracing_subscriber::fmt::layer().with_ansi(false).with_writer(file);
 			fmt.boxed()
 		}