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

[SCM] Debian package checker branch, master, updated. 2.5.6-87-g00f0e3d



The following commit has been merged in the master branch:
commit 00f0e3d88d71cfe59878d68bccf73388f804612b
Author: Niels Thykier <niels@thykier.net>
Date:   Thu Apr 12 19:01:07 2012 +0200

    c/control-file: Rewrite loop
    
    Signed-off-by: Niels Thykier <niels@thykier.net>

diff --git a/checks/control-file b/checks/control-file
index 5a87c6d..083b45e 100644
--- a/checks/control-file
+++ b/checks/control-file
@@ -22,6 +22,8 @@ package Lintian::control_file;
 use strict;
 use warnings;
 
+use List::Util qw(first);
+
 use Lintian::Data ();
 use Lintian::Relation ();
 use Lintian::Tags qw(tag);
@@ -101,13 +103,13 @@ if ($@) {
 
 my @package_names = sort keys %{ $info->binaries };
 
-for my $binary_control (@binary_controls) {
-    tag 'build-info-in-binary-control-file-section', 'Package '.$binary_control->{'package'}
-        if ($binary_control->{'build-depends'} || $binary_control->{'build-depends-indep'} ||
-            $binary_control->{'build-conflicts'} || $binary_control->{'build-conflicts-indep'});
-    for my $field (keys %$binary_control) {
-        tag 'binary-control-field-duplicates-source', "field \"$field\" in package ".$binary_control->{'package'},
-            if ($header->{$field} && $binary_control->{$field} eq $header->{$field});
+foreach my $bin (@package_names) {
+    my $bfields = $info->binary_field ($bin);
+    tag 'build-info-in-binary-control-file-section', "Package $bin"
+        if (first { $bfields->{"build-$_"} } qw(depends depends-indep conflicts conflicts-indep) );
+    foreach my $field (keys %$bfields) {
+        tag 'binary-control-field-duplicates-source', "field \"$field\" in package $bin"
+            if ($info->source_field ($field) && $bfields->{$field} eq $info->source_field ($field));
     }
 }
 

-- 
Debian package checker


Reply to: