One of the packaging changes in the SRU which I don't see an explicit
bug for (so I'm attaching the question here) is:

+++ ubuntu-advantage-tools-27.3~21.10.1/debian/ubuntu-advantage-tools.postinst  
2021-09-21 13:02:06.000000000 +0000
[...]
+configure_log_file() {
+    log_path=$1
+    if [ ! -f $log_path ]; then
+        touch $log_path
+    fi
+    chmod 0600 $log_path
+    chown root:root $log_path
+}
[...]
-      if [ ! -f /var/log/ubuntu-advantage.log ]; then
-          touch /var/log/ubuntu-advantage.log
-      fi
-      chmod 0600 /var/log/ubuntu-advantage.log
-      chown root:root /var/log/ubuntu-advantage.log
+      configure_log_file /var/log/ubuntu-advantage.log
+      configure_log_file /var/log/ubuntu-advantage-timer.log
+      configure_log_file /var/log/ubuntu-advantage-license-check.log
+
[...]

It is unusual for maintainer scripts to pre-create log files.  If these
files are removed, do the tools recreate them with the correct
permissions?  (If so: is this code needed at all?)

** Changed in: ubuntu-advantage-tools (Ubuntu Impish)
       Status: In Progress => Incomplete

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1942929

Title:
  [SRU] ubuntu-advantage-tools (27.2.2 -> 27.3) Xenial, Bionic, Focal,
  Hirsute

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/ubuntu-advantage-tools/+bug/1942929/+subscriptions


-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to