Package: nanourl
Version: 0.1-7.1
Severity: serious
User: debian...@lists.debian.org
Usertags: piuparts

Hi,

the postinst script of your package looks plainly wrong:

==>  rm -f /etc/nanourl/*
this needs to be done during purge only and never when the package is
just removed
if /etc/nanourl/apache.conf is the only file you need to remove, you
won't even need extra code for this as thats a conffile and removed by
dpkg.

==> rm -rf /var/lib/mysql/nanoDB
WTF? You don't delete databases that way. And you would never issue a
DROP DATABASE in postrm remove, only during purge.

http://www.debian.org/doc/debian-policy/ch-maintainerscripts.html


And I don't even dare to take a look at the postinst script ...


piuparts caught this this during a test
  install nanourl
  remove nanourl (don't purge!)
  install nanourl
Now /etc/nanourl/apache.conf is missing.


Andreas


-- 
To UNSUBSCRIBE, email to debian-bugs-rc-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to