diff options
author | Thierry Vignaud <thierry.vignaud@gmail.com> | 2018-11-21 03:53:02 +0100 |
---|---|---|
committer | Thierry Vignaud <thierry.vignaud@gmail.com> | 2018-11-21 03:55:32 +0100 |
commit | 73aa98991a97501bc1e251689865d71ec20298a9 (patch) | |
tree | ecb5d860f8b671ea5b17b2f2b39a9f2482d60030 | |
parent | f6fb10dd51fff62d4ff274829f1b33a4cca68f0b (diff) | |
download | drakx-73aa98991a97501bc1e251689865d71ec20298a9.tar drakx-73aa98991a97501bc1e251689865d71ec20298a9.tar.gz drakx-73aa98991a97501bc1e251689865d71ec20298a9.tar.bz2 drakx-73aa98991a97501bc1e251689865d71ec20298a9.tar.xz drakx-73aa98991a97501bc1e251689865d71ec20298a9.zip |
remove debug statement
wrongly added with commit 6375ef3a8cd8b1b1e057334ab680c5c701123395
(spot by Martin Whitaker)
-rw-r--r-- | perl-install/install/pkgs.pm | 45 |
1 files changed, 16 insertions, 29 deletions
diff --git a/perl-install/install/pkgs.pm b/perl-install/install/pkgs.pm index 87ff120bb..c859fb880 100644 --- a/perl-install/install/pkgs.pm +++ b/perl-install/install/pkgs.pm @@ -842,18 +842,10 @@ sub _unselect_package { sub is_package_installed { my ($db, $pkg) = @_; my $check_installed; - my $done; - my $n=$pkg->name; $db->traverse_tag('name', [ $pkg->name ], sub { my ($p) = @_; - $done = 1; $check_installed ||= $pkg->compare_pkg($p) == 0; - log::l(">> ". $pkg->EVR . " vs ". $p->EVR. " --> check_installed=$check_installed\n"); - warn ">> ". $pkg->EVR . " vs ". $p->EVR. " --> check_installed=$check_installed\n"; - }); - my $real = `rpm -q $n`; - log::l("check_installed=0 for $n b/c pkg wasn't found in db VS $real") if !$done; return $check_installed; } @@ -873,21 +865,26 @@ sub _install_raw { log::l("rpm transactions start"); - my ($is_installing); + my ($is_installing, $verify_just_closed); my $exit_code = urpm::main_loop::run($packages, $packages->{state}, undef, undef, { - open_helper => sub { - my ($db, $packages, $type, $id) = @_; - log::l("OPENING (is_installing=$is_installing) ($db, $packages, $type, $id)\n"); - &$callback; - }, + open_helper => $callback, verify => sub { - my ($data, $cb_type, $pkg_id, $subtype, $amout, $total) = @_; - log::l("VERIFY($data, $cb_type, $pkg_id, $subtype, $amout, $total)\n"); + my ($_data, $_cb_type, $_pkg_id, $subtype, $_amout, $_total, $_pkg_name) = @_; + if ($subtype eq 'start') { + $is_installing = 0; + } elsif ($subtype eq 'stop') { + $is_installing = 1; + $verify_just_closed = 1; + } }, close_helper => sub { - my ($db, $packages, $type, $id) = @_; - log::l("CLOSING (is_installing=$is_installing) ($db, $packages, $type, $id)\n"); + my ($db, $packages, $_type, $id) = @_; &$callback; + # ignore first close after end of verify: + if ($is_installing && $verify_just_closed) { + undef $verify_just_closed; + return; + } return if !$is_installing; # don't check if it's installed if it's being verified my $pkg = defined $id && $packages->{depslist}[$id] or return; print $LOG $pkg->fullname . "\n"; @@ -901,17 +898,7 @@ sub _install_raw { } else { log::l($pkg->name . " not installed, " . URPM::rpmErrorString()); } - }, inst => sub { - my ($data, $cb_type, $pkg_id, $subtype, $amout, $total) = @_; - log::l("INSTALL($data, $cb_type, $pkg_id, $subtype, $amout, $total)\n"); - #eg: * VERIFY(urpm=HASH(0x35cd608), verify, 17729, progress, 1, 51, ) - if ($subtype eq 'start') { - $is_installing = 1; - } elsif ($subtype eq 'stop') { - $is_installing = 0; - } - &$callback; - }, + }, inst => $callback, trans => $callback, # FIXME: implement already_installed_or_not_installable bad_signature => sub { |