Remove task logs on 'pueue remove'

Nukesor · Mar 21, 2021 · 43a26de · 43a26de
1 parent a8b7443
commit 43a26de
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -8,6 +8,8 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 
 ### Fixed
 
+- Remove task logs on `pueue remove`. [#187](https://github.com/Nukesor/pueue/issues/187)
+
 ## [0.12.1] - 12-03-2021
 
 ### Fixed

diff --git a/client/cli.rs b/client/cli.rs
@@ -319,7 +319,7 @@ pub enum SubCommand {
         quiet: bool,
     },
 
-    /// Remove all finished tasks from the list (also clears logs).
+    /// Remove all finished tasks from the list.
     Clean {
         /// Only clean tasks that finished successfully.
         #[clap(short, long)]

diff --git a/daemon/network/message_handler/remove.rs b/daemon/network/message_handler/remove.rs
@@ -1,3 +1,4 @@
+use pueue_lib::log::clean_log_handles;
 use pueue_lib::network::message::*;
 use pueue_lib::state::SharedState;
 use pueue_lib::task::TaskStatus;
@@ -28,6 +29,8 @@ pub fn remove(task_ids: Vec<usize>, state: &SharedState) -> Message {
 
     for task_id in &not_running {
         state.tasks.remove(task_id);
+
+        clean_log_handles(*task_id, &state.settings.shared.pueue_directory);
     }
 
     let text = "Tasks removed from list";