X-Git-Url: https://git.donarmstrong.com/?a=blobdiff_plain;f=bin%2Fwanna-build;h=1d87779ebea538a579f5c962c8369d4c63ff6501;hb=09f13aba8cee37e89ff5df9ca10e72c9073d91d4;hp=5fb90f011cbaed3f70794b29ef77507c4b72be60;hpb=61d541aceb85df01eeab1e55d1d6613c195e3e83;p=wannabuild.git diff --git a/bin/wanna-build b/bin/wanna-build index 5fb90f0..1d87779 100755 --- a/bin/wanna-build +++ b/bin/wanna-build @@ -25,7 +25,7 @@ $basedir ||= "/var/lib/debbuild"; $dbbase ||= "build-db"; $transactlog ||= "transactions.log"; $mailprog ||= "/usr/sbin/sendmail"; -require "/etc/wanna-build.conf"; +require "/org/wanna-build/etc/wanna-build.conf"; die "$conf::basedir is not a directory\n" if ! -d $conf::basedir; die "dbbase is empty\n" if ! $dbbase; die "transactlog is empty\n" if ! $transactlog; @@ -38,30 +38,35 @@ use strict; use POSIX; use FileHandle; use File::Copy; -use GDBM_File; -use MLDBM qw(GDBM_File Storable); +use DBI; +use lib '/org/wanna-build/bin'; use WannaBuild; +use YAML::Tiny; +use Data::Dumper; +use Hash::Merge qw ( merge ); +use String::Format; +use Date::Parse; our ($verbose, $mail_logs, $list_order, $list_state, $curr_date, $op_mode, $user, $real_user, $distribution, $fail_reason, $opt_override, $import_from, $export_to, $opt_create_db, - %db, %otherdb, %otherdb_lock, %prioval, %sectval, + %prioval, %sectval, $info_all_dists, $arch, $category, %catval, %short_category, $short_date, $list_min_age, $dbbase, @curr_time, $build_priority, %new_vers, $binNMUver, %merge_srcvers, %merge_binsrc, - $lock_for_pid, $transactional); + $printformat, $ownprintformat + ); # global vars -$ENV{'PATH'} = "/bin:/usr/bin:/usr/local/bin"; +$ENV{'PATH'} = "/bin:/usr/bin:/usr/local/bin:/org/wanna-build/bin/"; +$ENV{'LC_ALL'} = 'C'; $verbose = 0; $mail_logs = ""; @curr_time = gmtime; $curr_date = strftime("%Y %b %d %H:%M:%S",@curr_time); $short_date = strftime("%m/%d/%y",@curr_time); $| = 1; -$lock_for_pid = -1; # -1 means normal procedure -$transactional = 0; # 0 means: work on main copy # map program invocation names to operation modes my %prognames = ( "uploaded-build" => "set-uploaded", @@ -121,6 +126,7 @@ my %options = "merge-sources" => { mode => "merge-sources" }, "pretend-avail" => { short => "p", mode => "pretend-avail" }, "merge-all" => { mode => "merge-all" }, + "merge-all-secondary" => { mode => "merge-all-secondary" }, info => { short => "i", mode => "info" }, 'binNMU' => { mode => 'set-binary-nmu', arg => \$binNMUver, code => sub { die "Invalid binNMU version: $binNMUver\n" @@ -158,9 +164,10 @@ my %options = { short => "O", arg => \$list_order, code => sub { die "Bad ordering character\n" - if $list_order !~ /^[PSpsncb]+$/; + if $list_order !~ /^[PSpsncbCW]+$/; } }, message => { short => "m", arg => \$fail_reason }, + # database is deprecated, use arch instead. database => { short => "b", arg => \$conf::dbbase }, arch => { short => "A", arg => \$arch }, user => { short => "U", arg => \$user }, @@ -175,27 +182,19 @@ my %options = code => sub { die "Argument of --min-age must be a non-zero number\n" if $list_min_age == 0; - $list_min_age *= 24*60*60; } }, "max-age" => { arg => \$list_min_age, code => sub { die "Argument of --max-age must be a non-zero number\n" if $list_min_age == 0; - $list_min_age *= -24*60*60; + $list_min_age *= -1; } }, + "format" => { arg => \$printformat }, + "own-format" => { arg => \$ownprintformat }, # special actions - import => { arg => \$import_from, mode => "import" }, export => { arg => \$export_to, mode => "export" }, - "lock-for" => { arg => \$lock_for_pid, mode => "lock-for" }, - "unlock-for" => { arg => \$lock_for_pid, mode => "unlock-for" }, - "act-on-behalf-of" => { arg => \$lock_for_pid }, - "start-transaction" => { mode => "start-transaction" }, - "commit-transaction" => { mode => "commit-transaction" }, - "transactional" => { flag => \$transactional }, + import => { arg => \$import_from, mode => "import" }, "manual-edit" => { mode => "manual-edit" }, - "create-maintenance-lock" => { mode => "maintlock-create" }, - "remove-maintenance-lock" => { mode => "maintlock-remove" }, - "clean-db" => { mode => "clean-db" }, ); while( @ARGV && $ARGV[0] =~ /^-/ ) { @@ -241,22 +240,29 @@ while( @ARGV && $ARGV[0] =~ /^-/ ) { $op_mode = $category ? "set-failed" : "set-building" if !$op_mode; # default operation -$list_order = $list_state eq "failed" ? 'fPcpsn' : 'PScpsn' - if !$list_order and $list_state; $distribution ||= "unstable"; die "Bad distribution '$distribution'\n" if !isin($distribution, keys %conf::distributions); +# If they didn't specify an arch, try to get it from database name which +# is in the form of $arch/build-db +# This is for backwards compatibity with older versions that didn't +# specify the arch yet. +$conf::dbbase =~ m#^([^/]+)#; +$arch ||= $1; + +# TODO: Check that it's an known arch (for that dist), and give +# a proper error. + if ($verbose) { my $version = '$Revision: db181a534e9d $ $Date: 2008/03/26 06:20:22 $ $Author: rmurray $'; $version =~ s/(^\$| \$ .*$)//g; - print "wanna-build $version for $distribution on $conf::dbbase\n"; + print "wanna-build $version for $distribution on $arch\n"; } if (!@ARGV && !isin( $op_mode, qw(list merge-quinn merge-partial-quinn import export - merge-packages manual-edit maintlock-create lock-for unlock-for - start-transaction commit-transaction - merge-sources maintlock-remove clean-db))) { + merge-packages manual-edit + merge-sources))) { warn "No packages given.\n"; usage(); } @@ -295,78 +301,58 @@ if (!$fail_reason) { $fail_reason = $line; } } -if ($op_mode eq "maintlock-create") { - create_maintlock(); - exit 0; -} -if ($op_mode eq "maintlock-remove") { - remove_maintlock(); - exit 0; -} -waitfor_maintlock() if $op_mode !~ /^(?:merge-|clean-db$)/; -if (!-f db_filename( $distribution ) && !$opt_create_db) { - if ($transactional) { - die "No running transaction for $distribution\n"; - } else { - die "Database for $distribution doesn't exist\n"; - } +my $yamlmap = (); +my $yamldir = "/org/wanna-build/etc/yaml"; +my @files = ('wanna-build.yaml'); +if ((getpwuid($>))[7]) { push (@files, ((getpwuid($>))[7])."/.wanna-build.yaml"); } +if ($user =~ /(buildd.*)-/) { push (@files, "$1.yaml") }; +if ($user) { push ( @files, "$user.yaml"); } +foreach my $file (@files) { + my $cfile = File::Spec->rel2abs( $file, $yamldir ); + if ($verbose >= 2) { print "Trying to read $file ($cfile) ...\n"; } + next unless -f $cfile; + if ($verbose >= 2) { print "Read $file ($cfile) ...\n"; } + my $m = YAML::Tiny->read( $cfile )->[0]; + $yamlmap = merge($m, $yamlmap); } - -# Locking for another process means that a longer running process (most likely -# wb) wants to do several steps at once, and manages the locks. -if ($op_mode eq "lock-for") { - lock_db( $distribution ); - exit 0; -} -if ($op_mode eq "unlock-for") { - unlock_db( $distribution ); - exit 0; +if (not $yamlmap) { + die "FATAL: no configuration found\n"; } +$list_order = $yamlmap->{"list-order"}{$list_state} if !$list_order and $list_state; +$list_order ||= $yamlmap->{"list-order"}{'default'}; -lock_db( $distribution ); +my $dbh; -if ($op_mode eq "start-transaction") { - copy ( db_filename_master( $distribution ), db_filename_transaction( $distribution )) - or die "Copy failed: $!"; - open LOG, ">", db_transactlog_transaction( $distribution ) - or die "Could not create logfile for transaction: $!"; - close LOG; - exit 0; +END { + if (defined $dbh) + { + $dbh->disconnect or warn $dbh->errstr; + } } -if ($op_mode eq "commit-transaction") { - # we need to copy here to preserve the owner and group of the file - copy ( db_filename_transaction( $distribution ), db_filename_master( $distribution )) - or die "Copy failed: $!"; - unlink db_filename_transaction( $distribution ); - open TLOG, "<", db_transactlog_transaction( $distribution ) - or die "Could not open logfile from transaction: $!"; - open LOG, ">>", db_transactlog_master( $distribution ) - or die "Could not open logfile: $!"; - while () { print LOG $_ }; - close TLOG; - close LOG; - unlink db_transactlog_transaction( $distribution ); - exit 0; +my $schema_suffix = ''; +if (isin( $op_mode, qw(list info)) && $distribution !~ /security/ && !(not -t and $user =~ /-/)) { + $dbh = DBI->connect("DBI:Pg:service=wanna-build") || + die "FATAL: Cannot open database: $DBI::errstr\n"; + $schema_suffix = '_public'; } - -END { - untie %db; - if ($lock_for_pid == -1) { - unlock_db( $distribution ); - } - foreach (keys %conf::distributions) { - untie %{$otherdb{$_}} if tied(%{$otherdb{$_}}); - unlock_db( $_ ) if $otherdb_lock{$_}; - } +else +{ + $dbh = DBI->connect("DBI:Pg:service=wanna-build-privileged") || + die "FATAL: Cannot open database: $DBI::errstr\n"; } -tie %db, 'MLDBM', db_filename( $distribution ), GDBM_WRCREAT, 0664 - or die "FATAL: Cannot open database\n"; +# TODO: This shouldn't be needed, file a bug. +$dbh->{pg_server_prepare} = 0; + +$dbh->begin_work or die $dbh->errstr; process(); +$dbh->commit; +$dbh->disconnect; + if ($mail_logs && $conf::log_mail) { send_mail( $conf::log_mail, "wanna-build $distribution state changes $curr_date", @@ -407,6 +393,7 @@ sub process { die "This operation is restricted to admin users\n" if (defined @conf::admin_users and !isin( $real_user, @conf::admin_users)); + lock_table(); parse_quinn_diff(1); last SWITCH; }; @@ -414,6 +401,7 @@ sub process { die "This operation is restricted to admin users\n" if (defined @conf::admin_users and !isin( $real_user, @conf::admin_users)); + lock_table(); parse_quinn_diff(0); last SWITCH; }; @@ -421,13 +409,15 @@ sub process { die "This operation is restricted to admin users\n" if (defined @conf::admin_users and !isin( $real_user, @conf::admin_users)); - parse_packages(); + lock_table(); + parse_packages(0); last SWITCH; }; /^merge-sources/ && do { die "This operation is restricted to admin users\n" if (defined @conf::admin_users and !isin( $real_user, @conf::admin_users)); + lock_table(); parse_sources(0); last SWITCH; }; @@ -435,74 +425,73 @@ sub process { pretend_avail( @ARGV ); last SWITCH; }; - /^merge-all/ && do { + /^merge-all$/ && do { die "This operation is restricted to admin users\n" if (defined @conf::admin_users and !isin( $real_user, @conf::admin_users)); + lock_table(); my @ARGS = @ARGV; @ARGV = ( $ARGS[0] ); - my $pkgs = parse_packages(); + my $pkgs = parse_packages(0); @ARGV = ( $ARGS[1] ); parse_quinn_diff(0); @ARGV = ( $ARGS[2] ); - my $build_deps = parse_sources(1); - call_edos_depcheck( $ARGS[0], $ARGS[2] ); - clean_db(); + my $srcs = parse_sources(1); + call_edos_depcheck( $ARGS[0], $srcs ); last SWITCH; }; - /^import/ && do { + /^merge-all-secondary/ && do { die "This operation is restricted to admin users\n" if (defined @conf::admin_users and !isin( $real_user, @conf::admin_users)); - %db = (); # clear all current contents - read_db( $import_from ); - last SWITCH; - }; - /^export/ && do { - write_db( $export_to ); + # This is in case the chroot has multiple unrelated + # dist, for instance unstable and experimental. + # This is not for stable and proposed-updates. + # The second packages file contains a combination + # of all Packages files known to the buildd, the + # first only for the current dist. + lock_table(); + my @ARGS = @ARGV; + @ARGV = ( $ARGS[0] ); + my $pkgs = parse_packages(0); + @ARGV = ( $ARGS[3] ); + my $pkgs = parse_packages(1); + @ARGV = ( $ARGS[1] ); + parse_quinn_diff(0); + @ARGV = ( $ARGS[2] ); + my $srcs = parse_sources(1); + call_edos_depcheck( $ARGS[3], $srcs ); last SWITCH; }; - /^manual-edit/ && do { + /^import/ && do { die "This operation is restricted to admin users\n" if (defined @conf::admin_users and !isin( $real_user, @conf::admin_users)); - my $tmpfile_pattern = "/tmp/wanna-build-$distribution.$$-"; - my ($tmpfile, $i); - for( $i = 0;; ++$i ) { - $tmpfile = $tmpfile_pattern . $i; - last if ! -e $tmpfile; - } - write_db( $tmpfile ); - my $editor = $ENV{'VISUAL'} || - "/usr/bin/sensible-editor"; - system "$editor $tmpfile"; - %db = (); # clear all current contents - read_db( $tmpfile ); - unlink( $tmpfile ); + $dbh->do("DELETE from " . table_name() . + " WHERE distribution = ?", undef, + $distribution) + or die $dbh->errstr; + forget_users(); + read_db( $import_from ); last SWITCH; }; - /^clean-db/ && do { - die "This operation is restricted to admin users\n" - if (defined @conf::admin_users and - !isin( $real_user, @conf::admin_users)); - clean_db(); + /^export/ && do { + export_db( $export_to ); last SWITCH; }; die "Unexpected operation mode $op_mode\n"; } if (not -t and $user =~ /-/) { - my $userinfo = $db{'_userinfo'}; - $userinfo = {} if (!defined($userinfo)); - - my $ui = $userinfo->{$user}; - $ui = {} if (!defined($ui)); - - $ui->{'Last-Seen'} = $curr_date; - $ui->{'User'} = $user; - - $userinfo->{$user} = $ui; - $db{'_userinfo'} = $userinfo; + my $userinfo = get_user_info($user); + if (!defined $userinfo) + { + add_user_info($user); + } + else + { + update_user_info($user); + } } } @@ -548,10 +537,10 @@ sub add_packages { add_one_depwait( $name, $version ); } elsif ($op_mode eq "set-build-priority") { - set_one_buildpri( $name, $version, 'BuildPri' ); + set_one_buildpri( $name, $version, 'buildpri' ); } elsif ($op_mode eq "set-permanent-build-priority") { - set_one_buildpri( $name, $version, 'PermBuildPri' ); + set_one_buildpri( $name, $version, 'permbuildpri' ); } elsif ($op_mode eq "set-binary-nmu") { set_one_binnmu( $name, $version ); @@ -565,64 +554,64 @@ sub add_one_building { my( $ok, $reason ); $ok = 1; - my $pkg = $db{$name}; + my $pkg = get_source_info($name); if (defined($pkg)) { - if ($pkg->{'State'} eq "Not-For-Us") { + if ($pkg->{'state'} eq "Not-For-Us") { $ok = 0; $reason = "not suitable for this architecture"; } - elsif ($pkg->{'State'} =~ /^Dep-Wait/) { + elsif ($pkg->{'state'} =~ /^Dep-Wait/) { $ok = 0; $reason = "not all source dependencies available yet"; } - elsif ($pkg->{'State'} =~ /^BD-Uninstallable/) { + elsif ($pkg->{'state'} =~ /^BD-Uninstallable/) { $ok = 0; $reason = "source dependencies are not installable"; } - elsif ($pkg->{'State'} eq "Uploaded" && - (version_lesseq($version, $pkg->{'Version'}))) { + elsif ($pkg->{'state'} eq "Uploaded" && + (version_lesseq($version, $pkg->{'version'}))) { $ok = 0; - $reason = "already uploaded by $pkg->{'Builder'}"; - $reason .= " (in newer version $pkg->{'Version'})" + $reason = "already uploaded by $pkg->{'builder'}"; + $reason .= " (in newer version $pkg->{'version'})" if !version_eq($pkg, $version); } - elsif ($pkg->{'State'} eq "Installed" && - version_less($version,$pkg->{'Version'})) { + elsif ($pkg->{'state'} eq "Installed" && + version_less($version,$pkg->{'version'})) { if ($opt_override) { - print "$name: Warning: newer version $pkg->{'Version'} ". + print "$name: Warning: newer version $pkg->{'version'} ". "already installed, but overridden.\n"; } else { $ok = 0; - $reason = "newer version $pkg->{'Version'} already in ". + $reason = "newer version $pkg->{'version'} already in ". "archive; doesn't need rebuilding"; print "$name: Note: If the following is due to an epoch ", " change, use --override\n"; } } - elsif ($pkg->{'State'} eq "Installed" && + elsif ($pkg->{'state'} eq "Installed" && pkg_version_eq($pkg,$version)) { $ok = 0; $reason = "is up-to-date in the archive; doesn't need rebuilding"; } - elsif ($pkg->{'State'} eq "Needs-Build" && - version_less($version,$pkg->{'Version'})) { + elsif ($pkg->{'state'} eq "Needs-Build" && + version_less($version,$pkg->{'version'})) { if ($opt_override) { - print "$name: Warning: newer version $pkg->{'Version'} ". + print "$name: Warning: newer version $pkg->{'version'} ". "needs building, but overridden."; } else { $ok = 0; - $reason = "newer version $pkg->{'Version'} needs building, ". + $reason = "newer version $pkg->{'version'} needs building, ". "not $version"; } } - elsif (isin($pkg->{'State'},qw(Building Built Build-Attempted))) { - if (version_less($pkg->{'Version'},$version)) { - print "$name: Warning: Older version $pkg->{'Version'} ", - "is being built by $pkg->{'Builder'}\n"; - if ($pkg->{'Builder'} ne $user) { - send_mail( $pkg->{'Builder'}, + elsif (isin($pkg->{'state'},qw(Building Built Build-Attempted))) { + if (version_less($pkg->{'version'},$version)) { + print "$name: Warning: Older version $pkg->{'version'} ", + "is being built by $pkg->{'builder'}\n"; + if ($pkg->{'builder'} ne $user) { + send_mail( $pkg->{'builder'}, "package takeover in newer version", "You are building package '$name' in ". "version $version\n". @@ -634,66 +623,66 @@ sub add_one_building { } else { if ($opt_override) { - print "User $pkg->{'Builder'} had already ", + print "User $pkg->{'builder'} had already ", "taken the following package,\n", "but overriding this as you request:\n"; - send_mail( $pkg->{'Builder'}, "package takeover", + send_mail( $pkg->{'builder'}, "package takeover", "The package '$name' (version $version) that ". - "was locked by you\n". + "was taken by you\n". "has been taken over by $user\n" ); } - elsif ($pkg->{'Builder'} eq $user) { + elsif ($pkg->{'builder'} eq $user) { print "$name: Note: already taken by you.\n"; print "$name: ok\n" if $verbose; return; } else { $ok = 0; - $reason = "already taken by $pkg->{'Builder'}"; - $reason .= " (in newer version $pkg->{'Version'})" - if !version_eq($pkg->{'Version'}, $version); + $reason = "already taken by $pkg->{'builder'}"; + $reason .= " (in newer version $pkg->{'version'})" + if !version_eq($pkg->{'version'}, $version); } } } - elsif ($pkg->{'State'} =~ /^Failed/ && + elsif ($pkg->{'state'} =~ /^Failed/ && pkg_version_eq($pkg, $version)) { if ($opt_override) { print "The following package previously failed ", - "(by $pkg->{'Builder'})\n", + "(by $pkg->{'builder'})\n", "but overriding this as you request:\n"; - send_mail( $pkg->{'Builder'}, "failed package takeover", + send_mail( $pkg->{'builder'}, "failed package takeover", "The package '$name' (version $version) that ". - "is locked by you\n". + "is taken by you\n". "and has failed previously has been taken over ". "by $user\n" ) - if $pkg->{'Builder'} ne $user; + if $pkg->{'builder'} ne $user; } else { $ok = 0; $reason = "build of $version failed previously:\n "; - $reason .= join( "\n ", split( "\n", $pkg->{'Failed'} )); + $reason .= join( "\n ", split( "\n", $pkg->{'failed'} )); $reason .= "\nalso the package doesn't need builing" - if $pkg->{'State'} eq 'Failed-Removed'; + if $pkg->{'state'} eq 'Failed-Removed'; } } } if ($ok) { my $ok = 'ok'; - if ($pkg->{'Binary-NMU-Version'}) { - print "$name: Warning: needs binary NMU $pkg->{'Binary-NMU-Version'}\n" . - "$pkg->{'Binary-NMU-Changelog'}\n"; + if ($pkg->{'binary_nmu_version'}) { + print "$name: Warning: needs binary NMU $pkg->{'binary_nmu_version'}\n" . + "$pkg->{'binary_nmu_changelog'}\n"; $ok = 'aok'; } else { print "$name: Warning: Previous version failed!\n" - if $pkg->{'Previous-State'} =~ /^Failed/ || - $pkg->{'State'} =~ /^Failed/; + if $pkg->{'previous_state'} =~ /^Failed/ || + $pkg->{'state'} =~ /^Failed/; } change_state( \$pkg, 'Building' ); - $pkg->{'Package'} = $name; - $pkg->{'Version'} = $version; - $pkg->{'Builder'} = $user; + $pkg->{'package'} = $name; + $pkg->{'version'} = $version; + $pkg->{'builder'} = $user; log_ta( $pkg, "--take" ); - $db{$name} = $pkg; + update_source_info($pkg); print "$name: $ok\n" if $verbose; } else { @@ -704,111 +693,111 @@ sub add_one_building { sub add_one_attempted { my $name = shift; my $version = shift; - my $pkg = $db{$name}; + my $pkg = get_source_info($name); if (!defined($pkg)) { print "$name: not registered yet.\n"; return; } - if ($pkg->{'State'} ne "Building" ) { - print "$name: not taken for building (state is $pkg->{'State'}). ", + if (($pkg->{'state'} ne "Building") && ($pkg->{'state'} ne "Build-Attempted")) { + print "$name: not taken for building (state is $pkg->{'state'}). ", "Skipping.\n"; return; } - if ($pkg->{'Builder'} ne $user) { - print "$name: not taken by you, but by $pkg->{'Builder'}. Skipping.\n"; + if ($pkg->{'builder'} ne $user) { + print "$name: not taken by you, but by $pkg->{'builder'}. Skipping.\n"; return; } elsif ( !pkg_version_eq($pkg, $version) ) { print "$name: version mismatch ". - "$(pkg->{'Version'} ". - "by $pkg->{'Builder'})\n"; + "$(pkg->{'version'} ". + "by $pkg->{'builder'})\n"; return; } change_state( \$pkg, 'Build-Attempted' ); log_ta( $pkg, "--attempted" ); - $db{$name} = $pkg; + update_source_info($pkg); print "$name: registered as uploaded\n" if $verbose; } sub add_one_built { my $name = shift; my $version = shift; - my $pkg = $db{$name}; + my $pkg = get_source_info($name); if (!defined($pkg)) { print "$name: not registered yet.\n"; return; } - if ($pkg->{'State'} ne "Building" ) { - print "$name: not taken for building (state is $pkg->{'State'}). ", + if (($pkg->{'state'} ne "Building") && ($pkg->{'state'} ne "Build-Attempted")) { + print "$name: not taken for building (state is $pkg->{'state'}). ", "Skipping.\n"; return; } - if ($pkg->{'Builder'} ne $user) { - print "$name: not taken by you, but by $pkg->{'Builder'}. Skipping.\n"; + if ($pkg->{'builder'} ne $user) { + print "$name: not taken by you, but by $pkg->{'builder'}. Skipping.\n"; return; } elsif ( !pkg_version_eq($pkg, $version) ) { print "$name: version mismatch ". - "$(pkg->{'Version'} ". - "by $pkg->{'Builder'})\n"; + "$(pkg->{'version'} ". + "by $pkg->{'builder'})\n"; return; } change_state( \$pkg, 'Built' ); log_ta( $pkg, "--built" ); - $db{$name} = $pkg; + update_source_info($pkg); print "$name: registered as built\n" if $verbose; } sub add_one_uploaded { my $name = shift; my $version = shift; - my $pkg = $db{$name}; + my $pkg = get_source_info($name); if (!defined($pkg)) { print "$name: not registered yet.\n"; return; } - if ($pkg->{'State'} eq "Uploaded" && + if ($pkg->{'state'} eq "Uploaded" && pkg_version_eq($pkg,$version)) { print "$name: already uploaded\n"; return; } - if (!isin( $pkg->{'State'}, qw(Building Built Build-Attempted))) { - print "$name: not taken for building (state is $pkg->{'State'}). ", + if (!isin( $pkg->{'state'}, qw(Building Built Build-Attempted))) { + print "$name: not taken for building (state is $pkg->{'state'}). ", "Skipping.\n"; return; } - if ($pkg->{'Builder'} ne $user) { - print "$name: not taken by you, but by $pkg->{'Builder'}. Skipping.\n"; + if ($pkg->{'builder'} ne $user) { + print "$name: not taken by you, but by $pkg->{'builder'}. Skipping.\n"; return; } # strip epoch -- buildd-uploader used to go based on the filename. # (to remove at some point) my $pkgver; - ($pkgver = $pkg->{'Version'}) =~ s/^\d+://; + ($pkgver = $pkg->{'version'}) =~ s/^\d+://; $version =~ s/^\d+://; # for command line use - if ($pkg->{'Binary-NMU-Version'} ) { - my $nmuver = binNMU_version($pkgver, $pkg->{'Binary-NMU-Version'}); + if ($pkg->{'binary_nmu_version'} ) { + my $nmuver = binNMU_version($pkgver, $pkg->{'binary_nmu_version'}); if (!version_eq( $nmuver, $version )) { print "$name: version mismatch ($nmuver registered). ", "Skipping.\n"; return; } } elsif (!version_eq($pkgver, $version)) { - print "$name: version mismatch ($pkg->{'Version'} registered). ", + print "$name: version mismatch ($pkg->{'version'} registered). ", "Skipping.\n"; return; } change_state( \$pkg, 'Uploaded' ); log_ta( $pkg, "--uploaded" ); - $db{$name} = $pkg; + update_source_info($pkg); print "$name: registered as uploaded\n" if $verbose; } @@ -816,13 +805,13 @@ sub add_one_failed { my $name = shift; my $version = shift; my ($state, $cat); - my $pkg = $db{$name}; + my $pkg = get_source_info($name); if (!defined($pkg)) { print "$name: not registered yet.\n"; return; } - $state = $pkg->{'State'}; + $state = $pkg->{'state'}; if ($state eq "Not-For-Us") { print "$name: not suitable for this architecture anyway. Skipping.\n"; @@ -836,17 +825,17 @@ sub add_one_failed { print "$name: Is already installed in archive. Skipping.\n"; return; } - elsif ($pkg->{'Builder'} && - (($user ne $pkg->{'Builder'}) && - !($pkg->{'Builder'} =~ /^(\w+)-\w+/ && $1 eq $user))) { + elsif ($pkg->{'builder'} && + (($user ne $pkg->{'builder'}) && + !($pkg->{'builder'} =~ /^(\w+)-\w+/ && $1 eq $user))) { print "$name: not taken by you, but by ". - "$pkg->{'Builder'}. Skipping.\n"; + "$pkg->{'builder'}. Skipping.\n"; return; } elsif ( !pkg_version_eq($pkg, $version) ) { print "$name: version mismatch ". - "$(pkg->{'Version'} ". - "by $pkg->{'Builder'})\n"; + "$(pkg->{'version'} ". + "by $pkg->{'builder'})\n"; return; } @@ -885,42 +874,42 @@ sub add_one_failed { } if (($cat eq "reminder-sent" || $cat eq "nmu-offered") && - exists $pkg->{'Failed-Category'} && - $pkg->{'Failed-Category'} ne $cat) { + defined $pkg->{'failed_category'} && + $pkg->{'failed_category'} ne $cat) { (my $action = $cat) =~ s/-/ /; $fail_reason .= "\n$short_date: $action"; } change_state( \$pkg, 'Failed' ); - $pkg->{'Builder'} = $user; - $pkg->{'Failed'} .= "\n" if $pkg->{'Failed'}; - $pkg->{'Failed'} .= $fail_reason; - $pkg->{'Failed-Category'} = $cat if $cat; - if (defined $pkg->{'PermBuildPri'}) { - $pkg->{'BuildPri'} = $pkg->{'PermBuildPri'}; + $pkg->{'builder'} = $user; + $pkg->{'failed'} .= "\n" if $pkg->{'failed'}; + $pkg->{'failed'} .= $fail_reason; + $pkg->{'failed_category'} = $cat if $cat; + if (defined $pkg->{'permbuildpri'}) { + $pkg->{'buildpri'} = $pkg->{'permbuildpri'}; } else { - delete $pkg->{'BuildPri'}; + delete $pkg->{'buildpri'}; } log_ta( $pkg, "--failed" ); - $db{$name} = $pkg; + update_source_info($pkg); print "$name: registered as failed\n" if $verbose; } sub add_one_notforus { my $name = shift; my $version = shift; - my $pkg = $db{$name}; + my $pkg = get_source_info($name); - if ($pkg->{'State'} eq 'Not-For-Us') { + if ($pkg->{'state'} eq 'Not-For-Us') { # reset Not-For-Us state in case it's called twice; this is # the only way to get a package out of this state... # There is no really good state in which such packages should # be put :-( So use Failed for now. change_state( \$pkg, 'Failed' ); - $pkg->{'Package'} = $name; - $pkg->{'Failed'} = "Was Not-For-Us previously"; - delete $pkg->{'Builder'}; - delete $pkg->{'Depends'}; + $pkg->{'package'} = $name; + $pkg->{'failed'} = "Was Not-For-Us previously"; + delete $pkg->{'builder'}; + delete $pkg->{'depends'}; log_ta( $pkg, "--no-build(rev)" ); print "$name: now not unsuitable anymore\n"; @@ -935,12 +924,12 @@ sub add_one_notforus { } else { change_state( \$pkg, 'Not-For-Us' ); - $pkg->{'Package'} = $name; - delete $pkg->{'Builder'}; - delete $pkg->{'Depends'}; - delete $pkg->{'BuildPri'}; - delete $pkg->{'Binary-NMU-Version'}; - delete $pkg->{'Binary-NMU-Changelog'}; + $pkg->{'package'} = $name; + delete $pkg->{'builder'}; + delete $pkg->{'depends'}; + delete $pkg->{'buildpri'}; + delete $pkg->{'binary_nmu_version'}; + delete $pkg->{'binary_nmu_changelog'}; log_ta( $pkg, "--no-build" ); print "$name: registered as unsuitable\n" if $verbose; @@ -953,38 +942,46 @@ sub add_one_notforus { "the Not-For-Us state is wrong.\n" ) if $conf::notforus_maint; } - $db{$name} = $pkg; + update_source_info($pkg); } sub add_one_needsbuild { my $name = shift; my $version = shift; my $state; - my $pkg = $db{$name}; + my $pkg = get_source_info($name); if (!defined($pkg)) { print "$name: not registered; can't give back.\n"; return; } - $state = $pkg->{'State'}; + $state = $pkg->{'state'}; - if ($state eq "Dep-Wait") { + if ($state eq "BD-Uninstallable") { if ($opt_override) { - print "$name: Forcing source dependency list to be cleared\n"; + print "$name: Forcing uninstallability mark to be removed. This is not permanent and might be reset with the next trigger run\n"; + + change_state( \$pkg, 'Needs-Build' ); + delete $pkg->{'builder'}; + delete $pkg->{'depends'}; + log_ta( $pkg, "--give-back" ); + update_source_info($pkg); + print "$name: given back\n" if $verbose; + return; } else { - print "$name: waiting for source dependencies. Skipping\n", + print "$name: has uninstallable build-dependencies. Skipping\n", " (use --override to clear dependency list and ", "give back anyway)\n"; return; } } - if ($state eq "BD-Uninstallable") { + elsif ($state eq "Dep-Wait") { if ($opt_override) { - print "$name: Forcing uninstallability mark to be removed\n"; + print "$name: Forcing source dependency list to be cleared\n"; } else { - print "$name: has uninstallable build-dependencies. Skipping\n", + print "$name: waiting for source dependencies. Skipping\n", " (use --override to clear dependency list and ", "give back anyway)\n"; return; @@ -1000,63 +997,69 @@ sub add_one_needsbuild { return; } } - if (defined ($pkg->{'Builder'}) && $user ne $pkg->{'Builder'} && - !($pkg->{'Builder'} =~ /^(\w+)-\w+/ && $1 eq $user)) { + if (defined ($pkg->{'builder'}) && $user ne $pkg->{'builder'} && + !($pkg->{'builder'} =~ /^(\w+)-\w+/ && $1 eq $user) && + !$opt_override) { print "$name: not taken by you, but by ". - "$pkg->{'Builder'}. Skipping.\n"; + "$pkg->{'builder'}. Skipping.\n"; return; } if (!pkg_version_eq($pkg, $version)) { - print "$name: version mismatch ($pkg->{'Version'} registered). ", + print "$name: version mismatch ($pkg->{'version'} registered). ", "Skipping.\n"; return; } - change_state( \$pkg, 'Needs-Build' ); - delete $pkg->{'Builder'}; - delete $pkg->{'Depends'}; + if ($distribution eq "unstable") { + change_state( \$pkg, 'BD-Uninstallable' ); + $pkg->{'bd_problem'} = "Installability of build dependencies not tested yet"; + } else { + change_state( \$pkg, 'Needs-Build' ); + } + $pkg->{'builder'} = undef; + $pkg->{'depends'} = undef; log_ta( $pkg, "--give-back" ); - $db{$name} = $pkg; + update_source_info($pkg); print "$name: given back\n" if $verbose; } sub set_one_binnmu { my $name = shift; my $version = shift; - my $pkg = $db{$name}; + my $pkg = get_source_info($name); my $state; if (!defined($pkg)) { print "$name: not registered; can't register for binNMU.\n"; return; } - my $db_ver = $pkg->{'Version'}; + my $db_ver = $pkg->{'version'}; if (!version_eq($db_ver, $version)) { print "$name: version mismatch ($db_ver registered). ", "Skipping.\n"; return; } - $state = $pkg->{'State'}; + $state = $pkg->{'state'}; - if (defined $pkg->{'Binary-NMU-Version'}) { + if (defined $pkg->{'binary_nmu_version'}) { if ($binNMUver == 0) { change_state( \$pkg, 'Installed' ); - delete $pkg->{'Builder'}; - delete $pkg->{'Depends'}; - delete $pkg->{'Binary-NMU-Version'}; - delete $pkg->{'Binary-NMU-Changelog'}; - } elsif ($binNMUver <= $pkg->{'Binary-NMU-Version'}) { - print "$name: already building binNMU $pkg->{'Binary-NMU-Version'}\n"; + delete $pkg->{'builder'}; + delete $pkg->{'depends'}; + delete $pkg->{'binary_nmu_version'}; + delete $pkg->{'binary_nmu_changelog'}; + } elsif ($binNMUver <= $pkg->{'binary_nmu_version'}) { + print "$name: already building binNMU $pkg->{'binary_nmu_version'}\n"; return; } else { - $pkg->{'Binary-NMU-Version'} = $binNMUver; - $pkg->{'Binary-NMU-Changelog'} = $fail_reason; - $pkg->{'Notes'} = 'out-of-date'; - $pkg->{'BuildPri'} = $pkg->{'PermBuildPri'} - if (defined $pkg->{'PermBuildPri'}); + $pkg->{'binary_nmu_version'} = $binNMUver; + $pkg->{'binary_nmu_changelog'} = $fail_reason; + $pkg->{'notes'} = 'out-of-date'; + $pkg->{'buildpri'} = $pkg->{'permbuildpri'} + if (defined $pkg->{'permbuildpri'}); } log_ta( $pkg, "--binNMU" ); - $db{$name} = $pkg; + update_source_info($pkg); return; } elsif ($binNMUver == 0) { print "${name}_$version: no scheduled binNMU to cancel.\n"; @@ -1069,20 +1072,27 @@ sub set_one_binnmu { } my $fullver = binNMU_version($version,$binNMUver); - if ( version_lesseq( $fullver, $pkg->{'Installed-Version'} ) ) + if ( version_lesseq( $fullver, $pkg->{'installed_version'} ) ) { - print "$name: binNMU $fullver is not newer than current version $pkg->{'Installed-Version'}\n"; + print "$name: binNMU $fullver is not newer than current version $pkg->{'installed_version'}\n"; return; } - change_state( \$pkg, 'Needs-Build' ); - delete $pkg->{'Builder'}; - delete $pkg->{'Depends'}; - $pkg->{'Binary-NMU-Version'} = $binNMUver; - $pkg->{'Binary-NMU-Changelog'} = $fail_reason; - $pkg->{'Notes'} = 'out-of-date'; + if ($distribution eq "unstable") { + change_state( \$pkg, 'BD-Uninstallable' ); + $pkg->{'bd_problem'} = "Installability of build dependencies not tested yet"; + } + else + { + change_state( \$pkg, 'Needs-Build' ); + } + delete $pkg->{'builder'}; + delete $pkg->{'depends'}; + $pkg->{'binary_nmu_version'} = $binNMUver; + $pkg->{'binary_nmu_changelog'} = $fail_reason; + $pkg->{'notes'} = 'out-of-date'; log_ta( $pkg, "--binNMU" ); - $db{$name} = $pkg; + update_source_info($pkg); print "${name}: registered for binNMU $fullver\n" if $verbose; } @@ -1090,14 +1100,14 @@ sub set_one_buildpri { my $name = shift; my $version = shift; my $key = shift; - my $pkg = $db{$name}; + my $pkg = get_source_info($name); my $state; if (!defined($pkg)) { print "$name: not registered; can't set priority.\n"; return; } - $state = $pkg->{'State'}; + $state = $pkg->{'state'}; if ($state eq "Not-For-Us") { print "$name: not suitable for this architecture. Skipping.\n"; @@ -1107,21 +1117,21 @@ sub set_one_buildpri { return; } if (!pkg_version_eq($pkg, $version)) { - print "$name: version mismatch ($pkg->{'Version'} registered). ", + print "$name: version mismatch ($pkg->{'version'} registered). ", "Skipping.\n"; return; } if ( $build_priority == 0 ) { - delete $pkg->{'BuildPri'} - if $key eq 'PermBuildPri' and defined $pkg->{'BuildPri'} - and $pkg->{'BuildPri'} == $pkg->{$key}; + delete $pkg->{'buildpri'} + if $key eq 'permbuildpri' and defined $pkg->{'buildpri'} + and $pkg->{'buildpri'} == $pkg->{$key}; delete $pkg->{$key}; } else { - $pkg->{'BuildPri'} = $build_priority - if $key eq 'PermBuildPri'; + $pkg->{'buildpri'} = $build_priority + if $key eq 'permbuildpri'; $pkg->{$key} = $build_priority; } - $db{$name} = $pkg; + update_source_info($pkg); print "$name: set to build priority $build_priority\n" if $verbose; } @@ -1129,13 +1139,13 @@ sub add_one_depwait { my $name = shift; my $version = shift; my $state; - my $pkg = $db{$name}; + my $pkg = get_source_info($name); if (!defined($pkg)) { print "$name: not registered yet.\n"; return; } - $state = $pkg->{'State'}; + $state = $pkg->{'state'}; if ($state eq "Dep-Wait") { print "$name: merging with previously registered dependencies\n"; @@ -1161,16 +1171,16 @@ sub add_one_depwait { print "$name: Is already uploaded. Skipping.\n"; return; } - elsif ($pkg->{'Builder'} && - $user ne $pkg->{'Builder'}) { + elsif ($pkg->{'builder'} && + $user ne $pkg->{'builder'}) { print "$name: not taken by you, but by ". - "$pkg->{'Builder'}. Skipping.\n"; + "$pkg->{'builder'}. Skipping.\n"; return; } elsif ( !pkg_version_eq($pkg,$version)) { print "$name: version mismatch ". - "($pkg->{'Version'} ". - "by $pkg->{'Builder'})\n"; + "($pkg->{'version'} ". + "by $pkg->{'builder'})\n"; return; } elsif ($fail_reason =~ /^\s*$/ || @@ -1179,21 +1189,21 @@ sub add_one_depwait { return; } change_state( \$pkg, 'Dep-Wait' ); - $pkg->{'Builder'} = $user; - if (defined $pkg->{'PermBuildPri'}) { - $pkg->{'BuildPri'} = $pkg->{'PermBuildPri'}; + $pkg->{'builder'} = $user; + if (defined $pkg->{'permbuildpri'}) { + $pkg->{'buildpri'} = $pkg->{'permbuildpri'}; } else { - delete $pkg->{'BuildPri'}; + delete $pkg->{'buildpri'}; } - my $deplist = parse_deplist( $pkg->{'Depends'} ); + my $deplist = parse_deplist( $pkg->{'depends'} ); my $new_deplist = parse_deplist( $fail_reason ); # add new dependencies, maybe overwriting old entries foreach (keys %$new_deplist) { $deplist->{$_} = $new_deplist->{$_}; } - $pkg->{'Depends'} = build_deplist($deplist); + $pkg->{'depends'} = build_deplist($deplist); log_ta( $pkg, "--dep-wait" ); - $db{$name} = $pkg; + update_source_info($pkg); print "$name: registered as waiting for dependencies\n" if $verbose; } @@ -1204,6 +1214,8 @@ sub parse_sources { my $name; my $full = shift; + my $db = get_all_source_info(); + local($/) = ""; # read in paragraph mode while( <> ) { my( $version, $arch, $section, $priority, $builddep, $buildconf, $binaries ); @@ -1222,41 +1234,38 @@ sub parse_sources { $pkgs{$name}{'ver'} = $version; $pkgs{$name}{'bin'} = $binaries; - my $pkg = $db{$name}; + $pkgs{$name}{'dep'} = $builddep; + $pkgs{$name}{'conf'} = $buildconf; + my $pkg = $db->{$name}; if (defined $pkg) { my $change = 0; - if ($arch eq "all" && !version_less( $version, $pkg->{'Version'} )) { + if ($arch eq "all" && !version_less( $version, $pkg->{'version'} )) { # package is now Arch: all, delete it from db change_state( \$pkg, 'deleted' ); log_ta( $pkg, "--merge-sources" ); - print "$name ($pkg->{'Version'}): deleted ". + print "$name ($pkg->{'version'}): deleted ". "from database, because now Arch: all\n" if $verbose; - delete $db{$name}; + del_source_info($name); + delete $db->{$name}; next; } # The "Version" should always be the source version -- # not a possible binNMU version number. - $pkg->{'Version'} = $version, $change++ - if ($pkg->{'State'} eq 'Installed' and - !version_eq( $pkg->{'Version'}, $version)); + $pkg->{'version'} = $version, $change++ + if ($pkg->{'state'} eq 'Installed' and + !version_eq( $pkg->{'version'}, $version)); # Always update priority and section, if available - $pkg->{'Priority'} = $priority, $change++ - if defined $priority and (not defined($pkg->{'Priority'}) or $pkg->{'Priority'} ne $priority); + $pkg->{'priority'} = $priority, $change++ + if defined $priority and (not defined($pkg->{'priority'}) or $pkg->{'priority'} ne $priority); - $pkg->{'Section'} = $section, $change++ - if defined $section and (not defined($pkg->{'Section'}) or $pkg->{'Section'} ne $section); + $pkg->{'section'} = $section, $change++ + if defined $section and (not defined($pkg->{'section'}) or $pkg->{'section'} ne $section); - # Store Build-Deps and Build-Conflicts for edos-debcheck later - $pkg->{'Build-Depends'} = $builddep, $change++ - if ($pkg->{'Build-Depends'} ne $builddep); - $pkg->{'Build-Conflicts'} = $builddep, $change++ - if ($pkg->{'Build-Conflicts'} ne $buildconf); - - $db{$name} = $pkg if $change; + update_source_info($pkg) if $change; } } # Now that we only have the latest source version, build the list @@ -1268,29 +1277,31 @@ sub parse_sources { } # remove installed packages that no longer have source available # or binaries installed - foreach $name (keys %db) { + foreach $name (keys %$db) { next if $name =~ /^_/; - my $pkg = $db{$name}; + my $pkg = $db->{$name}; if (not defined($pkgs{$name})) { change_state( \$pkg, 'deleted' ); log_ta( $pkg, "--merge-sources" ); - print "$name ($pkg->{'Version'}): ". + print "$name ($pkg->{'version'}): ". "deleted from database, because ". "not in Sources anymore\n" if $verbose; - delete $db{$name}; + del_source_info($name); + delete $db->{$name}; } else { - next if !isin( $pkg->{'State'}, qw(Installed) ); + next if !isin( $pkg->{'state'}, qw(Installed) ); if ($full && not defined $merge_srcvers{$name}) { change_state( \$pkg, 'deleted' ); log_ta( $pkg, "--merge-sources" ); - print "$name ($pkg->{'Version'}): ". + print "$name ($pkg->{'version'}): ". "deleted from database, because ". "binaries don't exist anymore\n" if $verbose; - delete $db{$name}; - } elsif ($full && version_less( $merge_srcvers{$name}, $pkg->{'Version'})) { - print "$name ($pkg->{'Version'}): ". + del_source_info($name); + delete $db->{$name}; + } elsif ($full && version_less( $merge_srcvers{$name}, $pkg->{'version'})) { + print "$name ($pkg->{'version'}): ". "package is Installed but binaries are from ". $merge_srcvers{$name}. "\n" if $verbose; @@ -1303,8 +1314,10 @@ sub parse_sources { # This function looks through a Packages file and sets the state of # packages to 'Installed' sub parse_packages { + my $depwait_only = shift; my $installed; + my $pkgs = get_all_source_info(); local($/) = ""; # read in paragraph mode while( <> ) { my( $name, $version, $depends, $source, $sourcev, $architecture, $provides, $binaryv, $binnmu ); @@ -1317,17 +1330,18 @@ sub parse_packages { /^Provides:\s*(.*)$/mi and $provides = $1; next if !$name || !$version; next if ($arch ne $architecture and $architecture ne "all"); - next if (defined ($installed->{$name}) and $installed->{$name}{'Version'} ne "" and - version_lesseq( $version, $installed->{$name}{'Version'} )); - $installed->{$name}{'Version'} = $version; - $installed->{$name}{'Depends'} = $depends; + next if (defined ($installed->{$name}) and $installed->{$name}{'version'} ne "" and + version_lesseq( $version, $installed->{$name}{'version'} )); + $installed->{$name}{'version'} = $version; + next if $depwait_only; + $installed->{$name}{'depends'} = $depends; $installed->{$name}{'all'} = 1 if $architecture eq "all"; undef $installed->{$name}{'Provider'}; $installed->{$name}{'Source'} = $source ? $source : $name; if ($provides) { foreach (split( /\s*,\s*/, $provides )) { if (not defined ($installed->{$_})) { - $installed->{$_}{'Version'} = ""; + $installed->{$_}{'version'} = ""; $installed->{$_}{'Provider'} = $name; } } @@ -1346,63 +1360,63 @@ sub parse_packages { next if defined($merge_srcvers{$name}) and $merge_srcvers{$name} eq $version; $merge_srcvers{$name} = $version; - my $pkg = $db{$name}; + my $pkg = $pkgs->{$name}; if (defined $pkg) { - if (isin( $pkg->{'State'}, qw(Not-For-Us)) || - (isin($pkg->{'State'}, qw(Installed)) && - version_lesseq($binaryv, $pkg->{'Installed-Version'}))) { - print "Skipping $name because State == $pkg->{'State'}\n" + if (isin( $pkg->{'state'}, qw(Not-For-Us)) || + (isin($pkg->{'state'}, qw(Installed)) && + version_lesseq($binaryv, $pkg->{'installed_version'}))) { + print "Skipping $name because State == $pkg->{'state'}\n" if $verbose >= 2; next; } - if ($pkg->{'Binary-NMU-Version'} ) { - my $nmuver = binNMU_version($pkg->{'Version'}, $pkg->{'Binary-NMU-Version'}); + if ($pkg->{'binary_nmu_version'} ) { + my $nmuver = binNMU_version($pkg->{'version'}, $pkg->{'binary_nmu_version'}); if (version_less( $binaryv, $nmuver )) { print "Skipping $name ($version) because have newer ". "version ($nmuver) in db.\n" if $verbose >= 2; next; } - } elsif (version_less($version, $pkg->{'Version'})) { + } elsif (version_less($version, $pkg->{'version'})) { print "Skipping $name ($version) because have newer ". - "version ($pkg->{'Version'}) in db.\n" + "version ($pkg->{'version'}) in db.\n" if $verbose >= 2; next; } if (!pkg_version_eq($pkg, $version) && - $pkg->{'State'} ne "Installed") { + $pkg->{'state'} ne "Installed") { warn "Warning: $name: newer version than expected appeared ". - "in archive ($version vs. $pkg->{'Version'})\n"; - delete $pkg->{'Builder'}; + "in archive ($version vs. $pkg->{'version'})\n"; + delete $pkg->{'builder'}; } - if (!isin( $pkg->{'State'}, qw(Uploaded) )) { + if (!isin( $pkg->{'state'}, qw(Uploaded) )) { warn "Warning: Package $name was not in uploaded state ". - "before (but in '$pkg->{'State'}').\n"; - delete $pkg->{'Builder'}; - delete $pkg->{'Depends'}; + "before (but in '$pkg->{'state'}').\n"; + delete $pkg->{'builder'}; + delete $pkg->{'depends'}; } } else { $pkg = {}; - $pkg->{'Version'} = $version; + $pkg->{'version'} = $version; } change_state( \$pkg, 'Installed' ); - $pkg->{'Package'} = $name; - $pkg->{'Installed-Version'} = $binaryv; - if (defined $pkg->{'PermBuildPri'}) { - $pkg->{'BuildPri'} = $pkg->{'PermBuildPri'}; + $pkg->{'package'} = $name; + $pkg->{'installed_version'} = $binaryv; + if (defined $pkg->{'permbuildpri'}) { + $pkg->{'buildpri'} = $pkg->{'permbuildpri'}; } else { - delete $pkg->{'BuildPri'}; + delete $pkg->{'buildpri'}; } - $pkg->{'Version'} = $version - if version_less( $pkg->{'Version'}, $version); - delete $pkg->{'Binary-NMU-Version'}; - delete $pkg->{'Binary-NMU-Changelog'}; + $pkg->{'version'} = $version + if version_less( $pkg->{'version'}, $version); + delete $pkg->{'binary_nmu_version'}; + delete $pkg->{'binary_nmu_changelog'}; log_ta( $pkg, "--merge-packages" ); - $db{$name} = $pkg; + update_source_info($pkg); print "$name ($version) is up-to-date now.\n" if $verbose; } @@ -1425,7 +1439,7 @@ sub pretend_avail { "(bad format)\n"; next; } - $installed->{$name}{'Version'} = $version; + $installed->{$name}{'version'} = $version; } check_dep_wait( "--pretend-avail", $installed ); @@ -1438,11 +1452,12 @@ sub check_dep_wait { # check all packages in state Dep-Wait if dependencies are all # available now my $name; - foreach $name (keys %db) { + my $db = get_all_source_info(); + foreach $name (keys %$db) { next if $name =~ /^_/; - my $pkg = $db{$name}; - next if $pkg->{'State'} ne "Dep-Wait"; - my $deps = $pkg->{'Depends'}; + my $pkg = $db->{$name}; + next if $pkg->{'state'} ne "Dep-Wait"; + my $deps = $pkg->{'depends'}; if (!$deps) { print "$name: was in state Dep-Wait, but with empty ", "dependencies!\n"; @@ -1454,10 +1469,10 @@ sub check_dep_wait { my @removed_deps; foreach (keys %$deplist) { if (!exists $installed->{$_} || - ($deplist->{$_}->{'Rel'} && $deplist->{$_}->{'Version'} && - !version_compare( $installed->{$_}{'Version'}, - $deplist->{$_}->{'Rel'}, - $deplist->{$_}->{'Version'}))) { + ($deplist->{$_}->{'rel'} && $deplist->{$_}->{'version'} && + !version_compare( $installed->{$_}{'version'}, + $deplist->{$_}->{'rel'}, + $deplist->{$_}->{'version'}))) { $allok = 0; $new_deplist->{$_} = $deplist->{$_}; } @@ -1469,19 +1484,19 @@ sub check_dep_wait { make_needs_build: change_state( \$pkg, 'Needs-Build' ); log_ta( $pkg, $action ); - delete $pkg->{'Builder'}; - delete $pkg->{'Depends'}; - print "$name ($pkg->{'Version'}) has all ", + delete $pkg->{'builder'}; + delete $pkg->{'depends'}; + print "$name ($pkg->{'version'}) has all ", "dependencies available now\n" if $verbose; $new_vers{$name}++; - $db{$name} = $pkg; + update_source_info($pkg); } elsif (@removed_deps) { - $pkg->{'Depends'} = build_deplist( $new_deplist ); - print "$name ($pkg->{'Version'}): some dependencies ", + $pkg->{'depends'} = build_deplist( $new_deplist ); + print "$name ($pkg->{'version'}): some dependencies ", "(@removed_deps) available now, but not all yet\n" if $verbose; - $db{$name} = $pkg; + update_source_info($pkg); } } } @@ -1494,6 +1509,8 @@ sub parse_quinn_diff { my %quinn_pkgs; my $dubious = ""; + my $pkgs = get_all_source_info(); + while( <> ) { my $change = 0; next if !m,^([-\w\d/]*)/ # section @@ -1508,69 +1525,69 @@ sub parse_quinn_diff { $priority = "unknown" if $priority eq "-"; $priority = "standard" if ($name eq "debian-installer"); - my $pkg = $db{$name}; + my $pkg = $pkgs->{$name}; # Always update section and priority. if (defined($pkg)) { - $pkg->{'Section'} = $section, $change++ if not defined - $pkg->{'Section'} or $section ne "unknown"; - $pkg->{'Priority'} = $priority, $change++ if not defined - $pkg->{'Priority'} or $priority ne "unknown"; + $pkg->{'section'} = $section, $change++ if not defined + $pkg->{'section'} or $section ne "unknown"; + $pkg->{'priority'} = $priority, $change++ if not defined + $pkg->{'priority'} or $priority ne "unknown"; } if (defined($pkg) && - $pkg->{'State'} =~ /^Dep-Wait/ && - version_less( $pkg->{'Version'}, $version )) { + $pkg->{'state'} =~ /^Dep-Wait/ && + version_less( $pkg->{'version'}, $version )) { change_state( \$pkg, 'Dep-Wait' ); - $pkg->{'Version'} = $version; - delete $pkg->{'Binary-NMU-Version'}; - delete $pkg->{'Binary-NMU-Changelog'}; + $pkg->{'version'} = $version; + delete $pkg->{'binary_nmu_version'}; + delete $pkg->{'binary_nmu_changelog'}; log_ta( $pkg, "--merge-quinn" ); $change++; print "$name ($version) still waiting for dependencies.\n" if $verbose; } elsif (defined($pkg) && - $pkg->{'State'} =~ /-Removed$/ && - version_eq($pkg->{'Version'}, $version)) { + $pkg->{'state'} =~ /-Removed$/ && + version_eq($pkg->{'version'}, $version)) { # reinstantiate a package that has been removed earlier # (probably due to a quinn-diff malfunction...) - my $newstate = $pkg->{'State'}; + my $newstate = $pkg->{'state'}; $newstate =~ s/-Removed$//; change_state( \$pkg, $newstate ); - $pkg->{'Version'} = $version; - $pkg->{'Notes'} = $notes; + $pkg->{'version'} = $version; + $pkg->{'notes'} = $notes; log_ta( $pkg, "--merge-quinn" ); $change++; print "$name ($version) reinstantiated to $newstate.\n" if $verbose; } elsif (defined($pkg) && - $pkg->{'State'} eq "Not-For-Us" && - version_less( $pkg->{'Version'}, $version )) { + $pkg->{'state'} eq "Not-For-Us" && + version_less( $pkg->{'version'}, $version )) { # for Not-For-Us packages just update the version etc., but # keep the state change_state( \$pkg, "Not-For-Us" ); - $pkg->{'Package'} = $name; - $pkg->{'Version'} = $version; - $pkg->{'Notes'} = $notes; - delete $pkg->{'Builder'}; + $pkg->{'package'} = $name; + $pkg->{'version'} = $version; + $pkg->{'notes'} = $notes; + delete $pkg->{'builder'}; log_ta( $pkg, "--merge-quinn" ); $change++; print "$name ($version) still Not-For-Us.\n" if $verbose; } elsif (!defined($pkg) || - $pkg->{'State'} ne "Not-For-Us" && - (version_less( $pkg->{'Version'}, $version ) || - ($pkg->{'State'} eq "Installed" && version_less($pkg->{'Installed-Version'}, $version)))) { + $pkg->{'state'} ne "Not-For-Us" && + (version_less( $pkg->{'version'}, $version ) || + ($pkg->{'state'} eq "Installed" && version_less($pkg->{'installed_version'}, $version)))) { - if (defined( $pkg->{'State'} ) && isin( $pkg->{'State'}, qw(Building Built Build-Attempted))) { - send_mail( $pkg->{'Builder'}, + if (defined( $pkg->{'state'} ) && isin( $pkg->{'state'}, qw(Building Built Build-Attempted))) { + send_mail( $pkg->{'builder'}, "new version of $name (dist=$distribution)", "As far as I'm informed, you're currently ". "building the package $name\n". - "in version $pkg->{'Version'}.\n\n". + "in version $pkg->{'version'}.\n\n". "Now there's a new source version $version. ". "If you haven't finished\n". "compiling $name yet, you can stop it to ". @@ -1578,53 +1595,54 @@ sub parse_quinn_diff { "Just to inform you...\n". "(This is an automated message)\n" ); print "$name: new version ($version) while building ". - "$pkg->{'Version'} -- sending mail ". - "to builder ($pkg->{'Builder'})\n" + "$pkg->{'version'} -- sending mail ". + "to builder ($pkg->{'builder'})\n" if $verbose; } change_state( \$pkg, 'Needs-Build' ); - $pkg->{'Package'} = $name; - $pkg->{'Version'} = $version; - $pkg->{'Section'} = $section; - $pkg->{'Priority'} = $priority; - $pkg->{'Notes'} = $notes; - delete $pkg->{'Builder'}; - delete $pkg->{'Binary-NMU-Version'}; - delete $pkg->{'Binary-NMU-Changelog'}; + $pkg->{'package'} = $name; + $pkg->{'version'} = $version; + $pkg->{'section'} = $section; + $pkg->{'priority'} = $priority; + $pkg->{'notes'} = $notes; + delete $pkg->{'builder'}; + delete $pkg->{'binary_nmu_version'}; + delete $pkg->{'binary_nmu_changelog'}; log_ta( $pkg, "--merge-quinn" ); $new_vers{$name}++; $change++; print "$name ($version) needs rebuilding now.\n" if $verbose; } elsif (defined($pkg) && - !version_eq( $pkg->{'Version'}, $version ) && - isin( $pkg->{'State'}, qw(Installed Not-For-Us) )) { + !version_eq( $pkg->{'version'}, $version ) && + isin( $pkg->{'state'}, qw(Installed Not-For-Us) )) { print "$name: skipping because version in db ". - "($pkg->{'Version'}) is >> than ". + "($pkg->{'version'}) is >> than ". "what quinn-diff says ($version) ". - "(state is $pkg->{'State'})\n" + "(state is $pkg->{'state'})\n" if $verbose; - $dubious .= "$pkg->{'State'}: ". - "db ${name}_$pkg->{'Version'} >> ". + $dubious .= "$pkg->{'state'}: ". + "db ${name}_$pkg->{'version'} >> ". "quinn $version\n" if !$partial; } elsif ($verbose >= 2) { - if ($pkg->{'State'} eq "Not-For-Us") { + if ($pkg->{'state'} eq "Not-For-Us") { print "Skipping $name because State == ". - "$pkg->{'State'}\n"; + "$pkg->{'state'}\n"; } - elsif (!version_less($pkg->{'Version'}, $version)) { + elsif (!version_less($pkg->{'version'}, $version)) { print "Skipping $name because version in db ". - "($pkg->{'Version'}) is >= than ". + "($pkg->{'version'}) is >= than ". "what quinn-diff says ($version)\n"; } } - $db{$name} = $pkg if $change; + update_source_info($pkg) if $change; } if ($dubious) { send_mail( $conf::db_maint, - "Dubious versions in $distribution $conf::dbbase database", + "Dubious versions in " . table_name() . " " + . $distribution . " table", "The following packages have a newer version in the ". "wanna-build database\n". "than what quinn-diff says, and this is strange for ". @@ -1640,49 +1658,34 @@ sub parse_quinn_diff { # anymore by quinn-diff. if ( !$partial ) { my $name; - foreach $name (keys %db) { + my $db = get_all_source_info(); + foreach $name (keys %$db) { next if $name =~ /^_/; - my $pkg = $db{$name}; - next if defined $pkg->{'Binary-NMU-Version'}; - next if !isin( $pkg->{'State'}, + my $pkg = $db->{$name}; + next if defined $pkg->{'binary_nmu_version'}; + next if !isin( $pkg->{'state'}, qw(Needs-Build Building Built Build-Attempted Uploaded Failed Dep-Wait BD-Uninstallable) ); - my $virtual_delete = $pkg->{'State'} eq 'Failed'; + my $virtual_delete = $pkg->{'state'} eq 'Failed'; if (!$quinn_pkgs{$name}) { change_state( \$pkg, $virtual_delete ? - $pkg->{'State'}."-Removed" : + $pkg->{'state'}."-Removed" : 'deleted' ); log_ta( $pkg, "--merge-quinn" ); - print "$name ($pkg->{'Version'}): ". + print "$name ($pkg->{'version'}): ". ($virtual_delete ? "(virtually) " : "") . "deleted ". "from database, because not in quinn-diff anymore\n" if $verbose; if ($virtual_delete) { - $db{$name} = $pkg; + update_source_info($pkg); } else { - delete $db{$name}; + del_source_info($name); } } } } } -sub send_reupload_mail { - my $to = shift; - my $pkg = shift; - my $version = shift; - my $dist = shift; - my $other_dist = shift; - - send_mail( $to, - "Please reupload ${pkg}_${'Version'} for $dist", - "You have recently built (or are currently building)\n". - "${pkg}_${'Version'} for $other_dist.\n". - "This version is now also needed in the $dist distribution.\n". - "Please reupload the files now present in the Debian archive\n". - "(best with buildd-reupload).\n" ); -} - # for sorting priorities and sections BEGIN { @@ -1765,67 +1768,148 @@ BEGIN { } sub sort_list_func { - my( $letter, $x, $ax, $bx ); - - foreach $letter (split( "", $list_order )) { - SWITCH: foreach ($letter) { - /P/ && do { - $x = $b->{'BuildPri'} <=> $a->{'BuildPri'}; - return $x if $x != 0; - last SWITCH; - }; - /p/ && do { - $x = $prioval{$a->{'Priority'}} <=> $prioval{$b->{'Priority'}}; - return $x if $x != 0; - last SWITCH; - }; - /s/ && do { - $x = $sectval{$a->{'Section'}} <=> $sectval{$b->{'Section'}}; - return $x if $x != 0; - last SWITCH; - }; - /n/ && do { - $x = $a->{'Package'} cmp $b->{'Package'}; - return $x if $x != 0; - last SWITCH; - }; - /b/ && do { - $x = $a->{'Builder'} cmp $b->{'Builder'}; - return $x if $x != 0; - last SWITCH; - }; - /c/ && do { - $ax = ($a->{'Notes'} =~ /^(out-of-date|partial)/) ? 0 : - ($a->{'Notes'} =~ /^uncompiled/) ? 2 : 1; - $bx = ($b->{'Notes'} =~ /^(out-of-date|partial)/) ? 0 : - ($b->{'Notes'} =~ /^uncompiled/) ? 2 : 1; - $x = $ax <=> $bx; - return $x if $x != 0; - last SWITCH; - }; - /f/ && do { - my $ca = exists $a->{'Failed-Category'} ? - $a->{'Failed-Category'} : "none"; - my $cb = exists $b->{'Failed-Category'} ? - $b->{'Failed-Category'} : "none"; - $x = $catval{$ca} <=> $catval{$cb}; - return $x if $x != 0; - last SWITCH; - }; - /S/ && do { - my $pa = $prioval{$a->{'Priority'}} > - $prioval{'standard'}; - my $pb = $prioval{$b->{'Priority'}} > - $prioval{'standard'}; - $x = $pa <=> $pb; - return $x if $x != 0; - last SWITCH; - }; - } + my $map_funcs = { + 'C' => ['<=>', sub { return (-1) * $_[0]->{'calprio'}; }], + 'W' => ['<=>', sub { return (-1) * $_[0]->{'state_days'}; }], + 'P' => ['<=>', sub { return (-1) * $_[0]->{'buildpri'}; }], + 'p' => ['<=>', sub { return $prioval{$_[0]->{'priority'}}; }], + 's' => ['<=>', sub { return $sectval{$_[0]->{'section'}}; }], + 'n' => ['cmp', sub { return $_[0]->{'package'}; }], + 'b' => ['cmp', sub { return $_[0]->{'builder'}; }], + 'c' => ['<=>', sub { return ($_[0]->{'notes'} =~ /^(out-of-date|partial)/) ? 0: ($_[0]->{'notes'} =~ /^uncompiled/) ? 2 : 1; }], + 'f' => ['<=>', sub { return $catval{ $_[0]->{'failed_category'} ? $_[0]->{'failed_category'}: "none" }; }], + 'S' => ['<=>', sub { return $prioval{$_[0]->{'priority'}} > $prioval{'standard'}; }], + }; + + foreach my $letter (split( //, $list_order )) { + my $r; + $r = &{$map_funcs->{$letter}[1]}($a) <=> &{$map_funcs->{$letter}[1]}($b) if $map_funcs->{$letter}[0] eq '<=>'; + $r = &{$map_funcs->{$letter}[1]}($a) cmp &{$map_funcs->{$letter}[1]}($b) if $map_funcs->{$letter}[0] eq 'cmp'; + return $r if $r != 0; } return 0; } +sub calculate_prio { + my $priomap = $yamlmap->{priority}; + my $pkg = shift; + my @s=split("/", $pkg->{'section'}); + $pkg->{'component'} = $s[0] if $s[1]; + $pkg->{'component'} ||= 'main'; + $pkg->{'calprio'} = 0; + foreach my $k (keys %{$priomap->{keys}}) { + $pkg->{'calprio'} += $priomap->{keys}->{$k}{$pkg->{$k}} if $pkg->{$k} and $priomap->{keys}->{$k}{$pkg->{$k}}; + } + + my $days = $pkg->{'state_days'}; + $days = $priomap->{'waitingdays'}->{'min'} if $priomap->{'waitingdays'}->{'min'} and $days < $priomap->{'waitingdays'}->{'min'}; + $days = $priomap->{'waitingdays'}->{'max'} if $priomap->{'waitingdays'}->{'max'} and $days > $priomap->{'waitingdays'}->{'max'}; + my $scale = $priomap->{'waitingdays'}->{'scale'} || 1; + $pkg->{'calprio'} += $days * $scale; + + $pkg->{'calprio'} += $pkg->{'permbuildpri'} if $pkg->{'permbuildpri'}; + $pkg->{'calprio'} += $pkg->{'buildpri'} if $pkg->{'buildpri'}; + + return $pkg; +} + + +sub use_fmt { + my $r; + + if (ref($_[0]) eq 'CODE') { + $r = &{$_[0]}; + } else { + $r = $_[0]; + } + + shift; + my $t = shift; + + $r ||= ""; + return $r unless $t; + + my $pkg = shift; + my $var = shift; + if (substr($t,0,1) eq '!') { + $t = substr($t,1); + return "" if $r; + } else { + return "" unless $r; + } + if ($t =~ /%/) { + return print_format($t, $pkg, $var); + } + return $t; +} +sub make_fmt { my $c = shift; my $pkg = shift; my $var = shift; return sub { use_fmt($c, $_[0], $pkg, $var); } }; + +sub print_format { + my $printfmt = shift; + my $pkg = shift; + my $var = shift; +=pod +Within an format string, the following values are allowed (need to be preceded by %). +This can be combined to e.g. +wanna-build --format='wanna-build -A %a --give-back %p_%v' -A mipsel --list=failed + +a Architecture +c section (e.g. libs or utils) +D in case of BD-Uninstallable the reason for the uninstallability +d distribution +E in case of Dep-Wait the packages being waited on, in case of Needs-Build the number in the queue +F in case of Failed the fail reason +P previous state +p Package name +S Package state +s Time in this state in full seconds since epoch +t time of state change +u Builder (e.g. buildd_mipsel-rem) +v Package version +V full Package version (i.e. with +b.., = %v%{+b}B%B +X the string normally between [], e.g. optional:out-of-date:calprio{61}:days{25} + +%{Text}? print Text in case ? is not empty; ? is never printed +%{!Text}? print Text in case ? is empty; ? is never printed +Text could contain further %. To start with !, use %! + +=cut + return stringf($printfmt, ( + 'p' => make_fmt( $pkg->{'package'}, $pkg, $var), + 'a' => make_fmt( $arch, $pkg, $var), + 's' => make_fmt( sub { return floor(str2time($pkg->{'state_change'})); }, $pkg, $var), + 'v' => make_fmt( $pkg->{'version'}, $pkg, $var), + 'V' => make_fmt( sub { $pkg->{'binary_nmu_version'} ? $pkg->{'version'}."+b".$pkg->{'binary_nmu_version'} : $pkg->{'version'} }, $pkg, $var), + 'S' => make_fmt( $pkg->{'state'}, $pkg, $var), + 'u' => make_fmt( $pkg->{'builder'}, $pkg, $var), + 'X' => make_fmt( sub { + my $c = "$pkg->{'priority'}:$pkg->{'notes'}"; + $c .= ":PREV-FAILED" if $pkg->{'previous_state'} =~ /^Failed/; + $c .= ":bp{" . $pkg->{'buildpri'} . "}" if defined $pkg->{'buildpri'}; + $c .= ":binNMU{" . $pkg->{'binary_nmu_version'} . "}" if defined $pkg->{'binary_nmu_version'}; + $c .= ":calprio{". $pkg->{'calprio'}."}"; + $c .= ":days{". $pkg->{'state_days'}."}"; + return $c; + }, $pkg, $var), + 'c' => make_fmt( $pkg->{'section'}, $pkg, $var), + 'P' => make_fmt( $pkg->{'previous_state'} || "unknwon", $pkg, $var), + 'E' => make_fmt( sub { return $pkg->{'depends'} if $pkg->{'state'} eq "Dep-Wait"; + return $var->{scnt}{'Needs-Build'} + 1 if $pkg->{'state'} eq 'Needs-Build'; + return ""; }, $pkg, $var), + 'F' => make_fmt( sub { return "" unless $pkg->{'failed'}; + my $failed = $pkg->{'failed'}; + $failed =~ s/\\/\\\\/g; + return $pkg->{'package'}."#".$arch."-failure\n ". + join("\\0a",split("\n",$failed))."\\0a\n"; }, $pkg, $var), + 'D' => make_fmt( sub { return "" unless $pkg->{'bd_problem'}; + return $pkg->{'package'}."#".$arch."-bd-problem\n". + join("\\0a",split("\n",$pkg->{'bd_problem'}))."\\0a\n"; }, $pkg, $var), + 'B' => make_fmt( sub { return $pkg->{'binary_nmu_version'} if defined $pkg->{'binary_nmu_version'}; }, $pkg, $var), + 'd' => make_fmt( $distribution, $pkg, $var), + 't' => make_fmt( $pkg->{'state_change'}, $pkg, $var), + )); +} + sub list_packages { my $state = shift; my( $name, $pkg, @list ); @@ -1833,105 +1917,119 @@ sub list_packages { my %scnt; my $ctime = time; - foreach $name (keys %db) { + my $db = get_all_source_info(state => $state, user => $user, category => $category, list_min_age => $list_min_age); + foreach $name (keys %$db) { next if $name =~ /^_/; - $pkg = $db{$name}; - next if $state ne "all" && $pkg->{'State'} !~ /^\Q$state\E$/i; - next if $user && (lc($state) ne 'needs-build' and $pkg->{'Builder'} ne $user); - next if $category && $pkg->{'State'} eq "Failed" && - $pkg->{'Failed-Category'} ne $category; - next if ($list_min_age > 0 && - ($ctime-parse_date($pkg->{'State-Change'})) < $list_min_age)|| - ($list_min_age < 0 && - ($ctime-parse_date($pkg->{'State-Change'})) > -$list_min_age); - push( @list, $pkg ); + push @list, calculate_prio($db->{$name}); } + # filter components + @list = grep { my $i = $_->{'component'}; grep { $i eq $_ } split /[, ]+/, $yamlmap->{"restrict"}{'component'} } @list; + + # first adjust ownprintformat, then set printformat accordingly + $printformat ||= $yamlmap->{"format"}{$ownprintformat}; + $printformat ||= $yamlmap->{"format"}{"default"}{$state}; + $printformat ||= $yamlmap->{"format"}{"default"}{"default"}; + undef $printformat if ($ownprintformat eq 'none'); + foreach $pkg (sort sort_list_func @list) { - print "$pkg->{'Section'}/$pkg->{'Package'}_$pkg->{'Version'}"; - print ": $pkg->{'State'}" + if ($printformat) { + print print_format($printformat, $pkg, {'cnt' => $cnt, 'scnt' => \%scnt})."\n"; + ++$cnt; + $scnt{$pkg->{'state'}}++; + next; + } + print print_format("%c/%p_%v", $pkg, {}); + print print_format(": %S", $pkg, {}) if $state eq "all"; - print " by $pkg->{'Builder'}" - if $pkg->{'State'} ne "Needs-Build" && $pkg->{'Builder'}; - print " [$pkg->{'Priority'}:$pkg->{'Notes'}"; - print ":PREV-FAILED" - if $pkg->{'Previous-State'} =~ /^Failed/; - print ":bp{" . $pkg->{'BuildPri'} . "}" - if exists $pkg->{'BuildPri'}; - print ":binNMU{" . $pkg->{'Binary-NMU-Version'} . "}" - if exists $pkg->{'Binary-NMU-Version'}; - print "]\n"; + print print_format("%{ by }u%u", $pkg, {}) + if $pkg->{'state'} ne "Needs-Build"; + print print_format(" [%X]\n", $pkg, {}); print " Reasons for failing:\n", " [Category: ", - exists $pkg->{'Failed-Category'} ? $pkg->{'Failed-Category'} : "none", + defined $pkg->{'failed_category'} ? $pkg->{'failed_category'} : "none", "]\n ", - join("\n ",split("\n",$pkg->{'Failed'})), "\n" - if $pkg->{'State'} =~ /^Failed/; - print " Dependencies: $pkg->{'Depends'}\n" - if $pkg->{'State'} eq "Dep-Wait"; - print " Reason: $pkg->{'Reason'}\n" - if $pkg->{'State'} eq "BD-Uninstallable"; - print " Previous state was $pkg->{'Previous-State'} until ", - "$pkg->{'State-Change'}\n" - if $verbose && $pkg->{'Previous-State'}; + join("\n ",split("\n",$pkg->{'failed'})), "\n" + if $pkg->{'state'} =~ /^Failed/; + print " Dependencies: $pkg->{'depends'}\n" + if $pkg->{'state'} eq "Dep-Wait"; + print " Reasons for BD-Uninstallable:\n ", + join("\n ",split("\n",$pkg->{'bd_problem'})), "\n" + if $pkg->{'state'} eq "BD-Uninstallable"; + print " Previous state was $pkg->{'previous_state'}\n" + if $verbose && $pkg->{'previous_state'}; + print " No previous state recorded\n" + if $verbose && !$pkg->{'previous_state'}; + print " State changed at $pkg->{'state_change'}\n" + if $verbose && $pkg->{'state_change'}; + print " Previous state $pkg->{'previous_state'} left $pkg->{'state_time'} ago\n" + if $verbose && $pkg->{'previous_state'}; print " Previous failing reasons:\n ", - join("\n ",split("\n",$pkg->{'Old-Failed'})), "\n" - if $verbose && $pkg->{'Old-Failed'}; + join("\n ",split("\n",$pkg->{'old_failed'})), "\n" + if $verbose && $pkg->{'old_failed'}; ++$cnt; - $scnt{$pkg->{'State'}}++ if $state eq "all"; + $scnt{$pkg->{'state'}}++ if $state eq "all"; } - if ($state eq "all") { + if ($state eq "all" && !$printformat) { foreach (sort keys %scnt) { print "Total $scnt{$_} package(s) in state $_.\n"; } } - print "Total $cnt package(s)\n"; + print "Total $cnt package(s)\n" unless $printformat; } sub info_packages { my( $name, $pkg, $key, $dist ); - my @firstkeys = qw(Package Version Builder State Section Priority - Installed-Version Previous-State State-Change); + my @firstkeys = qw(package version builder state section priority + installed_version previous_state state_change); my @dists = $info_all_dists ? keys %conf::distributions : ($distribution); + my %beautykeys = ( 'package' => 'Package', 'version' => 'Version', 'builder' => 'Builder', + 'state' => 'State', 'section' => 'Section', 'priority' => 'Priority', + 'installed_version' => 'Installed-Version', 'previous_state' => 'Previous-State', + 'state_change' => 'State-Change', + 'bd_problem' => 'BD-Problem', + 'binary_nmu_changelog' => 'Binary-NMU-Changelog', 'binary_nmu_version' => 'Binary-NMU-Version', + 'buildpri' => 'BuildPri', 'depends' => 'Depends', 'failed' => 'Failed', + 'failed_category' => 'Failed-Category', 'notes' => 'Notes', + 'distribution' => 'Distribution', 'old_failed' => 'Old-Failed', + 'permbuildpri' => 'PermBuildPri', 'rel' => 'Rel', + 'calprio' => 'CalculatedPri', 'state_days' => 'State-Days' + ); - foreach $dist (@dists) { - if ($dist ne $distribution) { - if (!-f db_filename( $dist ) || !open_other_db( $dist )) { - warn "Cannot open database for $dist!\n"; - @dists = grep { $_ ne $dist } @dists; - } - } - } - foreach $name (@_) { $name =~ s/_.*$//; # strip version foreach $dist (@dists) { - my $db = $dist ne $distribution ? $otherdb{$dist} : \%db; my $pname = "$name" . ($info_all_dists ? "($dist)" : ""); - $pkg = $db->{$name}; + $pkg = get_readonly_source_info($name); if (!defined( $pkg )) { print "$pname: not registered\n"; next; } + $pkg = calculate_prio($pkg); print "$pname:\n"; foreach $key (@firstkeys) { - next if !exists $pkg->{$key}; + next if !defined $pkg->{$key}; my $val = $pkg->{$key}; chomp( $val ); $val = "\n$val" if isin( $key, qw(Failed Old-Failed)); $val =~ s/\n/\n /g; - printf " %-20s: %s\n", $key, $val; + my $print_key = $key; + $print_key = $beautykeys{$print_key} if $beautykeys{$print_key}; + printf " %-20s: %s\n", $print_key, $val; } foreach $key (sort keys %$pkg) { next if isin( $key, @firstkeys ); my $val = $pkg->{$key}; + next if !defined($val); chomp( $val ); $val = "\n$val" if isin( $key, qw(Failed Old-Failed)); $val =~ s/\n/\n /g; - printf " %-20s: %s\n", $key, $val; + my $print_key = $key; + $print_key = $beautykeys{$print_key} if $beautykeys{$print_key}; + printf " %-20s: %s\n", $print_key, $val; } } } @@ -1942,7 +2040,7 @@ sub forget_packages { foreach $name (@_) { $name =~ s/_.*$//; # strip version - $pkg = $db{$name}; + $pkg = get_source_info($name); if (!defined( $pkg )) { print "$name: not registered\n"; next; @@ -1956,171 +2054,23 @@ sub forget_packages { $data .= sprintf " %-20s: %s\n", $key, $val; } send_mail( $conf::db_maint, - "$name deleted from DB $conf::dbbase", + "$name deleted from DB " . table_name() . " " . $distribution, "The package '$name' has been deleted from the database ". "by $user.\n\n". "Data registered about the deleted package:\n". "$data\n" ) if $conf::db_maint; change_state( \$pkg, 'deleted' ); log_ta( $pkg, "--forget" ); - delete $db{$name}; + del_source_info($name); print "$name: deleted from database\n" if $verbose; } } sub forget_users { - my( $name, $ui ); - my $change = 0; - - $ui = $db{'_userinfo'}; - foreach $name (@_) { - if (!defined( $ui->{$name} )) { - print "$name: not registered\n"; - next; - } - - delete $ui->{$name}; - $change++; - print "$name: deleted from database\n" if $verbose; - } - $db{'_userinfo'} = $ui if $change; -} - -sub lock_db { - my $dist = shift; - my $try = 0; - my $lockfile = db_lockfilename($dist); - local( *F ); - - print "Locking $dist database\n" if $verbose >= 2; - repeat: - if (!sysopen( F, $lockfile, O_WRONLY|O_CREAT|O_TRUNC|O_EXCL, 0644 )){ - if ($! == EEXIST) { - # lock file exists, wait - goto repeat if !open( F, "<$lockfile" ); - my $line = ; - close( F ); - if ($line !~ /^(\d+)\s+([\w\d.-]+)$/) { - warn "Bad lock file contents -- still trying\n"; - } - else { - my($pid, $usr) = ($1, $2); - if (kill( 0, $pid ) == 0 && $! == ESRCH) { - # process doesn't exist anymore, remove stale lock - print "Removing stale lock file (pid $pid, user $usr)\n"; - unlink( $lockfile ); - goto repeat; - } - if ($pid == $lock_for_pid) { - # We are allowed to use this lock. - return; - } - warn "Database locked by $usr -- please wait\n" if $try == 0; - } - if (++$try > 200) { - # avoid the END routine removes the lock - $main::keep_lock{$dist} = 1; - die "Lock still present after 200 * 5 seconds.\n"; - } - sleep 5; - goto repeat; - } - die "Can't create lock file $lockfile: $!\n"; - } - my $pid = $lock_for_pid == -1 ? $$ : $lock_for_pid; - F->print("$pid $real_user\n"); - F->close(); -} - -sub unlock_db { - my $dist = shift; - my $lockfile = db_lockfilename($dist); - - if (!$main::keep_lock{$dist}) { - print "Unlocking $dist database\n" if $verbose >= 2; - unlink $lockfile; - } -} - -sub create_maintlock { - my $lockfile = db_lockfilename("maintenance"); - my $try = 0; - local( *F ); - - print "Creating maintenance lock\n" if $verbose >= 2; - repeat: - if (!sysopen( F, $lockfile, O_WRONLY|O_CREAT|O_TRUNC|O_EXCL, 0644 )){ - if ($! == EEXIST) { - # lock file exists, wait - goto repeat if !open( F, "<$lockfile" ); - my $line = ; - close( F ); - if ($line !~ /^(\d+)\s+([\w\d.-]+)$/) { - warn "Bad maintenance lock file contents -- still trying\n"; - } - else { - my($pid, $usr) = ($1, $2); - if (kill( 0, $pid ) == 0 && $! == ESRCH) { - # process doesn't exist anymore, remove stale lock - print "Removing stale lock file (pid $pid, user $usr)\n"; - unlink( $lockfile ); - goto repeat; - } - warn "Maintenance lock already exists by $usr -- ". - "please wait\n" if $try == 0; - } - if (++$try > 120) { - die "Lock still present after 120 * 60 seconds.\n"; - } - sleep 60; - goto repeat; - } - die "Can't create maintenance lock $lockfile: $!\n"; - } - F->print(getppid(), " $real_user\n"); - F->close(); -} - -sub remove_maintlock { - my $lockfile = db_lockfilename("maintenance"); - - print "Removing maintenance lock\n" if $verbose >= 2; - unlink $lockfile; + $dbh->do("DELETE from " . user_table_name() . + " WHERE distribution = ?", undef, $distribution) or die $dbh->errstr; } -sub waitfor_maintlock { - my $lockfile = db_lockfilename("maintenance"); - my $try = 0; - local( *F ); - - print "Checking for maintenance lock\n" if $verbose >= 2; - repeat: - if (open( F, "<$lockfile" )) { - my $line = ; - close( F ); - if ($line !~ /^(\d+)\s+([\w\d.-]+)$/) { - warn "Bad maintenance lock file contents -- still trying\n"; - } - else { - my($pid, $usr) = ($1, $2); - if (kill( 0, $pid ) == 0 && $! == ESRCH) { - # process doesn't exist anymore, remove stale lock - print "Removing stale maintenance lock (pid $pid, user $usr)\n"; - unlink( $lockfile ); - return; - } - warn "Databases locked for general maintenance by $usr -- ". - "please wait\n" if $try == 0; - } - if (++$try > 120) { - die "Lock still present after 120 * 60 seconds.\n"; - } - sleep 60; - goto repeat; - } -} - - sub read_db { my $file = shift; @@ -2137,23 +2087,24 @@ sub read_db { while( /^(\S+):[ \t]*(.*)[ \t]*$/mg ) { my ($key, $val) = ($1, $2); + $key =~ s/-/_/g; + $key =~ tr/A-Z/a-z/; $val =~ s/\376\377/\n/g; $thispkg{$key} = $val; } check_entry( \%thispkg ); # add to db - if (exists($thispkg{'Package'})) { - $name = $thispkg{'Package'}; - $db{$name} = \%thispkg; - } - elsif(exists($thispkg{'User'})) { - my $userinfo = $db{'_userinfo'}; - $userinfo = {} if (!defined($userinfo)); - - $name = $thispkg{'User'}; - $userinfo->{$name} = \%thispkg; - - $db{'_userinfo'} = $userinfo; + if (exists($thispkg{'package'})) { + update_source_info(\%thispkg); + } + elsif(exists($thispkg{'user'})) { + # user in import, username in database. + $dbh->do('INSERT INTO ' . user_table_name() . + ' (username, distribution, last_seen)' . + ' values (?, ?, ?)', + undef, $thispkg{'user'}, $distribution, + $thispkg{'last_seen'}) + or die $dbh->errstr; } } close( F ); @@ -2167,142 +2118,102 @@ sub check_entry { return if $op_mode eq "manual-edit"; # no checks then # check for required fields - if (exists $pkg->{'User'}) { + if (exists $pkg->{'user'}) { return; } - if (!exists $pkg->{'Package'}) { + if (!exists $pkg->{'package'}) { print STDERR "Bad entry: ", join( "\n", map { "$_: $pkg->{$_}" } keys %$pkg ), "\n"; - die "Database entry lacks Package or User: field\n"; - } - if (!exists $pkg->{'Version'}) { - die "Database entry for $pkg->{'Package'} lacks Version: field\n"; + die "Database entry lacks package or username field\n"; } # if no State: field, generate one (for old db compat) - if (!exists($pkg->{'State'})) { - $pkg->{'State'} = - exists $pkg->{'Failed'} ? 'Failed' : 'Building'; + if (!exists($pkg->{'state'})) { + $pkg->{'state'} = + exists $pkg->{'failed'} ? 'Failed' : 'Building'; + } + if (!exists $pkg->{'version'} and $pkg->{'state'} ne 'Not-For-Us') { + die "Database entry for $pkg->{'package'} lacks Version: field\n"; } # check state field - die "Bad state $pkg->{'State'} of package $pkg->{Package}\n" - if !isin( $pkg->{'State'}, - qw(Needs-Build Building Built Build-Attempted Uploaded Installed Dep-Wait + die "Bad state $pkg->{'state'} of package $pkg->{Package}\n" + if !isin( $pkg->{'state'}, + qw(Needs-Build Building Built Build-Attempted Uploaded Installed Dep-Wait Dep-Wait-Removed Failed Failed-Removed Not-For-Us BD-Uninstallable ) ); } -sub write_db { +sub export_db { my $file = shift; my($name,$pkg,$key); - + print "Writing ASCII database to $file..." if $verbose >= 1; open( F, ">$file" ) or - die "Can't open database $file: $!\n"; + die "Can't open export $file: $!\n"; - foreach $name (sort keys %db) { - my $pkg = $db{$name}; - if ($name eq '_userinfo') { - foreach $user (sort keys %{$pkg}) { - my $ui = $pkg->{$user}; - print F "User: $user\n" - if (!defined($ui->{'User'})); - foreach $key (keys %{$ui}) { - my $val = $ui->{$key}; - $val =~ s/\n*$//; - $val =~ s/^/ /mg; - $val =~ s/^ $/ ./mg; - print F "$key: $val\n"; - } - print F "\n"; - } - } - else { - foreach $key (keys %{$pkg}) { - my $val = $pkg->{$key}; - $val =~ s/\n*$//; - $val =~ s/^/ /mg; - $val =~ s/^ $/ ./mg; - print F "$key: $val\n"; - } - print F "\n"; + my $db = get_all_source_info(); + foreach $name (keys %$db) { + next if $name =~ /^_/; + my $pkg = $db->{$name}; + foreach $key (keys %{$pkg}) { + my $val = $pkg->{$key}; + next if !defined($val); + $val =~ s/\n*$//; + $val =~ s/^/ /mg; + $val =~ s/^ +$/ ./mg; + print F "$key: $val\n"; } - } - close( F ); - print "done\n" if $verbose >= 1; + print F "\n"; + } + close( F ); + print "done\n" if $verbose >= 1; } sub change_state { my $pkgr = shift; my $pkg = $$pkgr; my $newstate = shift; - my $for_dist = shift; - my $db; - if ($for_dist) { - return if !open_other_db( $for_dist ); - $db = $otherdb{$for_dist}; - $pkg = \$db{$pkg->{'Package'}}; - } - else { - $db = \%db; - } - my $state = \$pkg->{'State'}; + my $state = \$pkg->{'state'}; return if defined($$state) and $$state eq $newstate; - $pkg->{'Previous-State'} = $$state if defined($$state); - $pkg->{'State-Change'} = $curr_date; + $pkg->{'previous_state'} = $$state if defined($$state); + $pkg->{'state_change'} = $curr_date; + $pkg->{'do_state_change'} = 1; if (defined($$state) and $$state eq 'Failed') { - $pkg->{'Old-Failed'} = - "-"x20 . " $pkg->{'Version'} " . "-"x20 . "\n" . - $pkg->{'Failed'} . "\n" . - $pkg->{'Old-Failed'}; - delete $pkg->{'Failed'}; - delete $pkg->{'Failed-Category'}; + $pkg->{'old_failed'} = + "-"x20 . " $pkg->{'version'} " . "-"x20 . "\n" . + $pkg->{'failed'} . "\n" . + $pkg->{'old_failed'}; + delete $pkg->{'failed'}; + delete $pkg->{'failed_category'}; } if (defined($$state) and $$state eq 'BD-Uninstallable') { - delete $pkg->{'Reason'}; + delete $pkg->{'bd_problem'}; } $$state = $newstate; } -sub open_other_db { - my $dist = shift; - - if (!tied(%{$otherdb{$dist}})) { - lock_db( $dist ); - $otherdb_lock{$dist} = 1; - if (!(tie %{$otherdb{$dist}}, 'MLDBM', db_filename($dist), 0, 0664)){ - warn "Serious warning: Cannot open database for $dist\n"; - unlock_db( $dist ); - $otherdb_lock{$dist} = 0; - return 0; - } - } - return 1; -} - sub log_ta { my $pkg = shift; my $action = shift; - my $db = \%db; my $dist = $distribution; my $str; my $prevstate; - $prevstate = $pkg->{'Previous-State'}; - $str = "$action($dist): $pkg->{'Package'}_$pkg->{'Version'} ". - "changed from $prevstate to $pkg->{'State'} ". + $prevstate = $pkg->{'previous_state'}; + $str = "$action($dist): $pkg->{'package'}_$pkg->{'version'} ". + "changed from $prevstate to $pkg->{'state'} ". "by $real_user as $user"; - my $transactlog = db_transactlog( $distribution ); - if (!open( LOG, ">>$transactlog" )) { - warn "Can't open log file $transactlog: $!\n"; - return; - } - print LOG "$curr_date: $str\n"; - close( LOG ); - - if (!($prevstate eq 'Failed' && $pkg->{'State'} eq 'Failed')) { + $dbh->do('INSERT INTO ' . transactions_table_name() . + ' (package, distribution, version, action, ' . + ' prevstate, state, real_user, set_user, time) ' . + ' values (?, ?, ?, ?, ?, ?, ?, ?, ?)', + undef, $pkg->{'package'}, $distribution, + $pkg->{'version'}, $action, $prevstate, $pkg->{'state'}, + $real_user, $user, 'now()') or die $dbh->errstr; + + if (!($prevstate eq 'Failed' && $pkg->{'state'} eq 'Failed')) { $str .= " (with --override)" if $opt_override; $mail_logs .= "$str\n"; @@ -2332,40 +2243,6 @@ sub send_mail { close( PIPE ); } -sub db_filename { - my $dist = shift; - return $transactional ? db_filename_transaction($dist) : db_filename_master($dist); -} -sub db_filename_master { - my $dist = shift; - return "$conf::basedir/$conf::dbbase-$dist"; -} -sub db_filename_transaction { - my $dist = shift; - return "$conf::basedir/$conf::dbbase-$dist-transaction"; -} - -sub db_lockfilename { - my $dist = shift; - return db_filename_master($dist) . ".lock"; -} - - -sub db_transactlog { - my $dist = shift; - return $transactional ? db_transactlog_transaction($dist) : db_transactlog_master($dist); -} -sub db_transactlog_master { - my $dist = shift; - $conf::dbbase =~ m#^([^/]+/)#; - return "$conf::basedir/$1$conf::transactlog"; -} -sub db_transactlog_transaction { - my $dist = shift; - $conf::dbbase =~ m#^([^/]+/)#; - return "$conf::basedir/$1$conf::transactlog-$dist-transaction"; -} - # for parsing input to dep-wait sub parse_deplist { my $deps = shift; @@ -2392,69 +2269,16 @@ sub parse_deplist { } my($dep, $rel, $relv) = ($1, $3, $4); $rel = ">>" if defined($rel) and $rel eq ">"; - $result{$dep}->{'Package'} = $dep; + $result{$dep}->{'package'} = $dep; if ($rel && $relv) { - $result{$dep}->{'Rel'} = $rel; - $result{$dep}->{'Version'} = $relv; + $result{$dep}->{'rel'} = $rel; + $result{$dep}->{'version'} = $relv; } } return 1 if $verify; return \%result; } -# for parsing Build-Depends from Sources -sub parse_srcdeplist { - my $pkg = shift; - my $deps = shift; - my $arch = shift; - my $dep; - my @results; - - foreach $dep (split( /\s*,\s*/, $deps )) { - my @alts = split( /\s*\|\s*/, $dep ); - # Anything with an | is ignored, as it can be configured on a - # per-buildd basis what will be installed - next if $#alts != 0; - $_ = shift @alts; - if (!/^([^\s([]+)\s*(\(\s*([<=>]+)\s*(\S+)\s*\))?(\s*\[([^]]+)\])?/) { - warn( "parse_srcdeplist: bad dependency $_\n" ); - next; - } - my($dep, $rel, $relv, $archlist) = ($1, $3, $4, $6); - if ($archlist) { - $archlist =~ s/^\s*(.*)\s*$/$1/; - my @archs = split( /\s+/, $archlist ); - my ($use_it, $ignore_it, $include) = (0, 0, 0); - foreach (@archs) { - if (/^!/) { - $ignore_it = 1 if substr($_, 1) eq $arch; - } else { - $use_it = 1 if $_ eq $arch; - $include = 1; - } - } - warn "Warning: inconsistent arch restriction on ", - "$pkg: $dep depedency\n" - if $ignore_it && $use_it; - next if $ignore_it || ($include && !$use_it); - } - my $neg = 0; - if ($dep =~ /^!/) { - $dep =~ s/^!\s*//; - $neg = 1; - } - my $result; - $result->{'Package'} = $dep; - $result->{'Neg'} = $neg; - if ($rel && $relv) { - $result->{'Rel'} = $rel; - $result->{'Version'} = $relv; - } - push @results, $result; - } - return \@results; -} - sub build_deplist { my $list = shift; my($key, $result); @@ -2462,106 +2286,15 @@ sub build_deplist { foreach $key (keys %$list) { $result .= ", " if $result; $result .= $key; - $result .= " ($list->{$key}->{'Rel'} $list->{$key}->{'Version'})" - if $list->{$key}->{'Rel'} && $list->{$key}->{'Version'}; + $result .= " ($list->{$key}->{'rel'} $list->{$key}->{'version'})" + if $list->{$key}->{'rel'} && $list->{$key}->{'version'}; } return $result; } -sub clean_db { - my %new_db; - tie %new_db, 'MLDBM', db_filename( $distribution ) . ".new", GDBM_WRCREAT, 0664 - or die "FATAL: Cannot create new database\n"; - %new_db = %db; - untie %db or die "FATAL: Cannot untie old database\n"; - system ("cp " . db_filename( $distribution ) . ".new " . - db_filename( $distribution ) ) == 0 - or die "FATAL: Cannot overwrite old database"; - unlink db_filename( $distribution ) . ".new"; - %db = %new_db; -} - - -sub get_unsatisfied_dep { - my $bd = shift; - my $pkgs = shift; - my $dep = shift; - my $savedep = shift; - - my $pkgname = $dep->{'Package'}; - - if (defined $pkgs->{$pkgname}{'Provider'}) { - # provides. leave them for buildd/sbuild. - return ""; - } - - # check cache - return $pkgs->{$pkgname}{'Unsatisfied'} if $savedep and defined($pkgs->{$pkgname}{'Unsatisfied'}); - - # Return unsatisfied deps to a higher caller to process - if ((!defined($pkgs->{$pkgname})) or - (defined($dep->{'Rel'}) and !version_compare( $pkgs->{$pkgname}{'Version'}, $dep->{'Rel'}, $dep->{'Version'} ) ) ) { - my %deplist; - $deplist{$pkgname} = $dep; - my $deps = build_deplist(\%deplist); - $pkgs->{$pkgname}{'Unsatisfied'} = $deps if $savedep; - return $deps; - } - - # set cache to "" to avoid infinite recursion - $pkgs->{$pkgname}{'Unsatisfied'} = "" if $savedep; - - if (defined $pkgs->{$dep->{'Package'}}{'Depends'}) { - my $deps = parse_deplist( $pkgs->{$dep->{'Package'}}{'Depends'} ); - foreach (keys %$deps) { - $dep = $$deps{$_}; - # recur on dep. - my $ret = get_unsatisfied_dep($bd,$pkgs,$dep,1); - if ($ret ne "") { - my $retdep = parse_deplist( $ret ); - foreach (keys %$retdep) { - $dep = $$retdep{$_}; - - $dep->{'Rel'} = '>=' if defined($dep->{'Rel'}) and $dep->{'Rel'} =~ '^='; - - if (defined($dep->{'Rel'}) and $dep->{'Rel'} =~ '^>' and defined ($pkgs->{$dep->{'Package'}}) and - version_compare($bd->{$pkgs->{$dep->{'Package'}}{'Source'}}{'ver'},'>>',$pkgs->{$dep->{'Package'}}{'Sourcev'})) { - if (not defined($merge_binsrc{$dep->{'Package'}})) { - # the uninstallable package doesn't exist in the new source; look for something else that does. - delete $$retdep{$dep->{'Package'}}; - foreach (sort (split( /\s*,\s*/, $bd->{$pkgs->{$dep->{'Package'}}{'Source'}}{'bin'}))) { - next if ($pkgs->{$_}{'all'} or not defined $pkgs->{$_}{'Version'}); - $dep->{'Package'} = $_; - $dep->{'Rel'} = '>>'; - $dep->{'Version'} = $pkgs->{$_}{'Version'}; - $$retdep{$_} = $dep; - last; - } - } - } else { - # sanity check to make sure the depending binary still exists, and the depended binary exists and dep-wait on a new version of it - if ( defined($merge_binsrc{$pkgname}) and defined($pkgs->{$dep->{'Package'}}{'Version'}) ) { - delete $$retdep{$dep->{'Package'}}; - $dep->{'Package'} = $pkgname; - $dep->{'Rel'} = '>>'; - $dep->{'Version'} = $pkgs->{$pkgname}{'Version'}; - $$retdep{$pkgname} = $dep; - } - delete $$retdep{$dep->{'Package'}} if (defined ($dep->{'Rel'}) and $dep->{'Rel'} =~ '^>'); - } - } - $ret = build_deplist($retdep); - $pkgs->{$pkgname}{'Unsatisfied'} = $ret if $savedep; - return $ret; - } - } - } - return ""; -} - sub call_edos_depcheck { my $packagesfile = shift; - my $sourcesfile = shift; + my $srcs = shift; my $key; return if defined ($conf::distributions{$distribution}{noadw}); @@ -2571,9 +2304,10 @@ sub call_edos_depcheck { # We also check everything in bd-uninstallable, as any new upload could # make that work again my %interesting_packages; - foreach $key (keys %db) { - my $pkg = $db{$key}; - if (defined $pkg and isin($pkg->{'State'}, qw/Needs-Build BD-Uninstallable/)) { + my $db = get_all_source_info(); + foreach $key (keys %$db) { + my $pkg = $db->{$key}; + if (defined $pkg and isin($pkg->{'state'}, qw/Needs-Build BD-Uninstallable/)) { $interesting_packages{$key} = undef; } } @@ -2590,11 +2324,11 @@ sub call_edos_depcheck { open SOURCES, '>', $tmpfile or die "Could not open temporary file $tmpfile\n"; for my $key (keys %interesting_packages) { - my $pkg = $db{$key}; + my $pkg = $db->{$key}; print SOURCES "Package: $key\n"; - print SOURCES "Version: $pkg->{'Version'}\n"; - print SOURCES "Build-Depends: $pkg->{'Build-Depends'}\n" if $pkg->{'Build-Depends'}; - print SOURCES "Build-Conflicts: $pkg->{'Build-Conflicts'}\n" if $pkg->{'Build-Conflicts'}; + print SOURCES "Version: $pkg->{'version'}\n"; + print SOURCES "Build-Depends: $srcs->{$key}{'dep'}\n" if $srcs->{$key}{'dep'}; + print SOURCES "Build-Conflicts: $srcs->{$key}{'conf'}\n" if $srcs->{$key}{'conf'}; print SOURCES "Architecture: all\n"; print SOURCES "\n"; } @@ -2626,20 +2360,30 @@ sub call_edos_depcheck { unlink( $tmpfile ); for my $key (keys %interesting_packages) { - my $pkg = $db{$key}; + my $pkg = $db->{$key}; my $change = - (defined $interesting_packages{$key} and $pkg->{'State'} eq 'Needs-Build') || - (not defined $interesting_packages{$key} and $pkg->{'State'} eq 'BD-Uninstallable'); + (defined $interesting_packages{$key} and $pkg->{'state'} eq 'Needs-Build') || + (not defined $interesting_packages{$key} and $pkg->{'state'} eq 'BD-Uninstallable'); + my $problemchange = $interesting_packages{$key} ne $pkg->{'bd_problem'}; if ($change) { if (defined $interesting_packages{$key}) { change_state( \$pkg, 'BD-Uninstallable' ); - $pkg->{'Reason'} = $interesting_packages{$key}; + $pkg->{'bd_problem'} = $interesting_packages{$key}; } else { change_state( \$pkg, 'Needs-Build' ); } + } + if ($problemchange) { + if (defined $interesting_packages{$key}) { + $pkg->{'bd_problem'} = $interesting_packages{$key}; + } + } + if ($change) { log_ta( $pkg, "--merge-all" ); - $db{$key} = $pkg; - print "edos-builddebchange changed state of ${key}_$pkg->{'Version'} to $pkg->{'State'}\n" if $verbose; + print "edos-builddebchange changed state of ${key}_$pkg->{'version'} to $pkg->{'state'}\n" if $verbose; + } + if ($change || $problemchange) { + update_source_info($pkg); } } } @@ -2651,6 +2395,7 @@ sub usage { Usage: $prgname Options: -v, --verbose: Verbose execution. + -A arch: Architecture this operation is for. --take: Take package for building [default operation] -f, --failed: Record in database that a build failed due to deficiencies in the package (that aren't fixable without a new @@ -2666,7 +2411,10 @@ Options: a new version number (source-version + "+b") --give-back: Mark a package as ready to build that is in state Building, Built or Build-Attempted. To give back a package in state Failed, use - --override + --override. This command will actually put the package in state + BD-Uninstallable, until the installability of its Build-Dependencies + were verified. This happens at each call of --merge-all, usually + every 15 minutes. --merge-quinn: Merge quinn-diff output into database. --merge-packages: Merge Packages files into database. --pretend-avail: Pretend that given packages are available now and give @@ -2686,15 +2434,6 @@ Options: automatically choosen --import FILE: Import database from a ASCII file FILE --export FILE: Export database to a ASCII file FILE - --lock-for PID: Locks the database for the process with this pid - --unlock-for PID: Unlocks the database for the process with this pid - --act-on-behalf-of PID: Ignores the log (if it is held by this pid) - --start-transaction: Creates a copy of the state of the database, for - use with --transactional. This overrides any previous uncommited - transaction. Should only be used after --lock-for - --commit-transaction: Atomically moves the copy back to the main, thus - commiting the changes - --transactional: Flag to indicate that we want to work on the copy The remaining arguments (depending on operation) usually start with "name_version", the trailer is ignored. This allows to pass the names @@ -2711,8 +2450,180 @@ sub pkg_version_eq { my $version = shift; return 1 - if (defined $pkg->{'Binary-NMU-Version'}) and - version_compare(binNMU_version($pkg->{'Version'}, - $pkg->{'Binary-NMU-Version'}),'=', $version); - return version_compare( $pkg->{'Version'}, "=", $version ); + if (defined $pkg->{'binary_nmu_version'}) and + version_compare(binNMU_version($pkg->{'version'}, + $pkg->{'binary_nmu_version'}),'=', $version); + return version_compare( $pkg->{'version'}, "=", $version ); +} + +sub table_name { + return '"' . $arch . $schema_suffix . '".packages'; +} + +sub user_table_name { + return '"' . $arch . $schema_suffix . '".users'; +} + +sub transactions_table_name { + return '"' . $arch . $schema_suffix . '".transactions'; +} + +sub get_readonly_source_info { + my $name = shift; + # SELECT FLOOR(EXTRACT('epoch' FROM age(localtimestamp, '2010-01-22 23:45')) / 86400) -- change to that? + my $pkg = $dbh->selectrow_hashref('SELECT *, extract(days from date_trunc(\'days\', now() - state_change)) as state_days FROM ' . + table_name() . ' WHERE package = ? AND distribution = ?', + undef, $name, $distribution); + return $pkg; +} + +sub get_source_info { + my $name = shift; + my $pkg = $dbh->selectrow_hashref('SELECT *, extract(days from date_trunc(\'days\', now() - state_change)) as state_days FROM ' . + table_name() . ' WHERE package = ? AND distribution = ?' . + ' FOR UPDATE', + undef, $name, $distribution); + return $pkg; } + +sub get_all_source_info { + my %options = @_; + + my $q = 'SELECT *, '. + 'extract(days from date_trunc(\'days\', now() - state_change)) as state_days, '. + 'date_trunc(\'seconds\', now() - state_change) as state_time'. + ' FROM ' . table_name() + . ' WHERE distribution = ? '; + my @args = ($distribution); + if ($options{state} && uc($options{state}) ne "ALL") { + $q .= ' AND upper(state) = ? '; + push @args, uc($options{state}); + } + + if ($options{user}) { + #this basically means "this user, or no user at all": + $q .= ' AND (builder = ? OR upper(state) = ?)'; + push @args, $options{user}; + push @args, "NEEDS-BUILD"; + } + + if ($options{category}) { + $q .= ' AND failed_category <> ? AND upper(state) = ? '; + push @args, $options{category}; + push @args, "FAILED"; + } + + if ($options{list_min_age} > 0) { + $q .= ' AND age(state_change) > ? '; + push @args, $options{list_min_age} . " days"; + } + + if ($options{list_min_age} < 0) { + $q .= ' AND age(state_change) < ? '; + push @args, -$options{list_min_age} . " days"; + } + + my $db = $dbh->selectall_hashref($q, 'package', undef, @args); + return $db; +} + +sub update_source_info { + my $pkg = shift; + + my $pkg2 = get_source_info($pkg->{'package'}); + if (! defined $pkg2) + { + add_source_info($pkg); + } + + $dbh->do('UPDATE ' . table_name() . ' SET ' . + 'version = ?, ' . + 'state = ?, ' . + 'section = ?, ' . + 'priority = ?, ' . + 'installed_version = ?, ' . + 'previous_state = ?, ' . + (($pkg->{'do_state_change'}) ? "state_change = now()," : ""). + 'notes = ?, ' . + 'builder = ?, ' . + 'failed = ?, ' . + 'old_failed = ?, ' . + 'binary_nmu_version = ?, ' . + 'binary_nmu_changelog = ?, ' . + 'failed_category = ?, ' . + 'permbuildpri = ?, ' . + 'buildpri = ?, ' . + 'depends = ?, ' . + 'rel = ?, ' . + 'bd_problem = ? ' . + 'WHERE package = ? AND distribution = ?', + undef, + $pkg->{'version'}, + $pkg->{'state'}, + $pkg->{'section'}, + $pkg->{'priority'}, + $pkg->{'installed_version'}, + $pkg->{'previous_state'}, + $pkg->{'notes'}, + $pkg->{'builder'}, + $pkg->{'failed'}, + $pkg->{'old_failed'}, + $pkg->{'binary_nmu_version'}, + $pkg->{'binary_nmu_changelog'}, + $pkg->{'failed_category'}, + $pkg->{'permbuildpri'}, + $pkg->{'buildpri'}, + $pkg->{'depends'}, + $pkg->{'rel'}, + $pkg->{'bd_problem'}, + $pkg->{'package'}, + $distribution) or die $dbh->errstr; +} + +sub add_source_info { + my $pkg = shift; + $dbh->do('INSERT INTO ' . table_name() . + ' (package, distribution) values (?, ?)', + undef, $pkg->{'package'}, $distribution) or die $dbh->errstr; +} + +sub del_source_info { + my $name = shift; + $dbh->do('DELETE FROM ' . table_name() . + ' WHERE package = ? AND distribution = ?', + undef, $name, $distribution) or die $dbh->errstr; +} + +sub get_user_info { + my $name = shift; + my $user = $dbh->selectrow_hashref('SELECT * FROM ' . + user_table_name() . ' WHERE username = ? AND distribution = ?', + undef, $name, $distribution); + return $user; +} + +sub update_user_info { + my $user = shift; + $dbh->do('UPDATE ' . user_table_name() . + ' SET last_seen = now() WHERE username = ?' . + ' AND distribution = ?', + undef, $user, $distribution) + or die $dbh->errstr; +} + + +sub add_user_info { + my $user = shift; + $dbh->do('INSERT INTO ' . user_table_name() . + ' (username, distribution, last_seen)' . + ' values (?, ?, now())', + undef, $user, $distribution) + or die $dbh->errstr; +} + +sub lock_table() +{ + $dbh->do('LOCK TABLE ' . table_name() . + ' IN EXCLUSIVE MODE', undef) or die $dbh->errstr; +} +