lintian: r390 - in trunk: checks debian testset testset/relations/debian

[marc@marcbrockschmidt.de]

Author: he
Date: 2004-12-25 01:58:09 +0100 (Sat, 25 Dec 2004)
New Revision: 390

Modified:
   trunk/checks/fields
   trunk/checks/fields.desc
   trunk/debian/changelog
   trunk/testset/relations/debian/control
   trunk/testset/tags.relations
Log:
checks/fields:
  + [HE] Warn if people use the Bugs field to refer to the Debian BTS, the
    new check is called redundant-bugs-field. (Closes: #245883)     


Modified: trunk/checks/fields
===================================================================
--- trunk/checks/fields	2004-12-23 23:15:53 UTC (rev 389)
+++ trunk/checks/fields	2004-12-25 00:58:09 UTC (rev 390)
@@ -475,6 +475,17 @@
 	tag "redundant-origin-field", "" if $origin =~ /^\s*debian\s*$/i;
 }
 
+#----- Bugs
+
+if (open(FH, "fields/bugs")) {
+	my $bugs = <FH>;
+	close FH;
+
+	unfold('bugs', \$bugs);
+
+	tag "redundant-bugs-field", "" if $bugs =~ m{^\s*debbugs://bugs.debian.org/?\s*$}i;
+}
+
 #----- Field checks (without checking the value)
 
 for my $field (glob("fields/*")) {

Modified: trunk/checks/fields.desc
===================================================================
--- trunk/checks/fields.desc	2004-12-23 23:15:53 UTC (rev 389)
+++ trunk/checks/fields.desc	2004-12-25 00:58:09 UTC (rev 390)
@@ -487,3 +487,9 @@
 Ref: devref 5.10.2.1
 Info: The version number of your source package has a debian revision with
  three parts, which is usually used by binary NMUs.
+
+Tag: redundant-bugs-field
+Type: warning
+Info: You use the Bugs field though the field value is the default 
+ (debbugs://bugs.debian.org/). In this case the field is redundant and
+ should be removed.

Modified: trunk/debian/changelog
===================================================================
--- trunk/debian/changelog	2004-12-23 23:15:53 UTC (rev 389)
+++ trunk/debian/changelog	2004-12-25 00:58:09 UTC (rev 390)
@@ -10,6 +10,8 @@
     + [HE] Warn if the debian revision has three parts, as this is the sign of
       a binary NMU. New check's name is binary-nmu-debian-revision-in-source.
       (Closes: #244830)
+    + [HE] Warn if people use the Bugs field to refer to the Debian BTS, the
+      new check is called redundant-bugs-field. (Closes: #245883)      
   * checks/manpages:
     + [HE] Don't compare the manpage filename extension and the content of
       .TH case-sensitive. Report + patch by Jay Berkenbilt <ejb@ql.org>,

Modified: trunk/testset/relations/debian/control
===================================================================
--- trunk/testset/relations/debian/control	2004-12-23 23:15:53 UTC (rev 389)
+++ trunk/testset/relations/debian/control	2004-12-25 00:58:09 UTC (rev 390)
@@ -6,7 +6,7 @@
 Maintainer: Debian QA Group <packages@qa.debian.org>
 Standards-Version: 3.1.1
 Origin: Debian
-Bugs: foobar
+Bugs: debbugs://bugs.debian.org/
 
 Package: relations
 Architecture: all

Modified: trunk/testset/tags.relations
===================================================================
--- trunk/testset/tags.relations	2004-12-23 23:15:53 UTC (rev 389)
+++ trunk/testset/tags.relations	2004-12-25 00:58:09 UTC (rev 390)
@@ -19,15 +19,19 @@
 I: relations-multiple-libs: unknown-field-in-control origin
 I: relations: unknown-field-in-control bugs
 I: relations: unknown-field-in-control origin
+W: relations binary: description-synopsis-starts-with-a-capital-letter
 W: relations source: ancient-standards-version 3.1.1
 W: relations source: build-depends-without-arch-dep
 W: relations source: redundant-origin-field
 W: relations source: virtual-package-depends-without-real-package-depends build-depends: mail-transport-agent
+W: relations-multiple-libs binary: description-synopsis-starts-with-a-capital-letter
 W: relations-multiple-libs: description-synopsis-might-not-be-phrased-properly
+W: relations-multiple-libs: redundant-bugs-field
 W: relations-multiple-libs: redundant-origin-field
 W: relations: package-has-a-duplicate-relation depends: relations, relations (<< 3)
 W: relations: package-relation-with-self depends: relations
 W: relations: package-relation-with-self depends: relations (<< 3)
 W: relations: package-relation-with-self replaces: relations
+W: relations: redundant-bugs-field
 W: relations: redundant-origin-field
 W: relations: virtual-package-depends-without-real-package-depends depends: mail-transport-agent