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

[SCM] Debian package checker branch, master, updated. 2.5.12-25-g1457b7f



The following commit has been merged in the master branch:
commit 1457b7f8f64f93db449dd3afe4408849f6dbcae5
Author: Niels Thykier <niels@thykier.net>
Date:   Sat Apr 20 21:23:46 2013 +0200

    c/fields.desc: Clarify some vcs-* related tags
    
    Clarify that vcs-field-not-canonical is based on Alioth giving 301
    HTTP status codes on "non-canonical" URIs.  Also mention that the tag
    (and vcs-field-bitrotted) are based on "data lists" rather than
    actually sending HTTP requests.
      In theory, the latter should be redundant given Lintian's design
    criteria, but it is probably best to assume that users will not read
    the (full) documentation.
    
    Signed-off-by: Niels Thykier <niels@thykier.net>

diff --git a/checks/fields.desc b/checks/fields.desc
index 41b2119..ffcd621 100644
--- a/checks/fields.desc
+++ b/checks/fields.desc
@@ -1009,12 +1009,27 @@ Info: The VCS-* field contains an uncanonical URI. Please update to use
  the current canonical URI instead. This reduces the network bandwidth used
  and makes debcheckout work independent of the port forwarding and
  redirections properly working.
+ .
+ The definition of canoical used here is based the HTTP status code.
+ For non-canonical URIs, Alioth returned <tt>301 Moved
+ Permanently</tt>.  The new URI provided by Alioth (in the
+ <tt>Location</tt>-field) was then considered "canonical".
+ .
+ Note that this check is based on a list of known URIs.  Lintian did not
+ send a HTTP request to the URI to test this.  Should the canonical URI
+ have changed, please file a bug against Lintian so the check can be
+ updated accordingly.
 
 Tag: vcs-field-bitrotted
 Severity: normal
 Certainty: certain
 Info: The VCS-* field uses an address which no longer works.  Please
  update it to use the current canonical URI instead.
+ .
+ Note that this check is based on a list of known URIs.  Lintian did not
+ send a HTTP request to the URI to test this.  Should the URI actually
+ work, please file a bug against Lintian so the check can be updated
+ accordingly.
 
 Tag: vcs-git-uses-invalid-user-uri
 Severity: normal
diff --git a/debian/changelog b/debian/changelog
index 35caec0..52565b8 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -7,6 +7,14 @@ lintian (2.5.13) UNRELEASED; urgency=low
     + [NT] Recognise "<dist>-backports-sloppy" as a valid
       distribution.  Thanks to Romain Francoise for the
       report and the patch.  (Closes: #705170)
+  * checks/fields.desc:
+    + [NT] Clarify that the "canonical URI" for Vcs fields is
+      based on HTTP 301 responds codes from Alioth.  Thanks to
+      Torquil Sørensen for the report.  (Closes: #705835)
+    + [NT] Clarify in the tag description of some tags related
+      to "Vcs-*" fields that the tags are based on a data list
+      (and is not a result of a HTTP request while checking the
+      package).
   * checks/po-debconf:
     + [NT] Remove tests for commands that are provided by
       Lintian's dependencies.

-- 
Debian package checker


Reply to: