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

[SCM] Debian package checker branch, master, updated. 1.24.4-84-g6601a34



The following commit has been merged in the master branch:
commit 6601a349a547e37879005047505c8fabbae82998
Author: Adam D. Barratt <adam@adam-barratt.org.uk>
Date:   Mon Sep 1 23:15:13 2008 +0100

    Strip trailing whitespace from template names before checking if they are used
    
    Avoid flagging a template as unused if its definition is followed by a string
    of whitespace.
    
    The problem was seen "in the wild" in the make-ssl-cert package which declares
    a template as:
    
      "Template: make-ssl-cert/vulnerable_prng "

diff --git a/checks/debconf b/checks/debconf
index 43731ba..1ca45c3 100644
--- a/checks/debconf
+++ b/checks/debconf
@@ -468,6 +468,8 @@ for my $file (qw(config prerm postrm preinst postinst)) {
 }
 
 foreach my $template (@templates_seen) {
+    $template =~ s/\s+\Z//;
+
     my $used = 0;
 
     if ($templates_used{$template}) {
diff --git a/debian/changelog b/debian/changelog
index 4a7f26d..3652f6b 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -11,6 +11,9 @@ lintian (2.0.0~rc1) experimental; urgency=low
   * checks/changelog-file{,.desc}:
     + [RA] Warn about closing a bug number less than 100; this is likely
       a typo or placeholder.  Thanks, Raphael Gessiert.  (Closes: #497350)
+  * checks/debconf:
+    + [ADB] Avoid flagging a template as unused if its definition is
+      followed by a string of whitespace.
   * checks/debhelper{,.desc}:
     + [FL] Do not require versioned build-depends for compat level 5
       anymore.  Sarge is not supported anymore.  (Closes: #496826)

-- 
Debian package checker


Reply to: