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

[lintian] 03/03: Avoid to check lintian for license check



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

broucaries-guest pushed a commit to branch master
in repository lintian.

commit 3f614f2ab008ff30935dcffa64ff85bf174f6960
Author: Bastien ROUCARIÈS <roucaries.bastien+debian@gmail.com>
Date:   Sun Apr 6 19:32:34 2014 +0200

    Avoid to check lintian for license check
    
    Lintian trip a few license check due to embedding the verbatim text of license.
    
    Avoid to add override for lintian.
    
    Signed-off-by: Bastien ROUCARIÈS <roucaries.bastien+debian@gmail.com>
---
 checks/cruft.pm | 27 ++++++++++++++++-----------
 1 file changed, 16 insertions(+), 11 deletions(-)

diff --git a/checks/cruft.pm b/checks/cruft.pm
index e6651a7..b586389 100644
--- a/checks/cruft.pm
+++ b/checks/cruft.pm
@@ -306,7 +306,8 @@ my @file_checks = (
 our @EOL_TERMINATORS_FILES = qw(control changelog);
 
 sub run {
-    my (undef, undef, $info) = @_;
+    my (undef, undef, $info, $proc) = @_;
+    my $source_pkg = $proc->pkg_src;
     my $droot = $info->debfiles;
 
     if (-e "$droot/files" and not -z "$droot/files") {
@@ -353,7 +354,7 @@ sub run {
     }elsif (not $info->native) {
         check_diffstat($info->diffstat, \%warned);
     }
-    find_cruft($info, \%warned, $atdinbd, $ltinbd);
+    find_cruft($source_pkg, $info, \%warned, $atdinbd, $ltinbd);
 
     for my $file (@EOL_TERMINATORS_FILES) {
         my $path = $info->debfiles($file);
@@ -507,7 +508,7 @@ sub check_debfiles {
 # "source-contains" tag.  The tag isn't entirely accurate, but it's better
 # than creating yet a third set of tags, and this gets the severity right.
 sub find_cruft {
-    my ($info, $warned, $atdinbd, $ltinbd) = @_;
+    my ($source_pkg, $info, $warned, $atdinbd, $ltinbd) = @_;
     my $prefix = ($info->native ? 'diff-contains' : 'source-contains');
     my @worklist;
 
@@ -678,7 +679,7 @@ sub find_cruft {
             }
             close($fd);
         }
-        full_text_check($entry, $info, $name, $basename, $dirname,
+        full_text_check($source_pkg, $entry, $info, $name, $basename, $dirname,
             $info->unpacked($entry));
     }
     return;
@@ -755,7 +756,7 @@ sub check_missing_source {
 # note that it does not replace licensecheck(1)
 # and is only used for autoreject by ftp-master
 sub full_text_check {
-    my ($entry, $info, $name, $basename, $dirname, $path) = @_;
+    my ($source_pkg, $entry, $info, $name, $basename, $dirname, $path) = @_;
 
     # license string in debian/changelog are probably just change
     # Ignore these strings in d/README.{Debian,source}.  If they
@@ -791,9 +792,8 @@ sub full_text_check {
 
         if(
             _license_check(
-                $name, $NON_DISTRIBUTABLE_LICENSES,
-                $block,  $blocknumber,
-                \$cleanedblock, \%matchedkeyword,
+                $source_pkg, $name,$NON_DISTRIBUTABLE_LICENSES,$block,
+                $blocknumber,\$cleanedblock,\%matchedkeyword,
                 \%licenseproblemhash
             )
           ){
@@ -806,7 +806,7 @@ sub full_text_check {
             next BLOCK;
         }
 
-        _license_check($name, $NON_FREE_LICENSES,
+        _license_check($source_pkg, $name, $NON_FREE_LICENSES,
             $block,  $blocknumber,\$cleanedblock, \%matchedkeyword,
             \%licenseproblemhash);
 
@@ -1109,10 +1109,15 @@ sub _md5sum_based_check {
 
 # check bad license
 sub _license_check {
-    my ($name, $licensesdatas, $block, $blocknumber, $cleanedblock,
-        $matchedkeyword, $licenseproblemhash)
+    my ($source_pkg, $name, $licensesdatas, $block, $blocknumber,
+        $cleanedblock,$matchedkeyword, $licenseproblemhash)
       = @_;
     my $ret = 0;
+
+    # avoid to check lintian
+    if($source_pkg eq 'lintian') {
+        return $ret;
+    }
   LICENSE:
     foreach my $licenseproblem ($licensesdatas->all) {
         my $licenseproblemdata = $licensesdatas->value($licenseproblem);

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


Reply to: