Package: yada
Version: 0.49
Severity: serious
Tags: etch-ignore patch

Hi,

The postrm files that yada creates unconditionally source the debconf
confmodule:

        . /usr/share/debconf/confmodule
        db_version 2.0 || [ $? -lt 30 ]

Policy stats that packages cannot require the presence of any
non-essential packages in the postrm/purge phase. This means that yada
creates policy violating packages.

I've understood from recent statements of the release team that failing
to purge when debconf is not available is not RC for etch, hence the
etch-ignore tag. However, it would of course make sense to fix yada
quickly so that any package built with yada is as policy compliant as
possible.

The attached patch fixes the issue. It doesn't add pretty whitespace,
that may be something you might want to add to it to complete it.


Thijs
diff -Nru /tmp/vic5eorRM6/yada-0.49/debian/changelog /tmp/AGMh1FzQ0r/yada-0.49/debian/changelog
--- /tmp/vic5eorRM6/yada-0.49/debian/changelog	2006-11-09 13:21:08.000000000 +0100
+++ /tmp/AGMh1FzQ0r/yada-0.49/debian/changelog	2006-11-15 23:29:37.000000000 +0100
@@ -1,3 +1,10 @@
+yada (0.49-0.1) unstable; urgency=high
+
+  * Non-maintainer upload.
+  * Only conditionally include debconf confmodule in postrm.
+
+ -- Thijs Kinkhorst <[EMAIL PROTECTED]>  Wed, 15 Nov 2006 23:10:41 +0100
+
 yada (0.49) unstable; urgency=low
 
   * debian/packages:
diff -Nru /tmp/vic5eorRM6/yada-0.49/debian/yada /tmp/AGMh1FzQ0r/yada-0.49/debian/yada
--- /tmp/vic5eorRM6/yada-0.49/debian/yada	2006-11-09 13:21:08.000000000 +0100
+++ /tmp/AGMh1FzQ0r/yada-0.49/debian/yada	2006-11-15 23:18:25.000000000 +0100
@@ -3329,9 +3329,15 @@
                     if (($_ eq 'postinst' or $_ eq 'postrm' or defined $$par{$_})
                         and defined $$par{'config'})
                     {
+                        if ( $_ eq 'postrm' ) {
+                            print MAINT "\nif [ -f /usr/share/debconf/confmodule ]; then" or die;
+                        }
                         print MAINT "\n. /usr/share/debconf/confmodule\n"
                           . "db_version 2.0 || [ \$? -lt 30 ]\n"
                           or die;
+                        if ( $_ eq 'postrm' ) {
+                            print MAINT "fi\n" or die;
+                        }
                     }
                     if ($_ eq 'preinst' and defined $$par{'diversions'}) {
                         $alt = $$par{'diversions'};

Attachment: signature.asc
Description: This is a digitally signed message part

Reply via email to