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

Bug#641609: Lintian::Collect::Source::binary_relation broken



Package: lintian
Version: 2.5.3
Severity: minor
Tags: patch

Lintian::Collect::Source::binary_relation() doesn't work at all. The attached patch fixes it.

BTW, once it's fixed, we could use it to simplify implementation of {debhelper-but-no,weak-dependency-on}-misc-depends tags.

--
Jakub Wilk
diff --git a/lib/Lintian/Collect/Source.pm b/lib/Lintian/Collect/Source.pm
--- a/lib/Lintian/Collect/Source.pm
+++ b/lib/Lintian/Collect/Source.pm
@@ -23,6 +23,7 @@
 use warnings;
 use base 'Lintian::Collect::Package';
 
+use Carp qw(croak);
 use Cwd();
 
 use Lintian::Relation;
@@ -186,7 +187,7 @@
         my $merged;
         for my $f (@{ $special{$field} }) {
 	    # sub binary_relation Needs-Info :binary_field
-            my $value = $self->binary_field($f);
+            my $value = $self->binary_field($package, $f);
             $merged .= ', ' if (defined($merged) and defined($value));
             $merged .= $value if defined($value);
         }
@@ -201,7 +202,7 @@
     }
     $result = Lintian::Relation->new($result);
     $self->{binary_relation}->{$package}->{$field} = $result;
-    return $self->{binary_relation}->{$field};
+    return $result;
 }
 
 

Reply to: