Re: Bug#459403: libuuid1: missing depends on non-essential package passwd
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
Reply to: