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

[SCM] Debian package checker branch, master, updated. 2.5.10-135-g64fb9b8



The following commit has been merged in the master branch:
commit 64fb9b812b2f9f066b3fb43ba446ccf9be9afbe6
Author: Niels Thykier <niels@thykier.net>
Date:   Sun Sep 9 11:51:02 2012 +0200

    L::Profile: Properly disable scripts
    
    When disabling tags, ensure that scripts are properly disabled when
    their last tag is.  The frontend (in 2.5.11) relies on this to avoid
    running unnecessary checks.
    
    In 2.5.10.X the frontend does the right thing despite this bug, so the
    regression is not visible in any released version of Lintian.
    
    Signed-off-by: Niels Thykier <niels@thykier.net>

diff --git a/lib/Lintian/Profile.pm b/lib/Lintian/Profile.pm
index e0ef6f8..59ed808 100644
--- a/lib/Lintian/Profile.pm
+++ b/lib/Lintian/Profile.pm
@@ -255,7 +255,8 @@ sub disable_tags {
         croak "Unknown tag $tag" unless $ti;
         next unless exists $self->{'enabled-tags'}->{$tag};
         delete $self->{'enabled-tags'}->{$tag};
-        $self->{'enabled-checks'}->{$ti->script}--;
+        delete $self->{'enabled-checks'}->{$ti->script}
+            unless --$self->{'enabled-checks'}->{$ti->script};
     }
 }
 

-- 
Debian package checker


Reply to: