~ubuntu-branches/ubuntu/saucy/maas/saucy-updates

« back to all changes in this revision

Viewing changes to debian/maas-common.postinst

  • Committer: Package Import Robot
  • Author(s): Andres Rodriguez, Andres Rodriguez, Steve Langasek
  • Date: 2013-03-19 15:38:22 UTC
  • mfrom: (1.2.6)
  • Revision ID: package-import@ubuntu.com-20130319153822-17izmcpy6wjlyfqi
Tags: 1.3+bzr1455+dfsg-0ubuntu1
* New upstream bugfix release.
  - Fixes and returns the 'resource_url' with the 'canonical' url for
    a file that is fetched using the API (LP: #1154142)

[ Andres Rodriguez ]
* debian/control:
  - Change Conflicts/Replaces for Breaks/Replaces.
  - Conflicts on tftpd-hpa and dnsmasq.
  - Do not pre-depends, but Depends on ${misc:Depends} for 'maas'.

[ Steve Langasek ]
* postinst scripts are never called with 'reconfigure' as the script
  argument.  Remove references to this (mythical) invocation.
* always call 'set -e' from maintainer scripts instead of passing 'sh -e'
  as the interpreter, so that scripts will behave correctly when run via
  'sh -x'.
* invoke-rc.d is never allowed to not exist - simplify scripts (and make
  them better policy-compliant) by invoking unconditionally.  (The only
  possible exception is in the postrm, where it's *theoretically* possible
  for invoke-rc.d to be missing if the user has completely stripped
  down their system; that's a fairly unreasonable corner case, but we
  might as well be correct if it ever happens.)
* db_get+db_set is a no-op; don't call db_set to push back a value we just
  got from db_get.
* Omit superfluous calls to 'exit 0' at the end of each script.
* Remove maas-cluster-controller prerm script, which called debconf for no
  reason.
* Don't invoke debconf in the postrm script either, debhelper already does
  this for us.
* Other miscellaneous maintainer script fixes
* debian/maas-common.postinst: call adduser and addgroup unconditionally;
  the tools are already designed to DTRT, we don't need to check for the
  user/group existence before calling them nor should we worry about
  calling them only once on first install.
* debian/maas-common.postrm: delete the maas group, not just the user,
  as the comment in the code implies we should do.

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
#!/bin/sh -e
 
1
#!/bin/sh
 
2
 
 
3
set -e
2
4
 
3
5
add_user_group(){
4
6
        local user="maas"
5
7
        local group="maas"
6
 
        if ! getent group "$group" >/dev/null; then
7
 
                addgroup --quiet --system "$group" || true
8
 
        fi
9
 
        if ! getent passwd "$user" > /dev/null 2>&1; then
10
 
                adduser --quiet \
11
 
                    --system \
12
 
                    --group \
13
 
                    --no-create-home \
14
 
                    "$user" || true
15
 
        fi
 
8
        addgroup --quiet --system "$group" || true
 
9
        adduser --quiet --system --group --no-create-home "$user" || true
16
10
}
17
11
 
18
 
if [ "$1" = "configure" ]; then
19
 
        add_user_group
20
 
fi
 
12
add_user_group
21
13
 
22
14
#DEBHELPER#