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

Bug#1006347: lintian: Tag ‘improbable-bug-number-in-closes’ description inaccurate



Package: lintian
Version: 2.111.0
Severity: minor
Tags: patch

The description for Lintian tag ‘improbable-bug-number-in-closes’
currently reads:

    The most recent changelog closes a bug numbered less than 2000.
    While this is distantly possible, it's more likely a typo or a
    placeholder value that mistakenly wasn't filled in.

That implies the tag will not match any bug number 2000 or higher.

That description is inaccurate and misleading. The condition currently
matches any bug number < 50004 (so, higher than the 2000 described);
and matches any bug number > 1000000 (the description does not mention
any upper limit).

I suggest this change to the description, to accurately represent the
condition:

-----
modified   checks/changelog-file.desc
@@ -268,9 +268,10 @@ Ref: devref 6.3.4
 Tag: improbable-bug-number-in-closes
 Severity: normal
 Certainty: possible
-Info: The most recent changelog closes a bug numbered less than 2000.
- While this is distantly possible, it's more likely a typo or a
- placeholder value that mistakenly wasn't filled in.
+Info: The most recent changelog closes a bug report with a number that
+ is very low or very high. While this is distantly possible, it's more
+ likely a typo, or a placeholder value that was mistakenly left
+ unchanged.
 
 Tag: wrong-bug-number-in-closes
 Severity: normal
-----

An improvement might be to have the description include the current
values from the data file ‘data/changelog-file/bugs-number’, but I
don't know whether Lintian tag descriptions can be dynamically
populated that way.

-- 
 \            “Technology is neither good nor bad; nor is it neutral.” |
  `\                       —Melvin Kranzberg's First Law of Technology |
_o__)                                                                  |
Ben Finney <bignose@debian.org>

Attachment: signature.asc
Description: PGP signature


Reply to: