osmith has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/osmo-upf/+/36811?usp=email )


Change subject: debian/postinst: add checks, be verbose
......................................................................

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-upf refs/changes/11/36811/1

diff --git a/debian/postinst b/debian/postinst
index 4310e7e..3298043 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-upf.cfg
-               chmod 0660 /etc/osmocom/osmo-upf.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.2.0"; then
+                       if [ -e /etc/osmocom/osmo-upf.cfg ]; then
+                               chown -v osmocom:osmocom 
/etc/osmocom/osmo-upf.cfg
+                               chmod -v 0660 /etc/osmocom/osmo-upf.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 https://gerrit.osmocom.org/c/osmo-upf/+/36811?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-upf
Gerrit-Branch: master
Gerrit-Change-Id: I8994759df644d6edd8f937051b95690537b749be
Gerrit-Change-Number: 36811
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <osm...@sysmocom.de>
Gerrit-MessageType: newchange

Reply via email to