[SCM] Debian package checker branch, master, updated. 2.2.6-78-g6f56cdb
The following commit has been merged in the master branch:
commit 4a9ab7734d536fd643b659a95054654b12b8e347
Author: Russ Allbery <rra@debian.org>
Date: Sun Mar 8 20:37:54 2009 -0700
Add changelog for gz-file-not-gzip and tweak the check
diff --git a/checks/changelog-file b/checks/changelog-file
index a658fb9..8dba7b6 100644
--- a/checks/changelog-file
+++ b/checks/changelog-file
@@ -117,8 +117,8 @@ foreach (sort keys %{$info->index}) {
$max_compressed = 1;
}
}
- if (not $max_compressed) {
- unless ($is_a_symlink{$file1} or $file_info{$file1} !~ m/gzip compressed/) {
+ if (not $max_compressed and $file_info{$file1} !~ m/gzip compressed/) {
+ unless ($is_a_symlink{$file1}) {
tag "changelog-not-compressed-with-max-compression", "$file";
}
}
diff --git a/checks/files.desc b/checks/files.desc
index 6c3390f..e8ecc83 100644
--- a/checks/files.desc
+++ b/checks/files.desc
@@ -970,6 +970,8 @@ Info: The icon has a size that differs from the size specified by the name
Tag: gz-file-not-gzip
Severity: normal
Certainty: possible
-Info: The given file ends with .gz which suggests it is compressed with gzip
- but it doesn't appear to actually be one. gzip will fail with an error on
- such cases and should therefor be avoided or fixed, as necessary.
+Info: The given file ends with <tt>.gz</tt>, which normally indicates it
+ is compressed with gzip. However, it doesn't seem to be a
+ gzip-compressed file. gzip will fail with an error on such files.
+ Normally this indicates a mistake in the installation process of the
+ package.
diff --git a/debian/changelog b/debian/changelog
index 97c2233..0c1c130 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -7,6 +7,7 @@ lintian (2.2.7) UNRELEASED; urgency=low
- debian-control-has-unusual-field-spacing (pedantic)
- diversion-for-unknown-file
- embedded-zlib
+ - gz-file-not-gzip
- incorrect-libdir-in-la-file
- orphaned-diversion
- package-uses-local-diversion
@@ -31,6 +32,8 @@ lintian (2.2.7) UNRELEASED; urgency=low
* checks/changelog-file:
+ [RA] Avoid capturing parentheses in regexes due to their minor
performance penalty. Patch from Raphael Geissert.
+ + [RA] Correctly handle changelog.gz files that aren't compressed with
+ gzip. Patch from Raphael Geissert.
* checks/common_data.pm:
+ [RA] Move all information used only in one check script from here to
the relevant check script. Patch from Raphael Geissert.
@@ -67,7 +70,7 @@ lintian (2.2.7) UNRELEASED; urgency=low
other logic to assume that. Thanks, Gonéri Le Bouder.
* [RA] Move data from common_data.pm to here and use Lintian::Data to
get the list of known architectures. Patch from Raphael Geissert.
- * checks/files:
+ * checks/files{,.desc}:
+ [RA] Avoid capturing parentheses in regexes due to their minor
performance penalty. Patch from Raphael Geissert.
+ [RA] Precompile constant regexes with qr and only load the list of
@@ -77,6 +80,8 @@ lintian (2.2.7) UNRELEASED; urgency=low
+ [ADB] Use Lintian::Relation instead of Dep.
+ [RA] magpierss was renamed to libphp-magpierss. Patch from Marcelo
Jorge Vieira (metal). (Closes: #518684)
+ + [RA] Diagnose files ending in *.gz that aren't compressed with
+ gzip. Patch from Raphael Geissert. (Closes: #518873)
* checks/huge-usr-share:
+ [RA] Simplify architecture checking code and be robust against
whitespace in the field.
--
Debian package checker
Reply to: