Package: runit-helper
Version: 2.8.14
Severity: normal
Tags: patch

Hi,
I think runit-helper should remove log directories when
called with purge. See Debian Policy 10.8.
Patch attached.

Lorenzo

-- System Information:
Debian Release: bullseye/sid
  APT prefers unstable
  APT policy: (500, 'unstable'), (1, 'experimental')
Architecture: amd64 (x86_64)

Kernel: Linux 4.20.3-van (SMP w/4 CPU cores; PREEMPT)
Kernel taint flags: TAINT_OOT_MODULE
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8), 
LANGUAGE=en_US:en (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
Init: runit (via /run/runit.stopit)

-- no debconf information

-- debsums errors found:
debsums: changed file /lib/runit-helper/runit-helper (from runit-helper package)
>From 31304be015c0ea49a48fdb97e5944baa9b3396c6 Mon Sep 17 00:00:00 2001
From: Lorenzo Puliti <[email protected]>
Date: Mon, 7 Oct 2019 17:33:33 +0200
Subject: [PATCH] Remove log dir on purge

Remove the log directory and all its contents when postrm script
is called with 'purge'
---
 runit-helper | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/runit-helper b/runit-helper
index dfb5e0f..70c42f2 100755
--- a/runit-helper
+++ b/runit-helper
@@ -83,6 +83,12 @@ postrm () {
                        rmdir --ignore-fail-on-non-empty "$supervise"
                fi
        done
+       if [ "${action}" = 'purge' ] ; then
+            logdir="/var/log/runit/$NAME"
+           if [ -d "$logdir" ] ; then
+                rm -r "$logdir"
+           fi
+       fi
 }
 
 "$@"
-- 
2.23.0

Reply via email to