Package: postgrey
Severity: wishlist
Tags: patch

Hi,

I was doing some Q&A on Debian system users. postgrey's postinst script
has a syntax like:
> getent passwd postgrey > /dev/null || adduser

With this syntax, it is much more difficult to extract the list of user
accounts created by the pacakges.
I wish you could a simpler syntax, like:
> if ! getent passwd postgrey > /dev/null ; then
>         adduser ...
> fi

It really is priority wish list, feel free to close the bug if you
"wont-fix".

Thanks,

Franklin

-- System Information:
Debian Release: squeeze/sid
  APT prefers testing
  APT policy: (500, 'testing'), (101, 'unstable'), (10, 'experimental')
Architecture: i386 (i686)

Kernel: Linux 2.6.32-2-686 (SMP w/2 CPU cores)
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
--- postinst~   2010-03-15 00:04:29.000000000 +0100
+++ postinst    2010-03-15 00:06:17.000000000 +0100
@@ -10,12 +10,14 @@
     configure)
         DBDIR='/var/lib/postgrey'
 
-        getent passwd postgrey > /dev/null || \
+        if ! getent passwd postgrey > /dev/null ; then
             adduser --system --home "$DBDIR" --no-create-home \
                     --disabled-password --group postgrey
+       fi
         # migration from old 'nogroup' postgrey (1.23-1 and earlier):
-        getent group postgrey > /dev/null || \
+        if ! getent group postgrey > /dev/null ; then
             addgroup --system postgrey
+       fi
 
         install -d -o postgrey -g postgrey -m 0700 "$DBDIR"
 

Reply via email to