Hello!
Attached is the patch that seems to be suggested as the solution, based on
fw's comments. I'll upload this shortly...
-Kees
--
Kees Cook @debian.org
diff -u debfoster-2.7/debian/postrm debfoster-2.7/debian/postrm
--- debfoster-2.7/debian/postrm
+++ debfoster-2.7/debian/postrm
@@ -1,38 +1,9 @@
#!/bin/sh
-OLDKEEPERS="/etc/apt/keepers"
-KEEPERS="/var/lib/debfoster/keepers"
-
if [ "$1" = "purge" ] ; then
- if [ -f $OLDKEEPERS ] ; then
- echo "debfoster's list of wanted packages still exists. Do you"
- echo -n "want to remove this file? ($OLDKEEPERS) [Y/n] "
- read answer
- case $answer in
- y|Y|Yes|YES|yes|"")
- rm -f $OLDKEEPERS
- ;;
- *)
- echo "Not removing $OLDKEEPERS."
- ;;
- esac
- fi
- if [ -f $KEEPERS ] ; then
- echo "debfoster's list of wanted packages still exists. Do you"
- echo -n "want to remove this file? ($KEEPERS) [Y/n] "
- read answer
- case $answer in
- y|Y|Yes|YES|yes|"")
- rm -f $KEEPERS
- ;;
- *)
- echo "Not removing $KEEPERS."
- ;;
- esac
- fi
+ rm -f "/var/lib/debfoster/keepers"
fi
-
# generated by other debhelper scripts.
#DEBHELPER#
reverted:
--- debfoster-2.7/debian/postinst
+++ debfoster-2.7.orig/debian/postinst
@@ -1,64 +0,0 @@
-#! /bin/sh
-# postinst script for #PACKAGE#
-#
-# see: dh_installdeb(1)
-
-set -e
-
-# summary of how this script can be called:
-# * <postinst> `configure' <most-recently-configured-version>
-# * <old-postinst> `abort-upgrade' <new version>
-# * <conflictor's-postinst> `abort-remove' `in-favour' <package>
-# <new-version>
-# * <deconfigured's-postinst> `abort-deconfigure' `in-favour'
-# <failed-install-package> <version> `removing'
-# <conflicting-package> <version>
-# for details, see /usr/share/doc/packaging-manual/
-#
-# quoting from the policy:
-# Any necessary prompting should almost always be confined to the
-# post-installation script, and should be protected with a conditional
-# so that unnecessary prompting doesn't happen if a package's
-# installation fails and the `postinst' is called with `abort-upgrade',
-# `abort-remove' or `abort-deconfigure'.
-
-OLDKEEPERS="/etc/apt/keepers"
-NEWKEEPERS="/var/lib/debfoster/keepers"
-
-case "$1" in
- configure)
- if [ -f $OLDKEEPERS -a \! -f $NEWKEEPERS ] ; then
- echo "$OLDKEEPERS exists, do you want me to move it"
- echo -n "to its new location, $NEWKEEPERS? [Y/n] "
- read ans
- case $ans in
- Y|y|Yes|yes|YES|"")
- mv -fv $OLDKEEPERS $NEWKEEPERS
- ;;
- *)
- echo "You don't want to move the file. Fine, but you will
be asked"
- echo "again which packages you want to keep when you run
debfoster."
- echo "Or you could move $OLDKEEPERS to $NEWKEEPERS by hand."
- ;;
- esac
- fi
- ;;
-
- abort-upgrade|abort-remove|abort-deconfigure)
-
- ;;
-
- *)
- echo "postinst called with unknown argument \`$1'" >&2
- exit 0
- ;;
-esac
-
-# dh_installdeb will replace this with shell code automatically
-# generated by other debhelper scripts.
-
-#DEBHELPER#
-
-exit 0
-
-
diff -u debfoster-2.7/debian/changelog debfoster-2.7/debian/changelog
--- debfoster-2.7/debian/changelog
+++ debfoster-2.7/debian/changelog
@@ -1,3 +1,12 @@
+debfoster (2.7-1.1) unstable; urgency=low
+
+ * Non-maintainer upload.
+ * debian/{postinst,postrm}: remove policy-violating prompts for old
+ keepers file location that has not existed for 9 years now, and
+ unconditionally remove new keepers on purge (Closes: 553286).
+
+ -- Kees Cook <k...@debian.org> Sun, 24 Jan 2010 17:09:13 -0800
+
debfoster (2.7-1) unstable; urgency=low
* New upstream version. Closes: #448501.
