--- Begin Message ---
- To: Debian Bug Tracking System <submit@bugs.debian.org>
- Subject: lintian: False positive in debconf template check
- From: Rafael Laboissiere <rafael@debian.org>
- Date: Tue, 27 Nov 2007 20:03:50 +0100
- Message-id: <20071127190350.GG24209@localhost>
Package: lintian
Version: 1.23.36
Severity: normal
The check making-assumptions-about-interfaces-in-templates gives a false
positive for the following debconf template of the webcalendar package:
--<snip>---------------------------------------------------------------------
Template: webcalendar/conf/single_user_mode
Type: boolean
Default: false
_Description: Should WebCalendar be installed in single-user mode?
WebCalendar can be installed in single-user mode or multi-user mode. If it
is installed in single-user mode, no login will be required.
.
Using the single-user mode is not recommended unless the software runs
on a personal server protected by a firewall.
--</snip>--------------------------------------------------------------------
Looking at the following piece of code in /usr/share/lintian/checks/debconf:
--<snip>---------------------------------------------------------------------
if ($template->{description} =~ /[ \'\"](yes|no)[ \'\",;.]/i
and $type eq 'boolean') {
tag "making-assumptions-about-interfaces-in-templates",
$template->{template};
}
--</snip>--------------------------------------------------------------------
I think the warning was triggered by the "no login" phrase in the template.
Could you please fine tune the test to avoid this false positive?
Thanks,
Rafael
-- System Information:
Debian Release: 4.0
APT prefers testing
APT policy: (500, 'testing'), (500, 'stable')
Architecture: amd64 (x86_64)
Shell: /bin/sh linked to /bin/bash
Kernel: Linux 2.6.18-4-amd64
Locale: LANG=en_US, LC_CTYPE=en_US (charmap=UTF-8) (ignored: LC_ALL set to en_US.UTF-8)
Versions of packages lintian depends on:
ii binutils 2.18~cvs20070812-1 The GNU assembler, linker and bina
ii diffstat 1.43-2 produces graph of changes introduc
ii dpkg-dev 1.13.25 package building tools for Debian
ii file 4.17-5etch1 Determines file type using "magic"
ii gettext 0.16.1-1 GNU Internationalization utilities
ii intltool-debian 0.35.0+20060710.1 Help i18n of RFC822 compliant conf
ii libparse-debianchange 1.0-1 parse Debian changelogs and output
ii man-db 2.4.3-6 The on-line manual pager
ii perl [libdigest-md5-p 5.8.8-7 Larry Wall's Practical Extraction
lintian recommends no packages.
-- no debconf information
--- End Message ---
--- Begin Message ---
Source: lintian
Source-Version: 1.23.39
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.39.dsc
to pool/main/l/lintian/lintian_1.23.39.dsc
lintian_1.23.39.tar.gz
to pool/main/l/lintian/lintian_1.23.39.tar.gz
lintian_1.23.39_all.deb
to pool/main/l/lintian/lintian_1.23.39_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 453177@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: Fri, 07 Dec 2007 22:12:56 -0800
Source: lintian
Binary: lintian
Architecture: source all
Version: 1.23.39
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: 36017 356051 435963 448783 453177 454358 454688 454723 454730 454790
Changes:
lintian (1.23.39) unstable; urgency=low
.
The "Ubuntu and doc-base patch merge, with extra fixes" release.
.
* checks/*.desc:
+ [RA] Remove the unused Standards-Version header.
* checks/changelog-file{.desc,}:
+ [RA] Check the latest entry of the Debian changelog and any
NEWS.Debian file for common spelling errors. (Closes: #36017)
+ [RA] If this looks like a new package (Debian revision of -1 and
only one changelog entry), warn if it doesn't close a bug. Thanks,
Margarita Manterola. (Closes: #356051)
+ [RA] Check for lines over 80 columns in the most recent entry.
Thanks, Guillem Jover. (Closes: #435963)
* checks/copyright-file{.desc,}:
+ [RA] Moved spelling-error-in-copyright check to here.
* checks/debconf:
+ [RA] Go back to not warning about "no" in boolean debconf
questions. The word is too common in normal English prose for
reasons other than assuming a particular debconf interface. Thanks,
Rafael Laboissiere. (Closes: #453177)
* checks/debian-readme{.desc,}:
+ [RA] Moved spelling-error-in-readme-debian check to here.
* checks/description{.desc,}:
+ [RA] Moved spelling-error-in-description check to here.
* checks/fields:
+ [RA] Python documentation packages should still be in section doc.
Thanks, Michal Čihař. (Closes: #454688)
+ [RA] Warn about lib.*-dev packages not in section libdevel.
+ [RA] Warn about debug packages that aren't priority: extra. Thanks,
Joerg Jaspert. (Closes: #454358)
+ [RA] Ignore Original-Maintainer if the version contains ubuntu.
+ [RA] Only warn about Section for Python packages starting with
python-, not py, since py picks up too many things that aren't
Python modules.
+ [RA] Only warn about Section for Perl packages matching lib.*-perl
to avoid false positives for things like dh-make-perl. Thanks,
Damyan Ivanov. (Closes: #454723)
* checks/files:
+ [RA] Warn about packages providing files in /usr/lib/debug that
aren't named -dbg. Thanks, Joerg Jaspert.
* checks/lintian.desc:
+ [RA] Add bad-ubuntu-distribution-in-changes-file, merged from the
Ubuntu patch.
+ [RA] Fix the malformed-override long description. Thanks, Stefan
Fritsch.
* checks/menus{.desc,}:
+ [RA] Substantial overhaul and expansion of the doc-base control file
checks. Patch from Robert Luberda. (Closes: #448783)
* checks/nmu:
+ [RA] No packages with ubuntu in the version number are NMUs. Merged
from the Ubuntu patch.
* checks/patch-systems:
+ [RA] Ignore blank lines in 00list and don't report them as patches
without descriptions. Thanks, Julien BLACHE. (Closes: #454730)
* checks/spelling{.desc,}:
+ [RA] Subsumed into other check scripts and lib/Spelling.pm.
.
* frontend/lintian:
+ [RA] If the version number indicates an Ubuntu package, check
against a different list of allowable distributions. Merged from
the Ubuntu patch.
+ [RA] Skip check and collection *.desc files whose names start with a
period (mostly to avoid testing artifacts from editor lock files).
+ [RA] Restore previous override parsing and make the package name
optional again. Thanks, Stefan Fritsch. (Closes: #454790)
+ [RA] Check overrides for implausible tags.
.
* lib/Spelling.pm:
+ [RA] New module to do general spelling checks for specific
misspellings. Based on the previous checks/spelling and a patch by
Robert Luberda.
Files:
721e5584c1bbb5fe0115d8f14b2e6d6d 904 devel optional lintian_1.23.39.dsc
60c7fafc093656f1da0b2533896e82a2 362802 devel optional lintian_1.23.39.tar.gz
07ca14cf6ddc073163bf7eb74df4cb4b 306822 devel optional lintian_1.23.39_all.deb
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)
iD8DBQFHWjip+YXjQAr8dHYRAjVtAJ9Nkax/MkPdLDaPTuEzztjNESLZEQCfasIK
+zPl8GEi7OWape5uGN+U9xo=
=ENZp
-----END PGP SIGNATURE-----
--- End Message ---