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

Bug#377654: marked as done (lintian: "uninitialized value" in debconf checks if there is no template description)



Your message dated Mon, 21 Aug 2006 14:18:55 -0700
with message-id <E1GFHAd-0006tU-Gh@spohr.debian.org>
and subject line Bug#377654: fixed in lintian 1.23.23
has caused the attached Bug report to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what I am
talking about this indicates a serious mail system misconfiguration
somewhere.  Please contact me immediately.)

Debian bug tracking system administrator
(administrator, Debian Bugs database)

--- Begin Message ---
Package: lintian
Version: 1.23.22
Severity: minor
Tags: patch

Hi,

In case of an empty description, lintian first return the
no-template-description tag, and then go on with the template checks,
while the description does not exist. For example:

$ lintian dbconfig-common_1.8.17_all.deb
E: dbconfig-common: no-template-description
  dbconfig-common/internal/reconfiguring
Use of uninitialized value in pattern match (m//)
  at /usr/share/lintian/checks/debconf line 235.
Use of uninitialized value in pattern match (m//)
  at /usr/share/lintian/checks/debconf line 240.
Use of uninitialized value in pattern match (m//)
  at /usr/share/lintian/checks/debconf line 251.

The attached patch should fix this problem.

-- 
Thomas Huriaux
--- debconf	2006-07-10 15:28:42.000000000 +0200
+++ debconf.new	2006-07-10 15:33:19.000000000 +0200
@@ -232,6 +232,7 @@
     # Check the description against the best practices in the Developer's
     # Reference, but skip all templates where the short description contains
     # the string "for internal use".
+    if (exists $template->{description}) {
     $template->{description} =~ m/^([^\n]*)\n(.*)$/s;
     my ($short, $extended) = ($1, $2);
     unless (defined $short) {
@@ -293,6 +294,7 @@
 	    }
 	}
     }
+    }
 }
 
 # Check the maintainer scripts.

Attachment: signature.asc
Description: Digital signature


--- End Message ---
--- Begin Message ---
Source: lintian
Source-Version: 1.23.23

We believe that the bug you reported is fixed in the latest version of
lintian, which is due to be installed in the Debian FTP archive:

lintian_1.23.23.dsc
  to pool/main/l/lintian/lintian_1.23.23.dsc
lintian_1.23.23.tar.gz
  to pool/main/l/lintian/lintian_1.23.23.tar.gz
lintian_1.23.23_all.deb
  to pool/main/l/lintian/lintian_1.23.23_all.deb



A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed.  If you
have further comments please address them to 377654@bugs.debian.org,
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Russ Allbery <rra@debian.org> (supplier of updated lintian package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing ftpmaster@debian.org)


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Format: 1.7
Date: Mon, 21 Aug 2006 10:28:14 -0700
Source: lintian
Binary: lintian
Architecture: source all
Version: 1.23.23
Distribution: unstable
Urgency: low
Maintainer: Debian Lintian Maintainers <lintian-maint@debian.org>
Changed-By: Russ Allbery <rra@debian.org>
Description: 
 lintian    - Debian package checker
Closes: 333311 370720 372748 374167 374314 374899 375318 375638 376030 376109 376596 377141 377182 377616 377654 378054 378055 379176 379558 379749 380399 381800 381965 383203
Changes: 
 lintian (1.23.23) unstable; urgency=low
 .
   The "new Python policy" release
 .
   * checks/binaries.desc:
     + [RA] Greatly expand the binary-or-shlib-defines-rpath long
       description based on recent discussion in debian-policy.  Add a
       reference to the wiki page.  (Closes: #378054, #378055)
   * checks/changelog-file{.desc,}:
     + [RA] Diagnose changelog files that are symlinks, using the new way
       the changelog-file collection script processes them.
   * checks/common_data.pm:
     + [RA] Add Python-Version as a known source and binary field.
   * checks/copyright-file:
     + [RA] Catch "<fill in http/ftp site>" as well as the string without
       "http/".  Thanks, Jari Aalto.  (Closes: #374167)
   * checks/deb-format{.desc,}:
     + [RA] Remove deb-created-with-broken-tar; the bug was actually in
       dpkg, and the dpkg in stable has been fixed.  Thanks Jari Aalto and
       Thijs Kinkhorst.  (Closes: #375638, #376030)
   * checks/debconf:
     + [RA] Diagnose a missing template description only once and avoid
       Perl warnings.  Thanks, Thomas Huriaux.  (Closes: #377654)
   * checks/debhelper:
     + [RA] Use the standard control file parser rather than a custom
       parser that can't handle continuation lines.
     + [RA] Warn if dh_python is used without debian/pycompat in non-CDBS
       packages and check for the debhelper version required for Python
       policy.  Thanks, Raphaël Hertzog.  (Closes: #375318)
   * checks/fields{.desc,}:
     + [RA] Don't warn virtual-package-depends-without-real-package-depends
       for Recommends and rewrite the tag description to be more accurate.
       Thanks to Martin F. Krafft for the report.  (Closes: #370720)
     + [RA] ~ is now permitted in version numbers.  (Closes: #381965)
     + [RA] Catch and check dependencies correctly for debconf-updatepo run
       from the clean target.  Thanks, James Westby.  (Closes: #380399)
     + [RA] Warn when encountering an aspell dictionary package that isn't
       arch: all.  Thanks, Piotr Engelking.  (Closes: #374899)
     + [RA] Check the syntax of the new Python-Version field.
     + [RA] Check for missing build dependencies on python-central or
       python-support.  Thanks, Raphaël Hertzog.
     + [RA] Check for old << dependencies on Python in Python packages that
       lack a Python-Version control field, as these are probably packages
       that don't follow the new Python policy.  Thanks, Raphaël Hertzog.
   * checks/files{.desc,}:
     + [RA] Provide a better explanation of why Debian packages shouldn't
       ship files in /srv.  (Closes: #379176)
     + [RA] Warn about files installed into /etc/gconf/schemas.
     + [RA] Warn about files in /usr/lib/site-python.  Thanks, Thomas
       Viehmann and Raphaël Hertzog.  (Closes: #372748)
   * checks/manpages{.desc,}:
     + [RA] Warn about manpage short descriptions of the form "manual page
       for program" (usually generated by help2man).
   * checks/menu:
     + [RA] Ignore trailing whitespace in doc-base files.  Patch from Bas
       Zoetekouw.  (Closes: #379749)
   * checks/nmu{.desc,}:
     + [RA] When checking for an NMU changelog entry, require "NMU" be
       surrounded by word boundaries.
     + [RA] Warn if the changelog incorrectly indicates an NMU.  Thanks to
       George Danchev for the report.
   * checks/scripts{.desc,}:
     + [RA] Except mkstemp from possibly-insecure-handling-of-tmp-files for
       maintainer scripts written in Perl.
     + [RA] Mention [[ ]] explicitly in the possible-bashism long
       description as requiring a Korn shell and make it clearer that the
       primary arbiter of this test is the POSIX shell specification.
       (Closes: #381800 by clarification)
     + [RA] Use the Dep library to check dependencies.  Add php5 to the
       interpreter list and correctly handle the dependencies for
       version-less PHP scripts.  (Closes: #379558)
     + [RA] Check for proper dependencies if gconf-schemas is called in
       maintainer scripts and diagnose calling gconftool(-2) directly.
       Thanks, Josselin Mouette.  (Closes: #333311)
   * checks/version-substvars{.desc,}:
     + [RA] New check from Adeodato Simó for proper use of substvars for
       versioned dependencies between packages built from the same source
       package so that NMUs will work properly.  Also catch use of such a
       substvar for a versioned dependency on a package not built from the
       same source.  (Closes: #377141, #376596)
 .
   * collection/changelog-file:
     + [RA] Be much more careful about changelog files that are symlinks.
       Don't skip over changelog symlinks just because they point to
       non-existant files, mark unsafe symlinks or symlinks to non-existent
       files by creating copyright as a symlink to a non-existent file in
       the lab, and don't create an empty dummy changelog if we couldn't
       find a changelog.  Correctly handle changelog files that are
       symlinks to other files in the same directory.
 .
   * debian/copyright:
     + [RA] Add Subversion repository location.
 .
   * frontend/lintian:
     + [JvW] Give error if there is no "Description" field in .changes, this is
       required by dak
 .
   * frontend/lintian-info:
     + [RA] Add the -t or --tags option that, given a tag name, prints out
       the description.  (Closes: #376109)
 .
   * lib/Tags.pm:
     + [RA] If a tag has been overriden via any mechanism, do not add it to
       any of the statistics (tag, severity, or significance).  Before, the
       override only suppressed a record for the specific statistic the
       override was based on, meaning that classic tag-based overrides
       didn't suppress severity statistics.  Overridden error tags would
       therefore cause lintian to exit with non-zero status.  Thanks,
       Andree Leidenfrost and Martin F. Krafft.  (Closes: #377182, #374314)
 .
   * man/lintian.1:
     + [RA] Better document the --archivedir and --dist options and how
       lintian constructs the path to an archive.  (Closes: #383203)
     + [RA] Document new version-substvars check script.
   * man/lintian-info.1:
     + [RA] Document the new -t or --tags option.
 .
   * reporting/html_reports:
     + [RA] Include the lintian version in HTML page footers.  Thanks,
       Thijs Kinkhorst.  (Closes: #377616)
Files: 
 da8f32e61a502e9799c3c4ff44685904 803 devel optional lintian_1.23.23.dsc
 b180a3d7af42209a6a36be62ccd38f8d 307986 devel optional lintian_1.23.23.tar.gz
 7098ad61fa1d7283e50b8d2c56db9fda 264248 devel optional lintian_1.23.23_all.deb

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.5 (GNU/Linux)

iD8DBQFE6hIS+YXjQAr8dHYRArRZAJ4sOrVbFGtZ/3nXHbiJOf6OIxsaOwCaAp/H
ZbRw4ZGNgjRqlLfKHEGXu4g=
=Fori
-----END PGP SIGNATURE-----


--- End Message ---

Reply to: