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

[SCM] Debian package checker branch, master, updated. 2.5.10-99-ga75087f



The following commit has been merged in the master branch:
commit a75087f7ed9e68541e7d7cd84aebccb863d12410
Author: Niels Thykier <niels@thykier.net>
Date:   Fri Jul 27 10:22:22 2012 +0200

    L::Tag::Info: Pass $tagname to croak rather than $tag
    
    Also fix typo of Certainty field name.
    
    Signed-off-by: Niels Thykier <niels@thykier.net>

diff --git a/debian/changelog b/debian/changelog
index 0e35850..0da3a41 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -114,6 +114,10 @@ lintian (2.5.11) UNRELEASED; urgency=low
   * lib/Lintian/Path.pm:
     + [NT] Always resolve hardlinks relative to the package
       root.
+  * lib/Lintian/Tag/Info.pm:
+    + [NT] Fix poor error message due to the wrong variable
+      being unused.  Thanks to Christoph Biedl for the report
+      (Closes: #682941)
   * lib/Lintian/Unpacker.pm:
     + [NT] New file.
 
diff --git a/lib/Lintian/Tag/Info.pm b/lib/Lintian/Tag/Info.pm
index af5a9ad..cf3f1bd 100644
--- a/lib/Lintian/Tag/Info.pm
+++ b/lib/Lintian/Tag/Info.pm
@@ -87,8 +87,8 @@ sub new {
     $self = \%copy;
     croak "Missing Tag field" unless $self->{'tag'};
     $tagname = $self->{'tag'};
-    croak "Missing Severity field for $tag" unless $self->{'severity'};
-    croak "Missing Certainity field for $tag" unless $self->{'certainty'};
+    croak "Missing Severity field for $tagname" unless $self->{'severity'};
+    croak "Missing Certainty field for $tagname" unless $self->{'certainty'};
     $self->{'info'} = '' unless $self->{'info'};
     $self->{'script'} = $sn;
     $self->{'script-type'} = $st;
@@ -100,7 +100,7 @@ sub new {
     # certainty is wrong (or we introduced a new one but forgot to add
     # it to %CODES).
     unless ($self->code) {
-        croak "Cannot determine the code of $tag (severity:"
+        croak "Cannot determine the code of $tagname (severity:"
             . " $self->{'severity'}, certainity: $self->{'certainty'}).\n";
     }
 

-- 
Debian package checker


Reply to: