Re: Bug#459403: libuuid1: missing depends on non-essential package passwd

[Theodore Tso <tytso@mit.edu>]

On Sun, Jan 06, 2008 at 10:37:03AM +0100, Julien Cristau wrote:
> 
> the libuuid1 postinst contains:
> groupadd -f -K GID_MIN=1 -K GID_MAX=999 libuuid
> if ! grep -q libuuid /etc/passwd; then
>    useradd -d /var/lib/libuuid -K UID_MIN=1 -K UID_MAX=499 -g libuuid libuuid
> fi
> mkdir -p /var/lib/libuuid
> chown libuuid:libuuid /var/lib/libuuid
> chmod 2775 /var/lib/libuuid
> 
> The groupadd and useradd commands come from passwd, which is not
> Essential: yes, so a Depends is needed.
> 
> Moreover, the postinst succeeds even if any of these commands fail,
> because 'set -e' is missing at the top of the script.

So e2fsprogs which is "Essential: yes" depends on libuuid1, so
libuuid1 is effectively "Essential: yes", right?  So if I add a
dependency on passwd, it will effectively make passwd "Essential:
yes", as well, and according to policy I should bring it up for
comment on debian-policy.  So, I am doing so now.  Any objections if I
add a dependency on passwd for libuuid1?  The aternative would be to
roll-my-own useradd/adduser functionality, but that would be a real
PITA....

Thanks, regards,

							- Ted