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

[lintian] 01/01: Revert "When checking latest-debian-changelog-entry-without-new-version ignore any change of epoch. (Closes: #889991)"



This is an automated email from the git hooks/post-receive script.

lamby pushed a commit to branch master
in repository lintian.

commit 9b3e5b29380db6e5d422649a1717859bf9bf3e92
Author: Chris Lamb <lamby@debian.org>
Date:   Fri Feb 9 22:00:58 2018 +0000

    Revert "When checking latest-debian-changelog-entry-without-new-version ignore any change of epoch. (Closes: #889991)"
    
    This does not consider the case when the source package name changes.
    
    This reverts commit 536856d8206127d4db07c2ad218f33311cee6340.
---
 checks/changelog-file.desc                                | 3 ---
 checks/changelog-file.pm                                  | 6 ++----
 debian/changelog                                          | 3 ---
 t/tests/changelog-file-general/debian/debian/changelog.in | 3 +--
 4 files changed, 3 insertions(+), 12 deletions(-)

diff --git a/checks/changelog-file.desc b/checks/changelog-file.desc
index c37457a..9843234 100644
--- a/checks/changelog-file.desc
+++ b/checks/changelog-file.desc
@@ -210,9 +210,6 @@ Severity: normal
 Certainty: certain
 Info: The latest Debian changelog entry has a version number that's either
  the same or smaller than the version number of the entry before.
- .
- All versions used in a package's history must be unique, even with the
- leading epoch stripped off.
 
 Tag: latest-debian-changelog-entry-changed-to-native
 Severity: normal
diff --git a/checks/changelog-file.pm b/checks/changelog-file.pm
index b26663b..fda5d8f 100644
--- a/checks/changelog-file.pm
+++ b/checks/changelog-file.pm
@@ -318,10 +318,8 @@ sub run {
             my $second_version = $entries[1]->Version;
             if ($first_version and $second_version) {
                 tag 'latest-debian-changelog-entry-without-new-version'
-                  unless versions_gt(
-                    $first_version =~ s/^([^:]+)://r,
-                    $second_version =~ s/^([^:]+)://r
-                  ) or $entries[0]->Changes =~ /backport/i;
+                  unless versions_gt($first_version, $second_version)
+                  or $entries[0]->Changes =~ /backport/i;
                 tag 'latest-debian-changelog-entry-changed-to-native'
                   if $native_pkg and $second_version =~ m/-/;
             }
diff --git a/debian/changelog b/debian/changelog
index b4e69db..ac125ce 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -2,9 +2,6 @@ lintian (2.5.75) UNRELEASED; urgency=medium
 
   XXX: generate tag summary
 
-  * checks/changelog-file.desc:
-    + [CL] When checking latest-debian-changelog-entry-without-new-version
-      ignore any change of epoch.  (Closes: #889991)
   * checks/patch-systems.{desc,pm}:
     + [CL] Avoid false positives when checking for typos by ignoring files
       or patch descriptions that contain the words "typo" or "spelling".
diff --git a/t/tests/changelog-file-general/debian/debian/changelog.in b/t/tests/changelog-file-general/debian/debian/changelog.in
index d828c35..8b6e177 100644
--- a/t/tests/changelog-file-general/debian/debian/changelog.in
+++ b/t/tests/changelog-file-general/debian/debian/changelog.in
@@ -1,4 +1,4 @@
-{$source} (1:{$version}) unstable; urgency=low
+{$source} ({$version}) unstable; urgency=low
 
 
   * Test: {$testname}
@@ -21,7 +21,6 @@
     üüüüüüüüüüüüüüüüüüüüüüüüüüüüüüüüüüüüüüüüüüüüüüüüüüüüüüüüüüüüüüüüüüüüüüüüüüü
 
   * It might even fix TEMP-1234567-abcdef, though probably not.
-  * We refer to the epoch bump here.
 
  -- {$author}  {$date}
 

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/lintian/lintian.git


Reply to: