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

[SCM] Debian package checker branch, master, updated. 2.5.2-112-gb2e645d

The following commit has been merged in the master branch:
commit b2e645dc9909429f64771d6f413bcdde8d505d94
Author: Jakub Wilk <jwilk@debian.org>
Date:   Wed Sep 7 13:21:14 2011 +0200

    Interpret #nnnnnn-notation in checks/*.desc as a ref to a debbug
    
    Signed-off-by: Niels Thykier <niels@thykier.net>

diff --git a/checks/fields.desc b/checks/fields.desc
index 6c580cd..87f1fe2 100644
--- a/checks/fields.desc
+++ b/checks/fields.desc
@@ -90,7 +90,7 @@ Info: The special architecture value "any" only makes sense if it occurs
  alone or (in a *.dsc file) together with "all".  The value "all" may
  appear together with other architectures in a *.dsc file but must
  occur alone if used in a binary package.
-Ref: policy 5.6.8, http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=626775
+Ref: policy 5.6.8, #626775
 
 Tag: unknown-architecture
 Severity: normal
@@ -769,8 +769,7 @@ Info: The package seems to conflict with one of its dependencies,
 Tag: breaks-without-version
 Severity: normal
 Certainty: possible
-Ref: policy 7.3, policy 7.4,
- http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=605744
+Ref: policy 7.3, policy 7.4, #605744
 Info: This package declares a Breaks relationship with another package
  that has no version number.  Normally, Breaks should be used to indicate
  an incompatibility with a specific version of another package, or with
@@ -1044,7 +1043,7 @@ Info: The package appear to be a Java library and depending on one
  If there is otherwise a valid reason for this dependency, please override
  the tag.
 Ref: http://packages.qa.debian.org/j/java-common/news/20100405T221415Z.html,
- http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=227587
+ #227587
 
 Tag: documentation-package-not-architecture-independent
 Severity: normal
diff --git a/checks/files.desc b/checks/files.desc
index 1331a5d..c2713b0 100644
--- a/checks/files.desc
+++ b/checks/files.desc
@@ -248,8 +248,7 @@ Severity: serious
 Certainty: certain
 Info: Files in /etc/sudoers.d/ must be 0440 or sudo will refuse to
  parse them.
-Ref: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=588831,
-     http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=576527
+Ref: #588831, #576527
 
 Tag: special-file
 Severity: serious
@@ -760,7 +759,7 @@ Info: The directory /usr/lib/site-python has been deprecated as a
 Tag: python-module-in-wrong-location
 Severity: normal
 Certainty: possible
-Ref: python-policy 1.5, http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=576012
+Ref: python-policy 1.5, #576012
 Info: The package installs a Python module or debug information for a Python
  module in the wrong location for the given version of Python.
  .
@@ -769,7 +768,7 @@ Info: The package installs a Python module or debug information for a Python
 Tag: python-debug-in-wrong-location
 Severity: normal
 Certainty: possible
-Ref: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=576014
+Ref: #576014
 Info: The package appears to be installing debug modules in
  /usr/lib/debug/usr/lib/pyshared/pythonX.Y/.  However, gdb(1)
  will not look for it there, making it less useful.  The file
@@ -779,7 +778,7 @@ Info: The package appears to be installing debug modules in
 Tag: missing-dependency-on-python-central
 Severity: important
 Certainty: possible
-Ref: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=592533
+Ref: #592533
 Info: The package installs a file with the package name in
  /usr/share/pyshared-data/ without depending on python-central (&gt;= 0.6).
  .
@@ -1184,7 +1183,7 @@ Info: The given, apparently compressed, file is shipped in the package
 Tag: udev-rule-in-etc
 Severity: important
 Certainty: certain
-Ref: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=559208
+Ref: #559208
 Info: This package ships a udev rule and installs it under
  <tt>/etc/udev/rules.d</tt>, which is reserved for user-installed files.
  The correct directory for system rules is <tt>/lib/udev/rules.d</tt>.
diff --git a/checks/rules.desc b/checks/rules.desc
index 36df500..99f73de 100644
--- a/checks/rules.desc
+++ b/checks/rules.desc
@@ -203,7 +203,7 @@ Info: The package appears to use an <tt>ExtUtils::MakeMaker</tt>
 Tag: debian-rules-uses-or-modifies-user-only-variable
 Severity: normal
 Certainty: possible
-Ref: http://bugs.debian.org/631786
+Ref: #631786
 Info: The rules files appear to be reading or modifying a variable not
  intended for use by package maintainers.  The special variables
  .
diff --git a/checks/shared-libs.desc b/checks/shared-libs.desc
index ba2785c..f6dacac 100644
--- a/checks/shared-libs.desc
+++ b/checks/shared-libs.desc
@@ -138,8 +138,7 @@ Info: Although the package does not include any shared libraries, it does
  .
  Note this is sometimes triggered for packages with a private shared
  library due to a bug in Debhelper.
-Ref: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=204975,
- http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=633853
+Ref: #204975, #633853
 
 Tag: duplicate-entry-in-shlibs-control-file
 Severity: important
@@ -199,7 +198,7 @@ Info: The postinst script calls ldconfig even though no shared libraries are
  .
  Note this may be triggered by a bug in debhelper, that causes it to
  auto-generate an ldconfig snippet for packages that does not need it.
-Ref: policy 8.1.1, http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=204975
+Ref: policy 8.1.1, #204975
 
 Tag: udeb-postinst-must-not-call-ldconfig
 Severity: important
@@ -215,7 +214,7 @@ Info: The postrm script calls ldconfig even though no shared libraries are
  .
  Note this may be triggered by a bug in debhelper, that causes it to
  auto-generate an ldconfig snippet for packages that does not need it.
-Ref: policy 8.1.1, http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=204975
+Ref: policy 8.1.1, #204975
 
 Tag: postinst-must-call-ldconfig
 Severity: serious
diff --git a/debian/changelog b/debian/changelog
index 25c37a7..5f1f781 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -38,6 +38,7 @@ lintian (2.5.3) UNRELEASED; urgency=low
       dctrl parsers (notably grep-dctrl).  (Closes: #639928)
     + [JW] Fix a number of broken references in the tag
       descriptions.  (Closes: #639923)
+    + [JW] Rewrote Debian Bug references using #nnnnnn notation.
   * checks/binaries.desc:
     + [NT] Remove the reference to the Library Packaging Guide,
       since it is no longer available (in Debian).  (Closes: #640228)
@@ -208,6 +209,7 @@ lintian (2.5.3) UNRELEASED; urgency=low
   * lib/Lintian/Tag/Info.pm:
     + [JW] Create links for references pointing to files in html
       output.
+    + [JW] Parse "#nnnnnn" references as links to Debian bugs.
   * lib/Lintian/Tag/Override.pm:
     + [NT] New file.
   * lib/Lintian/Tags.pm:
diff --git a/lib/Lintian/Tag/Info.pm b/lib/Lintian/Tag/Info.pm
index e64b357..d9fe512 100644
--- a/lib/Lintian/Tag/Info.pm
+++ b/lib/Lintian/Tag/Info.pm
@@ -248,6 +248,9 @@ sub _format_reference {
             $text = qq(<a href="$ref">$ref</a>);
         } elsif ($ref =~ m,^/,) {
             $text = qq(<a href="file://$ref">$ref</a>);
+        } elsif ($ref =~ m,^#(\d+)$,) {
+            my $url = qq(http://bugs.debian.org/$1);
+            $text = qq(<a href="$url">$url</a>);
         }
         push (@refs, $text) if $text;
     }

-- 
Debian package checker
Reply to: