Control: reopen -1
Control: notfixed 2.0.34-2

>    * Partly revert f1f48dee so that directories are again setup correctly
>      on initial installations. (Closes: #885454)

You did that for munin, but not munin-node or munin-async.

I suggest this patch.

SR

-- 
Stefano Rivera
  http://tumbleweed.org.za/
  +1 415 683 3272
From e129c85f209e903db10827642110b831424422a0 Mon Sep 17 00:00:00 2001
From: Stefano Rivera <stefa...@debian.org>
Date: Thu, 10 May 2018 23:14:14 -0700
Subject: [PATCH] Revert more of f1f48dee so that directories are again setup
 correctly on initial installations of munin-node and munin-async. (Closes:
 #885454)

---
 debian/changelog            |  2 ++
 debian/munin-async.postinst |  7 +++++++
 debian/munin-node.postinst  | 11 +++++++++++
 3 files changed, 20 insertions(+)

diff --git a/debian/changelog b/debian/changelog
index 7ac9f6a8..905c6753 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,6 +1,8 @@
 munin (2.0.37-2) UNRELEASED; urgency=medium
 
   * Bump Standards-Version: to 4.1.4, no changes needed.
+  * Revert more of f1f48dee so that directories are again setup correctly on
+    initial installations of munin-node and munin-async. (Closes: #885454)
 
  -- Holger Levsen <hol...@debian.org>  Thu, 05 Apr 2018 17:47:11 +0000
 
diff --git a/debian/munin-async.postinst b/debian/munin-async.postinst
index a940e507..cf104824 100644
--- a/debian/munin-async.postinst
+++ b/debian/munin-async.postinst
@@ -15,9 +15,16 @@ add_munin_async_user() {
 	fi
 }
 
+initperms() {
+	chown munin-async:munin-async /var/lib/munin-async
+}
+
 case "$1" in
 	configure)
 		add_munin_async_user
+		if [ -z "$2" ]; then
+			initperms
+		fi
 		;;
 	abort-upgrade|abort-deconfigure|abort-remove)
 		:
diff --git a/debian/munin-node.postinst b/debian/munin-node.postinst
index 845e9424..4f54d6f9 100644
--- a/debian/munin-node.postinst
+++ b/debian/munin-node.postinst
@@ -4,6 +4,14 @@ set -e
 
 prevver="$2"
 
+initperms() {
+	chown munin:adm /var/log/munin
+	chmod 755 /var/log/munin
+
+	chown root:munin /etc/munin/plugin-conf.d
+	chmod 750 /etc/munin/plugin-conf.d
+}
+
 init_plugins() {
 	TMPFILE=`mktemp /tmp/munin-node.configure.XXXXXXXXXX`
 	TMPFILE_STDERR=`mktemp /tmp/munin-node.configure.err.XXXXXXXXXX`
@@ -41,6 +49,9 @@ init_plugins() {
 
 case "$1" in
 	configure)
+		if [ -z "$2" ]; then
+			initperms
+		fi
 		init_plugins
 		;;
         triggered)
-- 
2.17.0

Reply via email to