osmith has uploaded this change for review.

View Change

debian/postinst: add checks, be verbose

Do not attempt to change permissions/ownership if the package gets
upgraded from a version higher than the next release.

Do not fail if the user deleted the config file.

Be verbose when changing permissions.

Related: OS#4107
Change-Id: I8994759df644d6edd8f937051b95690537b749be
---
M debian/postinst
1 file changed, 31 insertions(+), 6 deletions(-)

git pull ssh://gerrit.osmocom.org:29418/osmo-smlc refs/changes/10/36810/1

diff --git a/debian/postinst b/debian/postinst
index faecd26..6c99b1f 100755
--- a/debian/postinst
+++ b/debian/postinst
@@ -16,12 +16,20 @@
 		fi
 
 		# Fix permissions of previous (root-owned) install (OS#4107)
-		chown osmocom:osmocom /etc/osmocom/osmo-smlc.cfg
-		chmod 0660 /etc/osmocom/osmo-smlc.cfg
-		chown root:osmocom /etc/osmocom
-		chmod 2775 /etc/osmocom
-		mkdir -p /var/lib/osmocom
-		chown -R osmocom:osmocom /var/lib/osmocom
+		if dpkg --compare-versions "$2" le "0.3.0"; then
+			if [ -e /etc/osmocom/osmo-smlc.cfg ]; then
+				chown -v osmocom:osmocom /etc/osmocom/osmo-smlc.cfg
+				chmod -v 0660 /etc/osmocom/osmo-smlc.cfg
+			fi
+
+			if [ -d /etc/osmocom ]; then
+				chown -v root:osmocom /etc/osmocom
+				chmod -v 2775 /etc/osmocom
+			fi
+
+			mkdir -p /var/lib/osmocom
+			chown -R -v osmocom:osmocom /var/lib/osmocom
+		fi
 		;;
 esac
 

To view, visit change 36810. To unsubscribe, or for help writing mail filters, visit settings.