I prepared another patch for this bug. IMHO, the patch from Prach
Pongpanich has a few shortcomings I tried to fix in mine.

I made a few tests, and the patched package seems to behave The Right
Way(tm).

Thanks, Giovanni.
-- 
Giovanni Mascellani <mascell...@poisson.phc.unipi.it>
Pisa, Italy

Web: http://poisson.phc.unipi.it/~mascellani
Jabber: g.mascell...@jabber.org / giova...@elabor.homelinux.org
diff -Nru debsums-2.0.48/debian/changelog debsums-2.0.48+nmu1/debian/changelog
--- debsums-2.0.48/debian/changelog	2009-12-20 23:01:26.000000000 +0100
+++ debsums-2.0.48+nmu1/debian/changelog	2010-05-10 23:35:28.000000000 +0200
@@ -1,3 +1,11 @@
+debsums (2.0.48+nmu1) unstable; urgency=low
+
+  * Non-maintainer upload.
+  * Ask before overwriting /etc/apt.conf.d/90debsums with ucf (closes: #57116).
+    - add ucf in Depends and Build-Depends.
+
+ -- Giovanni Mascellani <mascell...@poisson.phc.unipi.it>  Mon, 10 May 2010 23:35:14 +0200
+
 debsums (2.0.48) unstable; urgency=low
 
   * reword the --ignore-permissions documentation to say that it makes
diff -Nru debsums-2.0.48/debian/control debsums-2.0.48+nmu1/debian/control
--- debsums-2.0.48/debian/control	2009-12-20 23:01:26.000000000 +0100
+++ debsums-2.0.48+nmu1/debian/control	2010-05-10 22:57:06.000000000 +0200
@@ -9,7 +9,7 @@
 
 Package: debsums
 Architecture: all
-Depends: perl (>= 5.8.0-3), ${misc:Depends}
+Depends: perl (>= 5.8.0-3), ucf (>= 0.28), ${misc:Depends}
 Description: tool for verification of installed package files against MD5 checksums
  debsums can verify the integrity of installed package files against
  MD5 checksums installed by the package, or generated from a .deb
diff -Nru debsums-2.0.48/debian/postinst debsums-2.0.48+nmu1/debian/postinst
--- debsums-2.0.48/debian/postinst	2009-12-20 23:01:26.000000000 +0100
+++ debsums-2.0.48+nmu1/debian/postinst	2010-05-10 23:38:16.000000000 +0200
@@ -17,15 +17,18 @@
 . /usr/share/debconf/confmodule
 
 APT_CONF=/etc/apt/apt.conf.d/90debsums
-
 case "$1" in
     configure)
         db_get debsums/apt-autogen || true
-        if [ "$RET" = true ]; then
-            echo 'DPkg::Post-Invoke { "if [ -x /usr/bin/debsums ]; then /usr/bin/debsums --generate=nocheck -sp /var/cache/apt/archives; fi"; };' >$APT_CONF
-        else
-            rm -f $APT_CONF
+        APT_CONF_TMP=`tempfile`
+	chmod 644 $APT_CONF_TMP
+        if [ "$RET" != true ]; then
+            echo -n '//' > $APT_CONF_TMP
         fi
+        echo 'DPkg::Post-Invoke { "if [ -x /usr/bin/debsums ]; then /usr/bin/debsums --generate=nocheck -sp /var/cache/apt/archives; fi"; };' >> $APT_CONF_TMP
+        ucf --debconf-ok $APT_CONF_TMP $APT_CONF
+        rm -f $APT_CONF_TMP
+        ucfr debsums $APT_CONF
     ;;
 
     abort-upgrade|abort-remove|abort-deconfigure)
diff -Nru debsums-2.0.48/debian/postrm debsums-2.0.48+nmu1/debian/postrm
--- debsums-2.0.48/debian/postrm	2009-12-20 23:01:26.000000000 +0100
+++ debsums-2.0.48+nmu1/debian/postrm	2010-05-10 23:44:41.000000000 +0200
@@ -24,7 +24,16 @@
     ;;
 
     purge)
-        [ -f $APT_CONF ] && rm -f $APT_CONF
+        for ext in '' '~' '%' .bak .ucf-new .ucf-old .ucf-dist; do
+          rm -f $APT_CONF$ext
+        done
+	if which ucf >/dev/null; then
+	  ucf --purge $APT_CONF
+	fi
+	if which ucfr >/dev/null; then
+	  ucfr --purge debsums $APT_CONF
+	fi
+	
     ;;
 
     *)

Attachment: signature.asc
Description: OpenPGP digital signature

Reply via email to