usr/lib/ubuntu-mate/ubuntu-mate-settings-overlay:
* Not a very big fan of this script, especially as you are using a fixed 
filename. Please rather user mktemp to randomize the name and use a pattern.
* chown -R ${USER}: "${USER_DIR}" -> why do you need to chown as the user, I 
hope this script isn't used at root? (when is it executed exactly?)


Just a note (not a blocker), we tend to use with rm_conffiles and such 
<new_version_without_conffiles>~. That enables if you backported to ppa 
previous version or if there is a SRU (not applicable in this case) to ensure 
that even if new packages on older series were releases, those would get their 
conffiles cleaned up as well.

** Changed in: ubuntu-mate-settings (Ubuntu)
       Status: New => Incomplete

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

Title:
  ubuntu-mate-settings package needs updating

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu-mate/+bug/1427182/+subscriptions

-- 
ubuntu-bugs mailing list
[email protected]
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to