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

[SCM] Debian package checker branch, master, updated. 1.24.2-30-g7360d38



The following commit has been merged in the master branch:
commit 7360d38822f2bc4f8129c318cbee285a89009722
Author: Adam D. Barratt <adam@adam-barratt.org.uk>
Date:   Fri Jul 25 02:02:10 2008 +0100

    Use ::Collect::field() to determine the package's architecture
    
    * checks/huge-usr-share:
      + [ADB] Replace (the single) direct access to fields/* with ::field()

diff --git a/checks/huge-usr-share b/checks/huge-usr-share
index 71c4782..9f48baf 100644
--- a/checks/huge-usr-share
+++ b/checks/huge-usr-share
@@ -26,6 +26,7 @@ sub run {
 
 my $pkg = shift;
 my $type = shift;
+my $info = shift;
 
 # Threshold in kB of /usr/share to trigger this warning
 # Consider that the changelog alone can be quite big, and cannot be moved away
@@ -35,10 +36,9 @@ my $THRESHOLD_PERC = 50;
 
 my $arch;
 
-# read architecture file
-if (open(IN, '<', "fields/architecture")) {
-    chop($arch = <IN>);
-    close(IN);
+# read architecture
+if (defined $info->field('architecture')) {
+    $arch = $info->field('architecture');
 }
 
 # If this is a arch-independent package, this test doesn't apply
diff --git a/debian/changelog b/debian/changelog
index 699ff24..59ce049 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -24,6 +24,8 @@ lintian (1.24.3) unstable; urgency=low
   * checks/files:
     + [ADB] Begin switching to Lintian::Collect.  Currently objdump-info
       parsing and fields/* reading have been migrated.
+  * checks/huge-usr-share:
+    + [ADB] Replace (the single) direct access to fields/* with ::field()
   * checks/manpages:
     + [RA] Suppress warnings about inability to break a line that contains
       a URL.  URLs often can't be usefully broken.  Based on a patch by

-- 
Debian package checker


Reply to: