tag 568741 +patch pending
tag 607622 +patch pending
thanks

On Fri, Dec 17, 2010 at 06:33:55PM +0100, Helmut Grohne wrote:
> notfound 568741 20091006-4.1
> thanks
> 
> > I just upgraded dictionaries-common and it presented me the message
> > about ingerman being broken from my initial report again. Was this bug
> > ever fixed? Is there something wrong about my system?
> 
> Sorry for the noise. I should have read the complete bug log before
> reopening. As Agustin Martin explained the package leaves a corrupt
> /var/lib/ispell/ingerman.compat file which is not yet deleted by
> preinst, however the package does no longer create the corrupt file, so
> it is no longer buggy.

However, the problem is still present for every ingerman where
reconfiguration was tried previously to the upgrade. Thus, I am leaving this
bug report as RC since it can break upgrades, thanks for reminding.

Furthermore, I noticed that same original problem exists for iswiss. I am 
cloning
this bug report to iswiss.

Attached patch ahould take care from preinst that bogus 'ingerman.compat'
and 'iswiss.compat' are removed. iswiss.config is also fixed for wrong name.

I think this should really care of all the upgrades for both iswiss and
ingerman.

I have a NMU ready for upload if needed. 

Roland, can you look at this shortly or I directly NMU?

-- 
Agustin
diff -wu igerman98-20091006/debian/changelog igerman98-20091006/debian/changelog
--- igerman98-20091006/debian/changelog
+++ igerman98-20091006/debian/changelog
@@ -1,3 +1,14 @@
+igerman98 (20091006-4.2) unstable; urgency=low
+
+  * Non-maintainer upload.
+  * Fix problems caused by bogus compat files (Closes: #568741, #607622):
+    - ingerman.preinst: remove bogus ingerman.compat if present.
+    - iswiss.preinst: remove bogus iswiss.compat if present.
+    - iswiss.config.in: Touch swiss.compat on reconfigure, not
+      iswiss.compat.
+
+ -- Agustin Martin Domingo <agmar...@debian.org>  Mon, 20 Dec 2010 12:12:10 +0100
+
 igerman98 (20091006-4.1) unstable; urgency=low
 
   * Non-maintainer upload.
diff -wu igerman98-20091006/debian/iswiss.config.in igerman98-20091006/debian/iswiss.config.in
--- igerman98-20091006/debian/iswiss.config.in
+++ igerman98-20091006/debian/iswiss.config.in
@@ -3,7 +3,7 @@
 #DEBHELPER#
 
 if ( $ARGV[0] eq "reconfigure" ){
-    my $flag_file = "/var/lib/ispell/iswiss.compat";
+    my $flag_file = "/var/lib/ispell/swiss.compat";
     open (REC_FLAG,"> $flag_file") 
 	or die "Could not open $flag_file for writing\n";
     print REC_FLAG "0";
only in patch2:
--- igerman98-20091006.orig/debian/ingerman.preinst
+++ igerman98-20091006/debian/ingerman.preinst
@@ -0,0 +1,12 @@
+#!/bin/sh
+
+set -e
+
+bogus_compat="/var/lib/ispell/ingerman.compat"
+
+if [ -f "${bogus_compat}" ] ; then
+    echo "Removing bogus compat file ${bogus_compat}" >&2
+    rm "${bogus_compat}"
+fi
+
+#DEBHELPER#
only in patch2:
--- igerman98-20091006.orig/debian/iswiss.preinst
+++ igerman98-20091006/debian/iswiss.preinst
@@ -0,0 +1,12 @@
+#!/bin/sh
+
+set -e
+
+bogus_compat="/var/lib/ispell/iswiss.compat"
+
+if [ -f "${bogus_compat}" ] ; then
+    echo "Removing bogus compat file ${bogus_compat}" >&2
+    rm "${bogus_compat}"
+fi
+
+#DEBHELPER#

Reply via email to