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

[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: