[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#696065: pu: package openldap/2.4.23-7.3



On Sun, Dec 16, 2012 at 02:03:16PM +0000, Adam D. Barratt wrote:
> On Sun, 2012-12-16 at 14:54 +0100, Wouter Verhelst wrote:
> > I've uploaded an NMU of openldap to DELAYED/7, targetted at stable, to
> > deal with #665199 (slapd in unstable can't read the data files from
> > slapd in stable, because the libdb version changed).
> 
> Even with a delay, this should still really have been discussed /before/
> the upload occurred.

Sorry 'bout that.

> > The fix is to dump
> > the database in prerm (when upgrading) rather than preinst, so that the
> > slapcat version which can actually read the database is actually still
> > on the system. I've tested this in a chroot by installing the package
> > and upgrading it to the version in wheezy, and that was successful.
> 
> Please could you attach a debdiff to this bug; thanks.

I've sent it to the original bug, but here it is again:

diff -u openldap-2.4.23/debian/slapd.preinst openldap-2.4.23/debian/slapd.preinst
--- openldap-2.4.23/debian/slapd.preinst
+++ openldap-2.4.23/debian/slapd.preinst
@@ -4,17 +4,6 @@
 
 . /usr/share/debconf/confmodule
 
-# This will be replaced with debian/slapd.scripts-common which includes
-# various helper functions and $OLD_VERSION and $SLAPD_CONF
-#SCRIPTSCOMMON#
-
-# If we are upgrading from an old version then stop slapd and attempt to
-# slapcat out the data so we can use it in postinst to do the upgrade
-
-if [ "$MODE" = upgrade ]; then
-	dump_databases
-fi
-
 #DEBHELPER#
 
 exit 0
diff -u openldap-2.4.23/debian/slapd.prerm openldap-2.4.23/debian/slapd.prerm
--- openldap-2.4.23/debian/slapd.prerm
+++ openldap-2.4.23/debian/slapd.prerm
@@ -4,6 +4,17 @@
 
 . /usr/share/debconf/confmodule
 
+# This will be replaced with debian/slapd.scripts-common which includes
+# various helper functions and $OLD_VERSION and $SLAPD_CONF
+#SCRIPTSCOMMON#
+
+# If we are upgrading from an old version then stop slapd and attempt to
+# slapcat out the data so we can use it in postinst to do the upgrade
+
+if [ "$MODE" = upgrade ]; then
+	dump_databases
+fi
+
 #DEBHELPER#
 
 exit 0
diff -u openldap-2.4.23/debian/changelog openldap-2.4.23/debian/changelog
--- openldap-2.4.23/debian/changelog
+++ openldap-2.4.23/debian/changelog
@@ -1,3 +1,10 @@
+openldap (2.4.23-7.3) stable; urgency=low
+
+  * Non-maintainer upload targeted at stable
+  * Dump the database in prerm if we're upgrading. Closes: #665199
+
+ -- Wouter Verhelst <wouter@debian.org>  Sun, 16 Dec 2012 12:44:59 +0100
+
 openldap (2.4.23-7.2) stable; urgency=low
 
   * Non-maintainer upload targeted at stable.

-- 
Copyshops should do vouchers. So that next time some bureaucracy requires you
to mail a form in triplicate, you can mail it just once, add a voucher, and
save on postage.

Attachment: signature.asc
Description: Digital signature


Reply to: