package caudium
reopen 336383
thanks

On Sat, Oct 29, 2005 at 07:48:16PM -0700, Debian Bug Tracking System wrote:
> #336383: caudium: please remove files created in postinst,
>    * please remove files created in postinst (Closes: #336383).
>      The changes:
>       - try to remove the dirs in /usr/local/share, if they're empty

You placed the two rmdirs in prerm, but they should be in postrm.
If they are in prerm they are always executed when the package is
upgraded.

>       - correctly handle the /usr/lib/caudium/caudium-images case

You surrounded the rm -r /usr/lib/caudium/caudium-images/auto by
        if [ -d /usr/lib/caudium/caudium-images -o -L 
/usr/lib/caudium/caudium-images ]; then
Unfortunately, this is exactly the same as before, because
/usr/lib/caudium/caudium-images is always either a directory or a
symlink, where the latter case was the wrong one :)

in prerm:
        if [ -d /usr/lib/caudium/caudium-images ]; then
          if [ -d /usr/lib/caudium/caudium-images/auto ]; then
            rm -r /usr/lib/caudium/caudium-images/auto
          fi
        fi
and in postrm:
        rm -r /usr/lib/caudium/caudium-images || true
        test -d /usr/lib/caudium && rmdir /usr/lib/caudium
should do well.

Also, removing the pidfile /var/run/caudium/caudium.pid before caudium
is actually stopped (it will be stopped below), is probably not a good
idea.


regards
   Mario
-- 
*axiom* welcher sensorische input bewirkte die output-aktion,
        den irc-chatter mit dem nick "dus" des irc-servers
        mittels eines kills zu verweisen?

Attachment: signature.asc
Description: Digital signature

Reply via email to