On Fri, Mar 21, 2008 at 03:10:28PM +0000, Meike Reichle wrote:
> Hi again,
> 
> please find attached an improved version of the previous patch. This
> time including an upgrade path in preinst.
> 
> Best regards,
> Meike

  The patch is obviously good and addresses real RC issues, though it
does not handle the submitter problems, which was that /var/isns is not
removed on purge. You have to also write a postrm that handles its
removal, something along the lines of:

if test "$1" = "purge"; then
    rm -rf /var/lib/isns
fi

> +     if [ -d /var/isns ]
> +     then
> +             mkdir -p /var/lib/isns/
> +             cp /var/isns/* /var/lib/isns/
> +             rm -rf /var/isns/
> +     fi

  You can just mv /var/insns/ /var/lib with the same result, and
probably a lot more efficiency btw.


-- 
·O·  Pierre Habouzit
··O                                                [EMAIL PROTECTED]
OOO                                                http://www.madism.org

Attachment: pgpcK352bKGpd.pgp
Description: PGP signature

Reply via email to