On Fri, Jul 20, 2012 at 9:24 AM, Ben Pfaff <b...@nicira.com> wrote:

> A Debian package is expected to remove all its configuration files (which
> includes all files in /etc) when it is purged, but the
> openvswitch-controller package wasn't doing that.  This fixes the problem.
>
> Debian bug #682187.
> CC: 682...@bugs.debian.org
> Reported-by: Andreas Beckmann <deb...@abeckmann.de>
> Signed-off-by: Ben Pfaff <b...@nicira.com>
> ---
>  debian/automake.mk                   |    1 +
>  debian/openvswitch-controller.postrm |   44
> ++++++++++++++++++++++++++++++++++
>  2 files changed, 45 insertions(+), 0 deletions(-)
>  create mode 100755 debian/openvswitch-controller.postrm
>
> diff --git a/debian/automake.mk b/debian/automake.mk
> index ae82168..b6cb12e 100644
> --- a/debian/automake.mk
> +++ b/debian/automake.mk
> @@ -22,6 +22,7 @@ EXTRA_DIST += \
>         debian/openvswitch-controller.install \
>         debian/openvswitch-controller.manpages \
>         debian/openvswitch-controller.postinst \
> +       debian/openvswitch-controller.postrm \
>         debian/openvswitch-datapath-module-_KVERS_.postinst.modules.in \
>         debian/openvswitch-datapath-dkms.postinst \
>         debian/openvswitch-datapath-dkms.prerm \
> diff --git a/debian/openvswitch-controller.postrm
> b/debian/openvswitch-controller.postrm
> new file mode 100755
> index 0000000..64b7909
> --- /dev/null
> +++ b/debian/openvswitch-controller.postrm
>
@@ -0,0 +1,44 @@
> +#!/bin/sh
> +# postrm script for openvswitch-controller
> +#
> +# see: dh_installdeb(1)
> +
> +set -e
> +
> +# summary of how this script can be called:
> +#        * <postrm> `remove'
> +#        * <postrm> `purge'
> +#        * <old-postrm> `upgrade' <new-version>
> +#        * <new-postrm> `failed-upgrade' <old-version>
> +#        * <new-postrm> `abort-install'
> +#        * <new-postrm> `abort-install' <old-version>
> +#        * <new-postrm> `abort-upgrade' <old-version>
> +#        * <disappearer's-postrm> `disappear' <overwriter>
> +#          <overwriter-version>
> +# for details, see http://www.debian.org/doc/debian-policy/ or
> +# the debian-policy package
> +
> +
> +case "$1" in
> +    purge)
> +        cd /etc/openvswitch-controller
>
if openvswitch-controller failed to install, because openvswitch-pki was
not installed, then the directory above would be empty. And it seems
that dpkg automatically removes empty directories before calling purge
from this script.. So the command above can fail.

> +        rm -f cacert.pem cert.pem privkey.pem req.pem
>
Also it seems that post install script might leave some temporary files
in this directory (if installation failed). Should we attempt to remove them
too?


> +        ;;
> +
> +    remove|upgrade|failed-upgrade|abort-install|abort-upgrade|disappear)
> +        ;;
> +
> +    *)
> +        echo "postrm called with unknown argument \`$1'" >&2
> +        exit 1
> +        ;;
> +esac
> +
> +# dh_installdeb will replace this with shell code automatically
> +# generated by other debhelper scripts.
> +
> +#DEBHELPER#
> +
> +exit 0
> +
> +
> --
> 1.7.2.5
>
> _______________________________________________
> dev mailing list
> dev@openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev
>


git am says:
/home/aatteka/git/openvswitch/.git/rebase-apply/patch:67: new blank line at
EOF.
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to