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

[SCM] Debian package checker branch, master, updated. 2.4.3-35-gc3209ba



The following commit has been merged in the master branch:
commit 8bf3d100f769b9166ca73dcdaa995df0e6196d65
Author: Niels Thykier <niels@thykier.net>
Date:   Thu Dec 23 19:25:36 2010 +0100

    Since the "empty directory" check is skipped for "base-files", we
    might as well not loop over all the files in the package for it.
    This also saves a $pkg eq 'base-files' in every iteration for other
    packages.

diff --git a/checks/files b/checks/files
index 07b7fea..f575361 100644
--- a/checks/files
+++ b/checks/files
@@ -1204,15 +1204,16 @@ if ($pkg_section !~ m,games$, and $games > 0 and $other == 0) {
 # so just ignore them.
 #
 # python-support needs a directory for each package even it might be empty
-foreach my $dir (@{$info->sorted_index}) {
-    next if $dir eq "" or $info->index->{$dir}->{type} ne 'd';
-    next if ($dir =~ m{^var/} or $dir =~ m{^etc/});
-    next if $pkg eq 'base-files';
-    if (dir_counts($info, $dir) == 0) {
-	if ($dir ne 'usr/lib/perl5/'
-	    and $dir ne 'usr/share/perl5/'
-	    and $dir !~ m;^usr/share/python-support/;) {
-	    tag "package-contains-empty-directory", $dir;
+if($pkg ne 'base-files'){
+    foreach my $dir (@{$info->sorted_index}) {
+	next if $dir eq "" or $info->index->{$dir}->{type} ne 'd';
+	next if ($dir =~ m{^var/} or $dir =~ m{^etc/});
+	if (dir_counts($info, $dir) == 0) {
+	    if ($dir ne 'usr/lib/perl5/'
+		and $dir ne 'usr/share/perl5/'
+		and $dir !~ m;^usr/share/python-support/;) {
+		tag "package-contains-empty-directory", $dir;
+	    }
 	}
     }
 }

-- 
Debian package checker


Reply to: