Bug#947125: buster-pu: package cyrus-imapd/3.0.8-6+deb10u4
Control: tags -1 + confirmed On Mon, 2019-12-23 at 18:55 +0100, Xavier wrote: > Le 22/12/2019 à 17:32, Adam D. Barratt a écrit : [...] > > The patch also adds "CONVERSATIONS|SEARCH_INDEXED|SORTCACHE" to > > another > > clause, which was the bit I was querying. > > Sorry, all values set in cyrus-db-types.txt are now included in this > script to avoid another failure Thanks. It might be worth being explicit about that in the changelog. Please go ahead. Regards, Adam
Bug#947125: buster-pu: package cyrus-imapd/3.0.8-6+deb10u4
Le 22/12/2019 à 17:32, Adam D. Barratt a écrit : > On Sat, 2019-12-21 at 19:44 +0100, Xavier wrote: >> Le 21/12/2019 à 16:18, Adam D. Barratt a écrit : >>> Control: tags -1 + moreinfo >>> >>> On Sat, 2019-12-21 at 14:43 +0100, Xavier Guimard wrote: > [...] >>> +cyrus-imapd (3.0.8-6+deb10u4) buster; urgency=medium >>> + >>> + * Add BACKUP type to cyrus-upgrade-db (Closes: #930764) >>> [...] >>> - DUPLICATE|PTS|STATUSCACHE|TLS|ZONEINFO|SEEN) >>> + DUPLICATE|PTS|STATUSCACHE|TLS|ZONEINFO|SEEN|CONVERSATIO >>> NS|S >>> EARCH_INDEXED|SORTCACHE) >>> >>> What's that change for? (As none of the mentioned items is >>> "BACKUP".) >> [...] >> $ dpkg --configure --pending >> Setting up cyrus-common (3.0.8-6) ... >> Creating/updating cyrus user account... >> The user `cyrus' is already a member of `sasl'. >> /usr/lib/cyrus/bin/upgrade-db: Unknown type of DB: BACKUP >> dpkg: error processing package cyrus-common (--configure): >>installed cyrus-common package post-installation script subprocess >> returned error exit status 1 >> >> BACKUP is listed in /usr/lib/cyrus/cyrus-db-types.txt and then >> required for upgrade > > Thanks. I'd got that far on my own. :-) > > The patch also adds "CONVERSATIONS|SEARCH_INDEXED|SORTCACHE" to another > clause, which was the bit I was querying. Sorry, all values set in cyrus-db-types.txt are now included in this script to avoid another failure
Bug#947125: buster-pu: package cyrus-imapd/3.0.8-6+deb10u4
On Sat, 2019-12-21 at 19:44 +0100, Xavier wrote: > Le 21/12/2019 à 16:18, Adam D. Barratt a écrit : > > Control: tags -1 + moreinfo > > > > On Sat, 2019-12-21 at 14:43 +0100, Xavier Guimard wrote: [...] > > +cyrus-imapd (3.0.8-6+deb10u4) buster; urgency=medium > > + > > + * Add BACKUP type to cyrus-upgrade-db (Closes: #930764) > > [...] > > - DUPLICATE|PTS|STATUSCACHE|TLS|ZONEINFO|SEEN) > > + DUPLICATE|PTS|STATUSCACHE|TLS|ZONEINFO|SEEN|CONVERSATIO > > NS|S > > EARCH_INDEXED|SORTCACHE) > > > > What's that change for? (As none of the mentioned items is > > "BACKUP".) > [...] > $ dpkg --configure --pending > Setting up cyrus-common (3.0.8-6) ... > Creating/updating cyrus user account... > The user `cyrus' is already a member of `sasl'. > /usr/lib/cyrus/bin/upgrade-db: Unknown type of DB: BACKUP > dpkg: error processing package cyrus-common (--configure): >installed cyrus-common package post-installation script subprocess > returned error exit status 1 > > BACKUP is listed in /usr/lib/cyrus/cyrus-db-types.txt and then > required for upgrade Thanks. I'd got that far on my own. :-) The patch also adds "CONVERSATIONS|SEARCH_INDEXED|SORTCACHE" to another clause, which was the bit I was querying. Regards, Adam
Bug#947125: buster-pu: package cyrus-imapd/3.0.8-6+deb10u4
Le 21/12/2019 à 16:18, Adam D. Barratt a écrit : > Control: tags -1 + moreinfo > > On Sat, 2019-12-21 at 14:43 +0100, Xavier Guimard wrote: >> cyrus-imapd has a RC bug. This little patch tested by reporters fixes >> the problem. > > Fun timing, given +deb10u3 only just went through security... :-p > > +cyrus-imapd (3.0.8-6+deb10u4) buster; urgency=medium > + > + * Add BACKUP type to cyrus-upgrade-db (Closes: #930764) > [...] > - DUPLICATE|PTS|STATUSCACHE|TLS|ZONEINFO|SEEN) > + DUPLICATE|PTS|STATUSCACHE|TLS|ZONEINFO|SEEN|CONVERSATIONS|S > EARCH_INDEXED|SORTCACHE) > > What's that change for? (As none of the mentioned items is "BACKUP".) Hi, sorry: * I took cyrus-imapd just few month ago and haven't seen that bug (recently upgraded as "grave"). I took maintenance because some RC bugs risked to expel cyrus-imapd from Buster * I didn't test this patch but reported does (https://bugs.debian.org/930764) * I missed to add link to the RC bug in this BTS The bug is: $ dpkg --configure --pending Setting up cyrus-common (3.0.8-6) ... Creating/updating cyrus user account... The user `cyrus' is already a member of `sasl'. /usr/lib/cyrus/bin/upgrade-db: Unknown type of DB: BACKUP dpkg: error processing package cyrus-common (--configure): installed cyrus-common package post-installation script subprocess returned error exit status 1 BACKUP is listed in /usr/lib/cyrus/cyrus-db-types.txt and then required for upgrade Cheers, Xavier
Bug#947125: buster-pu: package cyrus-imapd/3.0.8-6+deb10u4
Control: tags -1 + moreinfo On Sat, 2019-12-21 at 14:43 +0100, Xavier Guimard wrote: > cyrus-imapd has a RC bug. This little patch tested by reporters fixes > the problem. Fun timing, given +deb10u3 only just went through security... :-p +cyrus-imapd (3.0.8-6+deb10u4) buster; urgency=medium + + * Add BACKUP type to cyrus-upgrade-db (Closes: #930764) [...] - DUPLICATE|PTS|STATUSCACHE|TLS|ZONEINFO|SEEN) + DUPLICATE|PTS|STATUSCACHE|TLS|ZONEINFO|SEEN|CONVERSATIONS|S EARCH_INDEXED|SORTCACHE) What's that change for? (As none of the mentioned items is "BACKUP".) Regards, Adam
Bug#947125: buster-pu: package cyrus-imapd/3.0.8-6+deb10u4
Package: release.debian.org Severity: normal Tags: buster User: release.debian@packages.debian.org Usertags: pu Hi, cyrus-imapd has a RC bug. This little patch tested by reporters fixes the problem. Cheers, Xavier diff --git a/debian/changelog b/debian/changelog index 391230f..c96adf9 100644 --- a/debian/changelog +++ b/debian/changelog @@ -1,3 +1,9 @@ +cyrus-imapd (3.0.8-6+deb10u4) buster; urgency=medium + + * Add BACKUP type to cyrus-upgrade-db (Closes: #930764) + + -- Xavier Guimard Sat, 21 Dec 2019 14:39:58 +0100 + cyrus-imapd (3.0.8-6+deb10u3) buster-security; urgency=medium * Add patch to avoid mailbox creation as administrator diff --git a/debian/cyrus-upgrade-db b/debian/cyrus-upgrade-db index d7037b2..c50a4ce 100755 --- a/debian/cyrus-upgrade-db +++ b/debian/cyrus-upgrade-db @@ -62,6 +62,7 @@ upgradealldb() { DO_UPGRADE_DB= if [ "${NEW_DBVALUE}" != "${OLD_DBVALUE}" ] ; then + echo "Upgrading $OLD_DBKEY from $OLD_DBVALUE to $NEW_DBVALUE ..." DO_UPGRADE_DB=yes fi @@ -82,11 +83,14 @@ upgradealldb() { return 1 fi ;; - DUPLICATE|PTS|STATUSCACHE|TLS|ZONEINFO|SEEN) + DUPLICATE|PTS|STATUSCACHE|TLS|ZONEINFO|SEEN|CONVERSATIONS|SEARCH_INDEXED|SORTCACHE) ;; USERDENY) DBFILE=user_deny.db ;; + BACKUP) + DBFILE=backups.db + ;; *) echo "$0: Unknown type of DB: $OLD_DBKEY" 1>&2 return 1