X-Git-Url: https://git.donarmstrong.com/?p=bugscan.git;a=blobdiff_plain;f=scanlib.pm;h=82ad6cf0c76bb24c9c337611e3ace97a67b7537b;hp=1550007cb26fcc77c722704df54ba18976a7e896;hb=b466d60ef751700a50478b94761418d5a5ef55c1;hpb=a56dfd870271c1aa46516ced24f5b2b2667024e5 diff --git a/scanlib.pm b/scanlib.pm index 1550007..82ad6cf 100644 --- a/scanlib.pm +++ b/scanlib.pm @@ -21,7 +21,12 @@ use Debbugs::Packages; use Debbugs::Versions; use Debbugs::Status; use Fcntl qw(O_RDONLY); +use strict; +use warnings; require bugcfg; +package scanlib; + +our (%comments,%premature,%exclude,%maintainer,%section,%packagelist,%NMU,%debbugssection,%bugs); sub readcomments() { # Read bug commentary @@ -30,8 +35,8 @@ sub readcomments() { # Prefix a bug number with a * to force it to be listed even if it's closed. # (This deals with prematurely closed bugs) - local($index); # Bug-number for current comment - local($file); # Name of comments-file + my $index; # Bug-number for current comment + my $file; # Name of comments-file %comments = (); # Initialize our data %premature = (); @@ -62,10 +67,10 @@ sub readcomments() { # Read the list of maintainer sub readmaintainers() { - local ($pkg); # Name of package - local ($mnt); # Maintainer name & email + my $pkg; # Name of package + my $mnt; # Maintainer name & email - open(M, $maintainerlist) or die "open $maintainerlist: $!\n"; + open(M, $bugcfg::maintainerlist) or die "open $bugcfg::maintainerlist: $!\n"; while () { chomp; m/^(\S+)\s+(\S.*\S)\s*$/ or die "Maintainers: $_ ?"; @@ -82,15 +87,15 @@ sub readmaintainers() { sub readsources() { - local($root); # Root of archive we are scanning - local($archive); # Name of archive we are scanning - local($sect); # Name of current section + my $root; # Root of archive we are scanning + my $archive; # Name of archive we are scanning + my $sect; # Name of current section $root=shift; $archive=shift; - for $sect ( @sections) { + for $sect (@bugcfg::sections) { open(P, "zcat $root/$sect/source/Sources.gz|") - or die open "open: $sect / $arch sourcelist: $!\n"; + or die open "open: $sect sourcelist: $!\n"; while (

) { chomp; next unless m/^Package:\s/; @@ -102,15 +107,15 @@ sub readsources() { } sub readpackages() { - local($root); # Root of archive we are scanning - local($archive); # Name of archive we are scanning - local($sect); # Name of current section - local($arch); # Name of current architecture + my $root; # Root of archive we are scanning + my $archive; # Name of archive we are scanning + my $sect; # Name of current section + my $arch; # Name of current architecture $root=shift; $archive=shift; - for $arch ( @architectures ) { - for $sect ( @sections) { + for $arch ( @bugcfg::architectures ) { + for $sect ( @bugcfg::sections) { open(P, "zcat $root/$sect/binary-$arch/Packages.gz|") or die "open: $root/$sect/binary-$arch/Packages.gz: $!\n"; while (

) { @@ -125,8 +130,8 @@ sub readpackages() { } sub readdebbugssources() { - local($file); - local($archive); + my $file; + my $archive; $file=shift; $archive=shift; @@ -143,7 +148,7 @@ sub readdebbugssources() { } sub readpseudopackages() { - open(P, $pseudolist) or die("open $pseudolist: $!\n"); + open(P, $bugcfg::pseudolist) or die("open $bugcfg::pseudolist: $!\n"); while (

) { chomp; s/\s.*//; @@ -154,28 +159,28 @@ sub readpseudopackages() { sub scanspool() { - local(@dirs); - local($dir); + my @dirs; + my $dir; - chdir($spooldir) or die "chdir $spooldir: $!\n"; + chdir($bugcfg::spooldir) or die "chdir $bugcfg::spooldir: $!\n"; - opendir(DIR, $spooldir) or die "opendir $spooldir: $!\n"; + opendir(DIR, $bugcfg::spooldir) or die "opendir $bugcfg::spooldir: $!\n"; @dirs=grep(m/^\d+$/,readdir(DIR)); closedir(DIR); for $dir (@dirs) { - scanspooldir("$spooldir/$dir"); + scanspooldir("$bugcfg::spooldir/$dir"); } } sub scanspooldir() { - local($dir) = @_; - local($f); # While we're currently processing - local(@list); # List of files to process - local($skip); # Flow control - local($walk); # index variable - local($taginfo); # Tag info + my ($dir) = @_; + my $f; # While we're currently processing + my @list; # List of files to process + my $skip; # Flow control + my $walk; # index variable + my $taginfo; # Tag info chdir($dir) or die "chdir $dir: $!\n"; @@ -192,13 +197,13 @@ sub scanspooldir() { next if (!defined($bug)); $skip=1; - for $walk (@priorities) { + for $walk (@bugcfg::priorities) { $skip=0 if $walk eq $bug->{'severity'}; } my @tags = split(' ', $bug->{'keywords'}); - for $tag (@tags) { - for $s (@skiptags) { + for my $tag (@tags) { + for my $s (@bugcfg::skiptags) { $skip=1 if $tag eq $s; } } @@ -218,18 +223,18 @@ sub scanspooldir() { } # only bother to check the versioning status for the distributions indicated by the tags - $status_oldstable = get_status($f, $bug, 'oldstable') if ($oldstable_tag); - $status_stable = get_status($f, $bug, 'stable') if ($stable_tag); - $status_testing = get_status($f, $bug, 'testing') if ($testing_tag); - $status_unstable = get_status($f, $bug, 'unstable') if ($unstable_tag); - $status_experimental = get_status($f, $bug, 'experimental') if ($experimental_tag); - - $relinfo = ""; - $relinfo .= (($oldstable_tag && $status_oldstable->{'pending'} eq 'pending') ? "O" : ""); - $relinfo .= (($stable_tag && $status_stable->{'pending'} eq 'pending') ? "S" : ""); - $relinfo .= (($testing_tag && $status_testing->{'pending'} eq 'pending') ? "T" : ""); - $relinfo .= (($unstable_tag && $status_unstable->{'pending'} eq 'pending') ? "U" : ""); - $relinfo .= (($experimental_tag && $status_experimental->{'pending'} eq 'pending') ? "E" : ""); + my $status_oldstable = get_status($f, $bug, 'oldstable') if ($oldstable_tag); + my $status_stable = get_status($f, $bug, 'stable') if ($stable_tag); + my $status_testing = get_status($f, $bug, 'testing') if ($testing_tag); + my $status_unstable = get_status($f, $bug, 'unstable') if ($unstable_tag); + my $status_experimental = get_status($f, $bug, 'experimental') if ($experimental_tag); + + my $relinfo = ""; + $relinfo .= (($oldstable_tag && $status_oldstable eq 'pending') ? "O" : ""); + $relinfo .= (($stable_tag && $status_stable eq 'pending') ? "S" : ""); + $relinfo .= (($testing_tag && $status_testing eq 'pending') ? "T" : ""); + $relinfo .= (($unstable_tag && $status_unstable eq 'pending') ? "U" : ""); + $relinfo .= (($experimental_tag && $status_experimental eq 'pending') ? "E" : ""); next if $relinfo eq '' and not $premature{$f}; $premature{$f}++ if $relinfo eq ''; @@ -250,7 +255,7 @@ sub scanspooldir() { next if ($merged[0] < $f); } - for $package (split /[,\s]+/, $bug->{'package'}) { + for my $package (split /[,\s]+/, $bug->{'package'}) { $_= $package; y/A-Z/a-z/; $_= $` if m/[^-+._a-z0-9]/; if (not defined $section{$_}) { if (defined $debbugssection{$_}) { @@ -260,7 +265,7 @@ sub scanspooldir() { } } - $packagelist{$_} .= " $f"; + push @{$packagelist{$_}}, $f; } if ($relinfo eq "") { # or $relinfo eq "U" # confuses e.g. #210306 @@ -275,12 +280,12 @@ sub scanspooldir() { sub readstatus() { - local ($bug); # Number of current bug - local ($subject); # Subject for current bug - local ($pkg); # Name of current package - local ($file); # Name of statusfile - local ($sect); # Section of current package - local ($mnt); # Maintainer of current package + my $bug; # Number of current bug + my $subject; # Subject for current bug + my $pkg; # Name of current package + my $file; # Name of statusfile + my $sect; # Section of current package + my $mnt; # Maintainer of current package $file=shift; open(P, $file) or die "open $file: $!"; @@ -289,9 +294,10 @@ sub readstatus() { if (m/^[0-9]+ \[/) { ($bug,$subject)=split(/ /, $_, 2); $bugs{$bug}=$subject; - $packagelist{$pkg} .= "$bug "; + push @{$packagelist{$pkg}}, $bug; } else { ($pkg,$sect, $mnt)=split(/ /, $_, 3); + next if (!defined($pkg)); $section{$pkg}=$sect; $maintainer{$pkg}=$mnt; } @@ -301,11 +307,11 @@ sub readstatus() { sub readNMUstatus() { - local ($bug); # Number of current bug - local ($source); # Source upload which closes this bug. - local ($version); # Version where this bug was closed. - local ($flag); # Whether this paragraph has been processed. - local ($field, $value); + my $bug; # Number of current bug + my $source; # Source upload which closes this bug. + my $version; # Version where this bug was closed. + my $flag; # Whether this paragraph has been processed. + my ($field, $value); for (split /\n/, LWP::UserAgent->new->request(HTTP::Request->new(GET => shift))->content) { chomp; @@ -350,8 +356,8 @@ sub htmlsanit { } sub wwwnumber() { - local ($number) = shift; # Number of bug to html-ize -# local ($section); # Section for the bug + my $number = shift; # Number of bug to html-ize +# my $section); # Section for the bug "' . htmlsanit($number) . ''; @@ -360,7 +366,7 @@ sub wwwnumber() { } sub wwwname() { - local ($name) = shift; # Name of package + my $name = shift; # Name of package "' . htmlsanit($name) . ''; @@ -371,11 +377,13 @@ my $_version_cache = {}; sub get_status() { my ($bugnr, $bug, $dist) = @_; - my $status = 'pending'; - my @versions = Debbugs::Status::getversions($bug->{'package'}, $dist, undef); my @sourceversions = Debbugs::Status::makesourceversions($bug->{'package'}, undef, @versions); + if (length($bug->{'done'}) and + (not @sourceversions or not @{$bug->{'fixed_versions'}})) { + return 'done'; + } if (@sourceversions) { my $max_buggy = Debbugs::Status::max_buggy(bug => $bugnr, sourceversions => \@sourceversions, @@ -383,18 +391,12 @@ sub get_status() { fixed => $bug->{'fixed_versions'}, version_cache => $_version_cache, package => $bug->{'package'}); - if ($max_buggy eq 'absent') { - $status = 'absent'; - } elsif ($max_buggy eq 'fixed') { - $status = 'done'; + if ($max_buggy eq 'absent' || $max_buggy eq 'fixed') { + return $max_buggy; } } - if (length($bug{'done'}) and - (not @sourceversions or not @{$status{'fixed_versions'}})) { - $status = 'done'; - } - return $status; + return 'pending'; } sub check_worry {