Re: Re: RFS: tenshi (updated)

[Hideki Yamane <henrich@debian.or.jp>]

Hi,

 tenshi-0.11-2 looks good for me, but I'll suggest you to check it
 with piuparts.

>0m18.2s DUMP:
>  (Reading database ... 10298 files and directories currently installed.)
>  Removing tenshi ...
>  Purging configuration files for tenshi ...
>  Removing tenshi system user../var/lib/dpkg/info/tenshi.postrm: 22: deluser: not found
>  .done

 deluser from adduser package, and adduser package is purged before removing
 tenshi. So, userdel should be used in this way, and I think return true
 anytime is not good habit, it sometimes hides problems.


diff -u tenshi-0.11/debian/postrm tenshi-0.11/debian/postrm
--- tenshi-0.11/debian/postrm
+++ tenshi-0.11/debian/postrm
@@ -8,7 +8,7 @@
          rm -rf /var/lib/tenshi ||true
          if getent passwd tenshi 2>&1 >/dev/null; then
              echo -n "Removing tenshi system user.."
-             deluser --quiet tenshi ||true
+             userdel tenshi
              echo ".done"
          fi