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

[lintian] 01/02: checks/fields.pm: Use next so we don't have to keep considering the Vcs-Browser quasi Vcs field.



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

lamby pushed a commit to branch master
in repository lintian.

commit f4aa87071025ea1b7a6b6f8051f71f5ab56764ff
Author: Chris Lamb <lamby@debian.org>
Date:   Thu Jan 4 23:07:52 2018 +0000

    checks/fields.pm: Use next so we don't have to keep considering the Vcs-Browser quasi Vcs field.
---
 checks/fields.pm | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/checks/fields.pm b/checks/fields.pm
index 734959c..755612f 100644
--- a/checks/fields.pm
+++ b/checks/fields.pm
@@ -1333,9 +1333,12 @@ sub run {
                     tag $tag, $parts[0], $canonicalized;
                 }
             }
-            tag 'vcs-browser-links-to-empty-view', $uri
-              if $vcs eq 'browser' and $uri =~ m%rev=0&sc=0%;
-            $seen_vcs{$vcs}++ if $vcs ne 'browser';
+            if ($vcs eq 'browser') {
+                tag 'vcs-browser-links-to-empty-view', $uri
+                  if $uri =~ m%rev=0&sc=0%;
+                next;
+            }
+            $seen_vcs{$vcs}++;
             if (    $info->field('maintainer', '') =~ /packages\@qa.debian.org/
                 and $uri !~ m/\.debian\.org/) {
                 tag 'orphaned-package-not-maintained-in-debian-infrastructure',

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


Reply to: