[SCM] Debian package checker branch, master, updated. 1.24.4-57-gcfdcbc8
The following commit has been merged in the master branch:
commit 31d04848eb6c3bdc7ca0770b89898d4bf9ca12a3
Author: Marc 'HE' Brockschmidt <he@debian.org>
Date: Sun Aug 10 16:37:35 2008 +0200
Review new classification and fix typos
Changed a few bits of the classification in order to make it more consistent,
from the release team point of view. Also includes fixes to minor typos in
descriptions.
diff --git a/checks/binaries.desc b/checks/binaries.desc
index fe769c8..0ac90ec 100644
--- a/checks/binaries.desc
+++ b/checks/binaries.desc
@@ -8,7 +8,7 @@ Info: This script checks binaries and object files for bugs.
Tag: arch-independent-package-contains-binary-or-object
Type: error
-Severity: important
+Severity: serious
Certainty: certain
Info: The package contains a binary or object file but is tagged
Architecture: all.
@@ -29,7 +29,7 @@ Info: The package installs an unstripped binary or object file.
Tag: library-in-debug-or-profile-should-not-be-stripped
Type: error
-Severity: normal
+Severity: important
Certainty: certain
Info: Libraries in <tt>.../lib/debug</tt> or in
<tt>.../lib/profile</tt> usually should not be stripped.
@@ -176,7 +176,7 @@ Info: This appears to be an ELF file but objdump -T doesn't recognize it
Tag: missing-dependency-on-libc
Type: error
-Severity: important
+Severity: serious
Certainty: certain
Ref: policy 8.6
Info: The listed file appears to be linked against the C library, but the
@@ -192,7 +192,7 @@ Info: The listed file appears to be linked against the C library, but the
Tag: missing-dependency-on-perlapi
Type: error
-Severity: important
+Severity: serious
Certainty: certain
Ref: perl-policy 4.4.2
Info: This package includes a *.so file in <tt>/usr/lib/perl5</tt>,
diff --git a/checks/changelog-file.desc b/checks/changelog-file.desc
index 9d77a39..9658418 100644
--- a/checks/changelog-file.desc
+++ b/checks/changelog-file.desc
@@ -115,7 +115,7 @@ Ref: policy 12.7
Tag: debian-changelog-file-is-a-symlink
Type: warning
Severity: normal
-Certainty: possible
+Certainty: certain
Info: The Debian changelog file is a symlink to a file in a different
directory or not found in this package. Please don't do this. It makes
package checking and manipulation unnecessarily difficult. Because it was
@@ -130,7 +130,7 @@ Info: The Debian changelog file is a symlink to a file in a different
Tag: debian-changelog-file-missing-or-wrong-name
Type: error
Severity: serious
-Certainty: possible
+Certainty: certain
Info: Each Debian package (which provides a /usr/share/doc/<i>pkg</i>
directory) must install a Debian changelog file in
/usr/share/doc/<i>pkg</i>/changelog.Debian.gz
@@ -143,7 +143,7 @@ Ref: policy 12.7
Tag: debian-changelog-file-contains-obsolete-user-emacs-settings
Type: warning
Severity: normal
-Certainty: possible
+Certainty: certain
Info: The add-log-mailing-address variable is no longer honored in
debian-changelog-mode, and should not appear in packages' changelog
files. Instead, put something like this in your ~/.emacs:
@@ -247,7 +247,7 @@ Info: The latest entry in NEWS.Debian has an unusual distribution name.
Tag: spelling-error-in-changelog
Type: warning
Severity: normal
-Certainty: possible
+Certainty: certain
Info: Lintian found a spelling error in the latest entry of the Debian
changelog. Lintian has a list of common misspellings that it looks for.
It does not have a dictionary like a spelling checker does.
@@ -267,7 +267,7 @@ Info: Lintian found a spelling error in the latest entry of the
Tag: new-package-should-close-itp-bug
Type: warning
Severity: normal
-Certainty: possible
+Certainty: certain
Info: This package appears to be the first packaging of a new upstream
software package (there is only one changelog entry and the Debian
revision is 1), but it does not close any bugs. The initial upload of a
diff --git a/checks/conffiles.desc b/checks/conffiles.desc
index 2504515..b9fbce0 100644
--- a/checks/conffiles.desc
+++ b/checks/conffiles.desc
@@ -18,11 +18,10 @@ Info: Files below <tt>/usr</tt> may not be marked as conffiles, since
Tag: non-etc-file-marked-as-conffile
Type: warning
Severity: important
-Certainty: possible
+Certainty: certain
Info: A file installed in some other directory than <tt>/etc</tt>
is marked as conffile. A conffile typically implies a configuration file, and
policy mandates such files to be in /etc
-Status: untested
Ref: policy 10.7.2
Tag: relative-conffile
diff --git a/checks/control-file.desc b/checks/control-file.desc
index 9d3d9e3..69c7532 100644
--- a/checks/control-file.desc
+++ b/checks/control-file.desc
@@ -14,7 +14,7 @@ Info: The Debian control file should be valid UTF-8, an encoding of
the Unicode character set.
.
There are many ways to convert a control from an obsoleted encoding
- like ISO-8859-1; you may for example use "iconv" like:
+ like ISO-8859-1; you may for example use "iconv" like this:
.
$ iconv -f ISO-8859-1 -t UTF-8 control > control.new
$ mv control.new control
diff --git a/checks/copyright-file.desc b/checks/copyright-file.desc
index 61f33dd..491d9d2 100644
--- a/checks/copyright-file.desc
+++ b/checks/copyright-file.desc
@@ -311,7 +311,7 @@ Info: Lintian found a spelling error in the copyright file. Lintian has a
Tag: possible-gpl-code-linked-with-openssl
Type: warning
-Severity: important
+Severity: serious
Certainty: wild-guess
Info: This package appears to be covered by the GNU GPL but depends on
the OpenSSL libssl package and does not mention a license exemption or
diff --git a/checks/cruft.desc b/checks/cruft.desc
index 1edea13..22da838 100644
--- a/checks/cruft.desc
+++ b/checks/cruft.desc
@@ -9,7 +9,7 @@ Needs-Info: debfiles, diffstat
Tag: native-package-with-dash-version
Type: warning
Severity: normal
-Certainty: possible
+Certainty: certain
Info: Native packaging should only be used if a piece of software was
written specifically to be turned into a Debian package. In this case,
the version number should not contain a debian revision part.
@@ -21,14 +21,14 @@ Info: Native packaging should only be used if a piece of software was
Tag: documentation-package-not-architecture-independent
Type: warning
Severity: normal
-Certainty: possible
+Certainty: certain
Info: Documentation packages usually shouldn't carry anything that requires
recompiling on various architectures, in order to save space on mirrors.
Tag: debian-files-list-in-source
Type: error
Severity: important
-Certainty: possible
+Certainty: certain
Info: Leaving <tt>debian/files</tt> causes problems for the autobuilders,
since that file will likely include the list of .deb files for another
architecture, which will cause dpkg-buildpackage run by the buildd to fail.
@@ -274,7 +274,7 @@ Info: The Debian diff or native package contains a file ending in
Tag: diff-contains-substvars
Type: warning
Severity: normal
-Certainty: possible
+Certainty: certain
Info: Lintian found a substvars file in the Debian diff for this source
package. The debian/substvars file is usually generated and modified
dynamically by debian/rules targets, in which case it must be removed by
@@ -284,7 +284,7 @@ Ref: policy 4.9
Tag: configure-generated-file-in-source
Type: warning
Severity: normal
-Certainty: possible
+Certainty: certain
Info: Leaving config.cache/status causes autobuilders problems.
config.cache and config.status are produced by GNU autoconf's configure
scripts. If they are left in the source package, autobuilders may pick
@@ -303,7 +303,7 @@ Info: Leaving config.cache/status causes autobuilders problems.
Tag: outdated-autotools-helper-file
Type: error
Severity: important
-Certainty: possible
+Certainty: certain
Info: The referenced file has a time stamp older than year 2004 and the
package does not build-depend on autotools-dev or automake and therefore
apparently does not update it. This usually means that the source package
diff --git a/checks/debconf.desc b/checks/debconf.desc
index b36933f..0f79d32 100644
--- a/checks/debconf.desc
+++ b/checks/debconf.desc
@@ -122,13 +122,13 @@ Info: The long description of one of the templates provided by this package
Tag: config-does-not-load-confmodule
Type: warning
Severity: normal
-Certainty: possible
+Certainty: certain
Info: The config script must load one of the debconf libraries.
Tag: postinst-uses-db-input
Type: warning
Severity: normal
-Certainty: possible
+Certainty: certain
Info: It is generally not a good idea for postinst scripts to use debconf
commands like <tt>db_input</tt>. Typically, they should restrict themselves
to <tt>db_get</tt> to request previously acquired information, and have the
@@ -182,7 +182,7 @@ Info: The debconf `config' script in the package control area must be
Tag: no-debconf-config
Type: warning
-Severity: normal
+Severity: important
Certainty: possible
Info: The package contains a `templates' file in its control area but has no
corresponding `config' script. This is occasionally OK, but is usually an
@@ -218,7 +218,7 @@ Ref: devref 6.5.1, debconf-devel(7)
Tag: malformed-prompt-in-templates
Type: warning
Severity: normal
-Certainty: possible
+Certainty: certain
Info: The short description of a select, multiselect, string and password
debconf template is a prompt and not a title. Avoid question style
prompts ("IP Address?") in favour of "opened" prompts ("IP address:").
@@ -231,7 +231,7 @@ Ref: devref 6.5.4.2
Tag: malformed-title-in-templates
Type: warning
Severity: normal
-Certainty: possible
+Certainty: certain
Info: The short description of a note debconf template should be written
as a title and therefore should not end with a period, question mark,
colon, or semicolon.
@@ -240,7 +240,7 @@ Ref: devref 6.5.4.2.4
Tag: malformed-question-in-templates
Type: warning
Severity: normal
-Certainty: possible
+Certainty: certain
Info: The short description of a boolean debconf template should be
phrased in the form of a question which should be kept short and should
generally end with a question mark. Terse writing style is permitted and
@@ -253,7 +253,7 @@ Ref: devref 6.5.4.2.2
Tag: using-question-in-extended-description-in-templates
Type: warning
Severity: normal
-Certainty: possible
+Certainty: certain
Info: The extended description of a debconf template should never include
a question.
.
@@ -264,7 +264,7 @@ Ref: devref 6.5.3.2
Tag: using-imperative-form-in-templates
Type: warning
Severity: normal
-Certainty: possible
+Certainty: certain
Info: Do not use useless imperative constructions such as "Please choose...",
"Enter...". The interface will make it obvious that the user needs to
choose or enter something.
diff --git a/checks/debhelper.desc b/checks/debhelper.desc
index 5e8410e..0b9ecb1 100644
--- a/checks/debhelper.desc
+++ b/checks/debhelper.desc
@@ -86,7 +86,7 @@ Info: The source package has an empty debian/compat file. This is an error,
Tag: declares-possibly-conflicting-debhelper-compat-versions
Type: error
Severity: important
-Certainty: possible
+Certainty: certain
Ref: debhelper(7)
Info: The source package declares the debhelper compatibility version
both in the <tt>debian/compat</tt> file and in the <tt>debian/rules</tt>
diff --git a/checks/debian-readme.desc b/checks/debian-readme.desc
index 08c93ce..ca1d85e 100644
--- a/checks/debian-readme.desc
+++ b/checks/debian-readme.desc
@@ -40,7 +40,7 @@ Info: The README.Debian file contains an email address (<..@unknown>)
Tag: spelling-error-in-readme-debian
Type: warning
Severity: normal
-Certainty: possible
+Certainty: certain
Info: Lintian found a spelling error in the README.Debian file. Lintian
has a list of common misspellings that it looks for. It does not have a
dictionary like a spelling checker does.
diff --git a/checks/description.desc b/checks/description.desc
index 74f4fd2..842eabb 100644
--- a/checks/description.desc
+++ b/checks/description.desc
@@ -140,7 +140,7 @@ Info: The extended description contains a "Homepage:" pseudo-header
Tag: spelling-error-in-description
Type: warning
Severity: normal
-Certainty: possible
+Certainty: certain
Info: Lintian found a spelling or capitalization error in the package
description. Lintian has a list of common misspellings that it looks
for. It does not have a dictionary like a spelling checker does. It is
diff --git a/checks/fields.desc b/checks/fields.desc
index 3833a73..288e3cb 100644
--- a/checks/fields.desc
+++ b/checks/fields.desc
@@ -89,7 +89,7 @@ Ref: policy 5.6.8
Tag: unknown-architecture
Type: warning
Severity: normal
-Certainty: certain
+Certainty: possible
Info: This package claims to be for an unknown architecture. The
architecture should be one of the values supported by dpkg or one of the
special values "all" or "any". The special value "source" is only used
@@ -115,7 +115,7 @@ Ref: policy 5.6.8
Tag: aspell-package-not-arch-all
Type: warning
Severity: normal
-Certainty: possible
+Certainty: certain
Info: This package appears to be an aspell dictionary package, but it is
not Architecture: all. The binary hashes should be built at install-time
by calling aspell-autobuildhash, so the contents of the package should be
@@ -128,7 +128,6 @@ Severity: serious
Certainty: certain
Info: The package does not have a `Maintainer:' field in its control file.
Ref: policy 5.3
-Tested: empty
Tag: maintainer-name-missing
Type: error
diff --git a/checks/files.desc b/checks/files.desc
index 8947a1a..f183b21 100644
--- a/checks/files.desc
+++ b/checks/files.desc
@@ -10,7 +10,7 @@ Info: This script checks if a binary package conforms to policy
Tag: package-contains-ancient-file
Type: error
Severity: important
-Certainty: possible
+Certainty: certain
Info: Your package contains a file that claims to have been generated
more than 20 years ago. This is most probably an error. Your package
will be rejected by the Debian archive scripts if it contains a file
@@ -292,7 +292,7 @@ Info: The file is tagged SETUID and SETGID. In some cases this is
Tag: non-standard-file-perm
Type: warning
Severity: normal
-Certainty: possible
+Certainty: certain
Info: The file has a mode different from 0644. In some cases this is
intentional, but in other cases this is a bug.
Ref: policy 10.9
@@ -890,7 +890,7 @@ Info: This package installs a file in <tt>/usr/lib/sgml</tt>. This was
Tag: file-name-ends-in-whitespace
Type: warning
Severity: normal
-Certainty: possible
+Certainty: certain
Info: This package installs a file or directory whose name ends in
whitespace. This might be intentional but it's normally a mistake. If
it is intentional, add a lintian override.
@@ -900,8 +900,8 @@ Info: This package installs a file or directory whose name ends in
Tag: package-contains-empty-directory
Type: warning
-Severity: normal
-Certainty: possible
+Severity: wishlist
+Certainty: certain
Info: This package installs an empty directory. This might be intentional
but it's normally a mistake. If it is intentional, add a lintian override.
.
diff --git a/checks/huge-usr-share.desc b/checks/huge-usr-share.desc
index 2ad21a3..98d6020 100644
--- a/checks/huge-usr-share.desc
+++ b/checks/huge-usr-share.desc
@@ -9,7 +9,7 @@ Info: This script checks whether an architecture-dependent package doesn't
Tag: arch-dep-package-has-big-usr-share
Type: info
Severity: wishlist
-Certainty: possible
+Certainty: certain
Info: The package has a significant amount of architecture-independent data in
/usr/share, while it is an architecture-dependent package.
This is wasteful of mirror space and bandwidth, as we then end up with
diff --git a/checks/infofiles.desc b/checks/infofiles.desc
index 80423ae..b1aba78 100644
--- a/checks/infofiles.desc
+++ b/checks/infofiles.desc
@@ -59,7 +59,7 @@ Ref: policy 12.2
Tag: postrm-calls-install-info
Type: warning
Severity: normal
-Certainty: possible
+Certainty: certain
Ref: policy 12.2
Info: The postrm script calls the <tt>install-info</tt> command. Usually,
this command should be called from the <tt>prerm</tt> maintainer script.
@@ -67,7 +67,7 @@ Info: The postrm script calls the <tt>install-info</tt> command. Usually,
Tag: preinst-calls-install-info
Type: error
Severity: important
-Certainty: possible
+Certainty: certain
Ref: policy 12.2
Info: The preinst script calls the <tt>install-info</tt> command. Usually,
this command should be called from the <tt>postinst</tt> maintainer script.
diff --git a/checks/menus.desc b/checks/menus.desc
index 427e416..e61d292 100644
--- a/checks/menus.desc
+++ b/checks/menus.desc
@@ -59,21 +59,21 @@ Ref: menu 5
Tag: preinst-calls-updatemenus
Type: error
Severity: important
-Certainty: possible
+Certainty: certain
Info: The preinst script calls the <tt>update-menus</tt> command. Usually,
this command should be called from the <tt>postinst</tt> maintainer script.
Tag: preinst-calls-installdocs
Type: error
Severity: important
-Certainty: possible
+Certainty: certain
Info: The preinst script calls the <tt>install-docs</tt> command. Usually,
this command should be called from the <tt>postinst</tt> maintainer script.
Tag: prerm-calls-updatemenus
Type: error
Severity: important
-Certainty: possible
+Certainty: certain
Info: The prerm script calls the <tt>update-menus</tt> command. Usually,
this command should be called from the <tt>postrm</tt> maintainer script.
@@ -290,7 +290,7 @@ Tag: doc-base-file-duplicated-field
Type: error
Severity: important
Certainty: certain
-Info: The doc-base control file contains duplicated field.
+Info: The doc-base control file contains a duplicated field.
Tag: doc-base-file-duplicated-format
Type: error
diff --git a/checks/nmu.desc b/checks/nmu.desc
index 5f49b42..6d7808e 100644
--- a/checks/nmu.desc
+++ b/checks/nmu.desc
@@ -67,7 +67,7 @@ Ref: devref 5.11.3
Tag: changelog-should-not-mention-nmu
Type: warning
Severity: normal
-Certainty: possible
+Certainty: certain
Info: The first line of the changelog entry for this package appears to
indicate it is a non-maintainer upload (by including either that string
or the string "NMU" and not saying that it's an acknowledgement), but the
diff --git a/checks/patch-systems.desc b/checks/patch-systems.desc
index eb39d8b..c06e4c7 100644
--- a/checks/patch-systems.desc
+++ b/checks/patch-systems.desc
@@ -9,7 +9,7 @@ Needs-Info: debfiles
Tag: dpatch-build-dep-but-no-patch-list
Type: warning
Severity: normal
-Certainty: possible
+Certainty: certain
Info: Using dpatch requires you to explicitly list all patches you want
to apply in debian/patches/00list. This package build-depends on dpatch,
but does not provide a patch list.
@@ -34,7 +34,7 @@ Info: dpatch files should carry a description of the function of the included
Tag: quilt-build-dep-but-no-series-file
Type: warning
Severity: normal
-Certainty: possible
+Certainty: certain
Info: Using quilt requires you to explicitly list all patches you want
to apply in debian/patches/series. This package build-depends on quilt,
but does not provide a patch list.
diff --git a/checks/version-substvars.desc b/checks/version-substvars.desc
index 0969f84..80d45e7 100644
--- a/checks/version-substvars.desc
+++ b/checks/version-substvars.desc
@@ -17,8 +17,8 @@ Info: The package is not safely binNMUable because an arch:any package
relationship. Please use (= ${binary:Version}) instead.
Tag: not-binnmuable-any-depends-all
-Type: warning
-Severity: normal
+Type: error
+Severity: important
Certainty: certain
Info: The package is not safely binNMUable because an arch:any package
depends on an arch:all package with a (= ${Source-Version}) or
@@ -26,8 +26,8 @@ Info: The package is not safely binNMUable because an arch:any package
instead.
Tag: not-binnmuable-all-depends-any
-Type: warning
-Severity: normal
+Type: error
+Severity: important
Certainty: certain
Info: The package is not safely binNMUable because an arch:all package
depends on an arch:any package with a strict (= ${Source-Version}), or
--
Debian package checker
Reply to: