Here's a patch to fix the DB and log file permissions after "plinth
--setup". I also changed it to use the new command "--setup-no-install",
which ensures that the required packages are already installed and won't
try to install them like --setup does.
From 9835d5fb3a707dd638a52246255569b9eb2a9de1 Mon Sep 17 00:00:00 2001
From: James Valleroy <jvalle...@mailbox.org>
Date: Thu, 22 Sep 2016 18:04:48 -0400
Subject: [PATCH] Add patch to fix permissions and use new setup command

---
 debian/patches/0004-fix-permissions.patch | 15 +++++++++++++++
 debian/patches/series                     |  1 +
 2 files changed, 16 insertions(+)
 create mode 100644 debian/patches/0004-fix-permissions.patch

diff --git a/debian/patches/0004-fix-permissions.patch b/debian/patches/0004-fix-permissions.patch
new file mode 100644
index 0000000..be11f92
--- /dev/null
+++ b/debian/patches/0004-fix-permissions.patch
@@ -0,0 +1,15 @@
+Subject: Fix permissions on DB and log file. Use new setup command.
+
+--- a/data/usr/lib/freedombox/setup.d/86_plinth
++++ b/data/usr/lib/freedombox/setup.d/86_plinth
+@@ -37,6 +37,9 @@
+ echo "Running Plinth setup..."
+ 
+ # Run plinth setup to install various necessary program
+-plinth --setup
++plinth --setup-no-install
++
++# Ensure that DB and log file permissions are correct
++chown -R plinth: /var/lib/plinth /var/log/plinth
+ 
+ echo "Done running Plinth setup."
diff --git a/debian/patches/series b/debian/patches/series
index 3055da5..478ac7a 100644
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -1,3 +1,4 @@
 0001-disable-privileged-actions-test.patch
 0002-skip-copy-doc-dir.patch
 0003-disable-restore-module.patch
+0004-fix-permissions.patch
-- 
2.9.3

Attachment: signature.asc
Description: OpenPGP digital signature

Reply via email to