[SVN] r694 - in /trunk/cyrus-imapd-2.2.13/debian: changelog cyrus-common-2.2.postrm

debian at incase.de debian at incase.de
Tue Feb 20 17:58:12 CET 2007


Author: sven
Date: Tue Feb 20 17:58:11 2007
New Revision: 694

URL: https://mail.incase.de/viewcvs?rev=694&root=cyrus22&view=rev
Log:
Fix bug #410840 (reported against cyrus21-common) for cyrus-common-2.2 --- Availability of debconf not guaranteed during postrm, despite Depends: debconf

Modified:
    trunk/cyrus-imapd-2.2.13/debian/changelog
    trunk/cyrus-imapd-2.2.13/debian/cyrus-common-2.2.postrm

Modified: trunk/cyrus-imapd-2.2.13/debian/changelog
URL: https://mail.incase.de/viewcvs/trunk/cyrus-imapd-2.2.13/debian/changelog?rev=694&root=cyrus22&r1=693&r2=694&view=diff
==============================================================================
--- trunk/cyrus-imapd-2.2.13/debian/changelog (original)
+++ trunk/cyrus-imapd-2.2.13/debian/changelog Tue Feb 20 17:58:11 2007
@@ -1,3 +1,13 @@
+cyrus-imapd-2.2 (2.2.13-11) UNRELEASED; urgency=low
+
+  * Fix a bug in cyrus-common-2.2 postrm script:
+    Even when a package Depends on debconf, it is not guaranteed to be there
+    when postrm is called.
+    Assume removal of spools is not desired if debconf isn't available (least
+    surprise / least chance of data loss principle).
+
+ -- Sven Mueller <sven at debian.org>  Tue, 20 Feb 2007 17:51:37 +0100
+
 cyrus-imapd-2.2 (2.2.13-10) unstable; urgency=high
 
   * High urgency due to #400747

Modified: trunk/cyrus-imapd-2.2.13/debian/cyrus-common-2.2.postrm
URL: https://mail.incase.de/viewcvs/trunk/cyrus-imapd-2.2.13/debian/cyrus-common-2.2.postrm?rev=694&root=cyrus22&r1=693&r2=694&view=diff
==============================================================================
--- trunk/cyrus-imapd-2.2.13/debian/cyrus-common-2.2.postrm (original)
+++ trunk/cyrus-imapd-2.2.13/debian/cyrus-common-2.2.postrm Tue Feb 20 17:58:11 2007
@@ -9,7 +9,14 @@
 set -e
 
 # debconf hook
-. /usr/share/debconf/confmodule
+if [ -e /usr/share/debconf/confmodule ]; then
+	. /usr/share/debconf/confmodule
+	DEBCONFEXISTS="true"
+	export DEBCONFEXISTS
+else
+	DEBCONFEXISTS="false"
+	export DEBCONFEXISTS
+fi
 
 # summary of how this script can be called:
 #        * <postrm> `remove'
@@ -36,12 +43,16 @@
 			rm usr/lib/cyrus/cyrus-hardwired-config.active || true
 
 			# Should we remove our precious data?
-			db_title "Cyrus IMAPd 2.2" || true
-			db_fset cyrus-common-2.2/removespools seen false || true
-			db_input high cyrus-common-2.2/removespools || true
-			db_go || true
+			if [ "$DEBCONFEXISTS" = "true" ]; then
+				db_title "Cyrus IMAPd 2.2" || true
+				db_fset cyrus-common-2.2/removespools seen false || true
+				db_input high cyrus-common-2.2/removespools || true
+				db_go || true
 
-			db_get cyrus-common-2.2/removespools
+				db_get cyrus-common-2.2/removespools
+			else
+				RET="false"
+			fi
 			[ "${RET}" = "true" ] && {
 				for i in /var/spool/cyrus /var/lib/cyrus \
 				         /var/spool/sieve /etc/sieve ; do




More information about the Pkg-Cyrus-imapd-Debian-devel mailing list