From e2e826259d3685d29831de382365eb655b011dac Mon Sep 17 00:00:00 2001 From: Kamil Jarosz Date: Sat, 14 Sep 2024 10:15:30 +0200 Subject: [PATCH] desktop: Do not store logs in config directory Logs are non-essential and can be removed, contrary to config files. This patch changes the log directory from config dir to cache dir. --- desktop/src/main.rs | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/desktop/src/main.rs b/desktop/src/main.rs index 72b468a6a..13d4a7682 100644 --- a/desktop/src/main.rs +++ b/desktop/src/main.rs @@ -155,11 +155,14 @@ async fn main() -> Result<(), Error> { let opt = Opt::parse(); let preferences = GlobalPreferences::load(opt.clone())?; + let logs_path = &preferences.cli.cache_directory.join("log"); + let log_path = preferences.log_filename_pattern().create_path(logs_path); + if let Some(parent) = log_path.parent() { + std::fs::create_dir_all(parent)?; + } + // [NA] `_guard` cannot be `_` or it'll immediately drop // https://docs.rs/tracing-appender/latest/tracing_appender/non_blocking/index.html - let log_path = preferences - .log_filename_pattern() - .create_path(&preferences.cli.config); let (non_blocking_file, _file_guard) = tracing_appender::non_blocking(File::create(log_path)?); let (non_blocking_stdout, _stdout_guard) = tracing_appender::non_blocking(std::io::stdout());