diff options
Diffstat (limited to 'usr.sbin/pkg_add')
-rw-r--r-- | usr.sbin/pkg_add/OpenBSD/Add.pm | 38 | ||||
-rw-r--r-- | usr.sbin/pkg_add/OpenBSD/CollisionReport.pm | 30 | ||||
-rw-r--r-- | usr.sbin/pkg_add/OpenBSD/Delete.pm | 51 | ||||
-rw-r--r-- | usr.sbin/pkg_add/OpenBSD/Dependencies.pm | 25 | ||||
-rw-r--r-- | usr.sbin/pkg_add/OpenBSD/Handle.pm | 19 | ||||
-rw-r--r-- | usr.sbin/pkg_add/OpenBSD/PackageInfo.pm | 7 | ||||
-rw-r--r-- | usr.sbin/pkg_add/OpenBSD/PackingElement.pm | 13 | ||||
-rw-r--r-- | usr.sbin/pkg_add/OpenBSD/ProgressMeter.pm | 21 | ||||
-rw-r--r-- | usr.sbin/pkg_add/OpenBSD/Replace.pm | 29 | ||||
-rw-r--r-- | usr.sbin/pkg_add/OpenBSD/Update.pm | 16 | ||||
-rw-r--r-- | usr.sbin/pkg_add/OpenBSD/UpdateSet.pm | 15 | ||||
-rw-r--r-- | usr.sbin/pkg_add/pkg_add | 33 | ||||
-rw-r--r-- | usr.sbin/pkg_add/pkg_delete | 11 |
13 files changed, 166 insertions, 142 deletions
diff --git a/usr.sbin/pkg_add/OpenBSD/Add.pm b/usr.sbin/pkg_add/OpenBSD/Add.pm index 76124a59aef..3f2af49af53 100644 --- a/usr.sbin/pkg_add/OpenBSD/Add.pm +++ b/usr.sbin/pkg_add/OpenBSD/Add.pm @@ -1,5 +1,5 @@ # ex:ts=8 sw=4: -# $OpenBSD: Add.pm,v 1.92 2009/11/11 11:13:16 espie Exp $ +# $OpenBSD: Add.pm,v 1.93 2009/11/11 12:04:19 espie Exp $ # # Copyright (c) 2003-2007 Marc Espie <espie@openbsd.org> # @@ -35,7 +35,7 @@ sub manpages_index while (my ($k, $v) = each %{$state->{mandirs}}) { my @l = map { $destdir.$_ } @$v; if ($state->{not}) { - print "Merging manpages in $destdir$k: ", join(@l), "\n" if $state->{verbose}; + $state->say("Merging manpages in $destdir$k: ", join(@l)) if $state->{verbose}; } else { try { OpenBSD::Makewhatis::merge($destdir.$k, \@l); @@ -82,8 +82,9 @@ sub record_partial_installation my $lastname = $last->realname($state); $last->{d} = $last->compute_digest($lastname, $old); if (!$old->equals($last->{d})) { - print "Adjusting ", $old->keyword, " for $lastname from ", - $old->stringize, " to ", $last->{d}->stringize, "\n"; + $state->say("Adjusting ", $old->keyword, + " for $lastname from ", $old->stringize, + " to ", $last->{d}->stringize); } } register_installation($n); @@ -278,7 +279,7 @@ sub install my ($self, $state) = @_; $self->SUPER::install($state); my $auth = $self->name; - print "adding ", $self->type, " $auth\n" if $state->{verbose}; + $state->say("adding ", $self->type, " ", $auth) if $state->{verbose}; return if $state->{not}; return if defined $self->{okay}; my $l=[]; @@ -334,8 +335,7 @@ sub install return; } if ($state->{not}) { - print "sysctl -w $name != ". - $self->{value}, "\n"; + $state->say("sysctl -w $name != ". $self->{value}); return; } VSystem($state->{very_verbose}, @@ -385,7 +385,7 @@ sub install if ($state->{replacing}) { if ($state->{not}) { - print "moving tempfile -> $destdir$fullname\n" if $state->{very_verbose}; + $state->say("moving tempfile -> $destdir$fullname") if $state->{very_verbose}; return; } File::Path::mkpath(dirname($destdir.$fullname)); @@ -396,13 +396,13 @@ sub install } else { rename($self->{tempname}, $destdir.$fullname) or Fatal "Can't move ", $self->{tempname}, " to $fullname: $!"; - print "moving ", $self->{tempname}, " -> $destdir$fullname\n" if $state->{very_verbose}; + $state->say("moving ", $self->{tempname}, " -> $destdir$fullname") if $state->{very_verbose}; undef $self->{tempname}; } } else { my $file = $self->prepare_to_extract($state); - print "extracting $destdir$fullname\n" if $state->{very_verbose}; + $state->say("extracting $destdir$fullname") if $state->{very_verbose}; if ($state->{not}) { $state->{archive}->skip; return; @@ -506,29 +506,29 @@ sub install my $origname = $destdir.$orig->fullname; if (-e $filename) { if ($state->{verbose}) { - print "The existing file $filename has NOT been changed\n"; + $state->say("The existing file $filename has NOT been changed"); if (defined $orig->{d}) { # XXX assume this would be the same type of file my $d = $self->compute_digest($filename, $orig->{d}); if ($d->equals($orig->{d})) { - print "(but it seems to match the sample file $origname)\n"; + $state->say("(but it seems to match the sample file $origname)"); } else { - print "It does NOT match the sample file $origname\n"; - print "You may wish to update it manually\n"; + $state->say("It does NOT match the sample file $origname"); + $state->say("You may wish to update it manually"); } } } } else { if ($state->{not}) { - print "The file $filename would be installed from $origname\n"; + $state->say("The file $filename would be installed from $origname"); } else { if (!copy($origname, $filename)) { Warn "File $filename could not be installed:\n\t$!\n"; } $self->set_modes($filename); if ($state->{verbose}) { - print "installed $filename from $origname\n"; + $state->say("installed $filename from $origname"); } } } @@ -593,8 +593,8 @@ sub install open(my $shells2, '>>', $destdir.OpenBSD::Paths->shells) or return; print $shells2 $fullname, "\n"; close $shells2; - print "Shell $fullname appended to $destdir", - OpenBSD::Paths->shells, "\n"; + $state->say("Shell $fullname appended to $destdir", + OpenBSD::Paths->shells); } package OpenBSD::PackingElement::Dir; @@ -605,7 +605,7 @@ sub install my $fullname = $self->fullname; my $destdir = $state->{destdir}; - print "new directory ", $destdir, $fullname, "\n" if $state->{very_verbose}; + $state->say("new directory ", $destdir, $fullname) if $state->{very_verbose}; return if $state->{not}; File::Path::mkpath($destdir.$fullname); $self->set_modes($destdir.$fullname); diff --git a/usr.sbin/pkg_add/OpenBSD/CollisionReport.pm b/usr.sbin/pkg_add/OpenBSD/CollisionReport.pm index 6bb9d1dd501..45d80da0086 100644 --- a/usr.sbin/pkg_add/OpenBSD/CollisionReport.pm +++ b/usr.sbin/pkg_add/OpenBSD/CollisionReport.pm @@ -1,5 +1,5 @@ # ex:ts=8 sw=4: -# $OpenBSD: CollisionReport.pm,v 1.20 2009/11/10 11:36:56 espie Exp $ +# $OpenBSD: CollisionReport.pm,v 1.21 2009/11/11 12:04:19 espie Exp $ # # Copyright (c) 2003-2006 Marc Espie <espie@openbsd.org> # @@ -41,7 +41,7 @@ sub find_collisions return $bypkg; } for my $pkg (installed_packages()) { - print "Looking for collisions in $pkg\n" if $verbose; + $state->say("Looking for collisions in $pkg") if $verbose; my $plist = OpenBSD::PackingList->from_installation($pkg, \&OpenBSD::PackingList::FilesOnly); next if !defined $plist; @@ -74,13 +74,13 @@ sub collision_report($$) my $clueless_bat2; my $found = 0; - print "Collision: the following files already exist\n"; + $state->say("Collision: the following files already exist"); if (!$state->{defines}->{dontfindcollisions}) { my $bypkg = find_collisions(\%todo, $state->{verbose}); for my $pkg (sort keys %$bypkg) { for my $item (sort @{$bypkg->{$pkg}}) { $found++; - print "\t$item ($pkg)\n"; + $state->say("\t$item ($pkg)"); } if ($pkg =~ m/^(?:partial\-|borked\.\d+$)/o) { $clueless_bat = $pkg; @@ -98,26 +98,26 @@ sub collision_report($$) my $old = $todo{$item}; my $d = $old->new($destdir.$item); if ($d->equals($old)) { - print "\t$item (same checksum)\n"; + $state->say("\t$item (same checksum)"); } else { - print "\t$item (different checksum)\n"; + $state->say("\t$item (different checksum)"); } } else { - print "\t$item\n"; + $state->say("\t$item"); } } } if (defined $clueless_bat) { - print "The package name $clueless_bat suggests that a former installation\n"; - print "of a similar package got interrupted. It is likely that\n"; - print "\tpkg_delete $clueless_bat\n"; - print "will solve the problem\n"; + $state->print("The package name $clueless_bat suggests that a former installation\n", + "of a similar package got interrupted. It is likely that\n", + "\tpkg_delete $clueless_bat\n", + "will solve the problem\n"); } if (defined $clueless_bat2) { - print "The package name $clueless_bat2 suggests remaining libraries\n"; - print "from a former package update. It is likely that\n"; - print "\tpkg_delete $clueless_bat2\n"; - print "will solve the problem\n"; + $state->print("The package name $clueless_bat2 suggests remaining libraries\n", + "from a former package update. It is likely that\n", + "\tpkg_delete $clueless_bat2\n". + "will solve the problem\n"); } my $dorepair = 0; if ($found == 0) { diff --git a/usr.sbin/pkg_add/OpenBSD/Delete.pm b/usr.sbin/pkg_add/OpenBSD/Delete.pm index 0413453e158..bd66c3b2e18 100644 --- a/usr.sbin/pkg_add/OpenBSD/Delete.pm +++ b/usr.sbin/pkg_add/OpenBSD/Delete.pm @@ -1,5 +1,5 @@ # ex:ts=8 sw=4: -# $OpenBSD: Delete.pm,v 1.84 2009/11/11 11:13:16 espie Exp $ +# $OpenBSD: Delete.pm,v 1.85 2009/11/11 12:04:19 espie Exp $ # # Copyright (c) 2003-2007 Marc Espie <espie@openbsd.org> # @@ -50,7 +50,7 @@ sub manpages_unindex while (my ($k, $v) = each %{$state->{mandirs}}) { my @l = map { $destdir.$_ } @$v; if ($state->{not}) { - print "Removing manpages in $destdir$k: ", join(@l), "\n" if $state->{verbose}; + $state->say("Removing manpages in $destdir$k: ", join(@l)) if $state->{verbose}; } else { eval { OpenBSD::Makewhatis::remove($destdir.$k, \@l); }; if ($@) { @@ -126,7 +126,7 @@ sub unregister_dependencies my $pkgname = $plist->pkgname; for my $name (OpenBSD::Requiring->new($pkgname)->list) { - print "remove dependency on $name\n" + $state->say("remove dependency on $name") if $state->{very_verbose} or $state->{not}; local $@; try { @@ -278,7 +278,7 @@ sub delete my ($self, $state) = @_; if ($state->{beverbose}) { - print "rmuser: ", $self->name, "\n"; + $state->say("rmuser: ", $self->name); } $self->record_shared($state->{recorder}, $state->{pkgname}); @@ -296,7 +296,7 @@ sub delete my ($self, $state) = @_; if ($state->{beverbose}) { - print "rmgroup: ", $self->name, "\n"; + $state->say("rmgroup: ", $self->name); } $self->record_shared($state->{recorder}, $state->{pkgname}); @@ -321,7 +321,7 @@ sub delete my ($self, $state) = @_; if ($state->{very_verbose}) { - print "rmdir: ", $self->fullname, "\n"; + $state->say("rmdir: ", $self->fullname); } $self->record_shared($state->{recorder}, $state->{pkgname}); @@ -384,38 +384,39 @@ sub delete if (-l $realname) { my $contents = readlink $realname; if ($contents ne $self->{symlink}) { - print "Symlink does not match: $realname ($contents vs. ", $self->{symlink},")\n"; + $state->say("Symlink does not match: $realname ($contents vs. ", $self->{symlink},")"); $self->do_not_delete($state); return; } } else { - print "Bogus symlink: $realname\n"; + $state->say("Bogus symlink: $realname"); $self->do_not_delete($state); return; } } else { if (-l $realname) { - print "Unexpected symlink: $realname\n"; + $state->say("Unexpected symlink: $realname"); $self->do_not_delete($state); } else { if (! -f $realname) { - print "File $realname does not exist\n"; + $state->say("File $realname does not exist"); return; } unless (defined($self->{link}) or $self->{nochecksum} or $state->{quick}) { if (!defined $self->{d}) { - print "Problem: ", $self->fullname, - " does not have a checksum\n"; - print "NOT deleting: $realname\n"; + $state->say("Problem: ", + $self->fullname, + " does not have a checksum\n", + "NOT deleting: $realname"); $state->log("Couldn't delete $realname (no checksum)\n"); return; } my $d = $self->compute_digest($realname, $self->{d}); if (!$d->equals($self->{d})) { - print "Problem: checksum doesn't match for ", - $self->fullname, "\n"; - print "NOT deleting: $realname\n"; + $state->say("Problem: checksum doesn't match for ", + $self->fullname, "\n", + "NOT deleting: $realname"); $state->log("Couldn't delete $realname (bad checksum)\n"); $self->do_not_delete($state); return; @@ -424,11 +425,11 @@ sub delete } } if ($state->{very_verbose}) { - print "deleting: $realname\n"; + $state->say("deleting: $realname"); } return if $state->{not}; if (!unlink $realname) { - print "Problem deleting $realname\n"; + $state->say("Problem deleting $realname"); $state->log("deleting $realname failed: $!\n"); } } @@ -532,7 +533,7 @@ sub delete } else { my $d = $self->compute_digest($realname, $orig->{d}); if ($d->equals($orig->{d})) { - print "File $realname identical to sample\n" if $state->{not} or $state->{verbose}; + $state->say("File $realname identical to sample") if $state->{not} or $state->{verbose}; } else { unless ($state->{extra}) { $self->mark_dir($state); @@ -542,9 +543,9 @@ sub delete } } return if $state->{not}; - print "deleting $realname\n" if $state->{verbose}; + $state->say("deleting $realname") if $state->{verbose}; if (!unlink $realname) { - print "Problem deleting $realname\n"; + $state->say("Problem deleting $realname"); $state->log("deleting $realname failed: $!\n"); } } @@ -586,8 +587,8 @@ sub delete open(my $shells2, '>', $destdir.OpenBSD::Paths->shells); print $shells2 @l; close $shells2; - print "Shell $fullname removed from $destdir", - OpenBSD::Paths->shells, "\n"; + $state->say("Shell $fullname removed from $destdir", + OpenBSD::Paths->shells); } } $self->SUPER::delete($state); @@ -601,7 +602,7 @@ sub delete my ($self, $state) = @_; my $realname = $self->realname($state); if ($state->{beverbose} && $state->{extra}) { - print "deleting extra file: $realname\n"; + $state->say("deleting extra file: $realname"); } return if $state->{not}; return unless -e $realname or -l $realname; @@ -610,7 +611,7 @@ sub delete $self->mark_dir($state); } elsif ($state->{extra}) { unlink($realname) or - print "problem deleting extra file $realname\n"; + $state->say("problem deleting extra file $realname"); } else { $state->log("You should also remove $realname\n"); $self->mark_dir($state); diff --git a/usr.sbin/pkg_add/OpenBSD/Dependencies.pm b/usr.sbin/pkg_add/OpenBSD/Dependencies.pm index 3d26a0c216a..56d52dfbb8e 100644 --- a/usr.sbin/pkg_add/OpenBSD/Dependencies.pm +++ b/usr.sbin/pkg_add/OpenBSD/Dependencies.pm @@ -1,5 +1,5 @@ # ex:ts=8 sw=4: -# $OpenBSD: Dependencies.pm,v 1.81 2009/11/10 11:36:56 espie Exp $ +# $OpenBSD: Dependencies.pm,v 1.82 2009/11/11 12:04:19 espie Exp $ # # Copyright (c) 2005-2007 Marc Espie <espie@openbsd.org> # @@ -86,7 +86,8 @@ sub find_in_already_done my $r = $solver->check_lib_spec($solver->{localbase}, $obj, $self->{known}); if ($r) { - print "found libspec $obj in package $r\n" if $state->{verbose}; + $state->say("found libspec $obj in package $r") + if $state->{verbose}; return $r; } else { return undef; @@ -101,7 +102,8 @@ sub find_in_extra_sources OpenBSD::SharedLibs::add_libs_from_system($state->{destdir}); for my $dir (OpenBSD::SharedLibs::system_dirs()) { if ($solver->check_lib_spec($dir, $obj, {system => 1})) { - print "found libspec $obj in $dir/lib\n" if $state->{verbose}; + $state->say("found libspec $obj in $dir/lib") + if $state->{verbose}; return 'system'; } } @@ -114,7 +116,8 @@ sub find_in_new_source OpenBSD::SharedLibs::add_libs_from_installed_package($dep); if ($solver->check_lib_spec($solver->{localbase}, $obj, {$dep => 1})) { - print "found libspec $obj in package $dep\n" if $state->{verbose}; + $state->say("found libspec $obj in package $dep") + if $state->{verbose}; return $dep; } return undef; @@ -129,7 +132,8 @@ sub find_elsewhere my $r = $solver->find_old_lib($state, $solver->{localbase}, $dep->{pattern}, $obj); if ($r) { - print "found libspec $obj in old package $r\n" if $state->{verbose}; + $state->say("found libspec $obj in old package $r") + if $state->{verbose}; return $r; } } @@ -152,7 +156,7 @@ sub find_in_already_done my ($self, $solver, $state, $obj) = @_; my $r = $self->{known_tags}->{$obj}; if (defined $r) { - print "Found tag $obj in $r\n" if $state->{verbose}; + $state->say("Found tag $obj in $r") if $state->{verbose}; } return $r; } @@ -392,16 +396,17 @@ sub adjust_old_dependencies my $oldname = $o->pkgname; - print "Adjusting dependencies for $pkgname/$oldname\n" - if $state->{beverbose}; + $state->say("Adjusting dependencies for ", + "$pkgname/$oldname") if $state->{beverbose}; my $d = OpenBSD::RequiredBy->new($pkgname); for my $dep (@{$o->{wantlist}}) { if (defined $self->{set}->{skipupdatedeps}->{$dep}) { - print "\tskipping $dep\n" + $state->say("\tskipping $dep") if $state->{beverbose}; next; } - print "\t$dep\n" if $state->{beverbose}; + $state->say("\t$dep") + if $state->{beverbose}; $d->add($dep); OpenBSD::Replace::adjust_dependency($dep, $oldname, $pkgname); diff --git a/usr.sbin/pkg_add/OpenBSD/Handle.pm b/usr.sbin/pkg_add/OpenBSD/Handle.pm index 64fd15014da..96006cf07b9 100644 --- a/usr.sbin/pkg_add/OpenBSD/Handle.pm +++ b/usr.sbin/pkg_add/OpenBSD/Handle.pm @@ -1,5 +1,5 @@ # ex:ts=8 sw=4: -# $OpenBSD: Handle.pm,v 1.10 2009/11/10 11:36:56 espie Exp $ +# $OpenBSD: Handle.pm,v 1.11 2009/11/11 12:04:19 espie Exp $ # # Copyright (c) 2007-2009 Marc Espie <espie@openbsd.org> # @@ -170,19 +170,18 @@ sub get_plist my $pkg = $handle->pkgname; if ($state->{verbose}) { - print $state->deptree_header($pkg); - print "parsing $pkg\n"; + $state->say($state->deptree_header($pkg), "parsing $pkg"); } my $plist = $location->grabPlist; unless (defined $plist) { - print "Can't find CONTENTS from ", $location->url, "\n"; + $state->say("Can't find CONTENTS from ", $location->url); $location->close_with_client_error; $location->wipe_info; $handle->set_error(BAD_PACKAGE); return; } if ($plist->localbase ne $state->{localbase}) { - print "Localbase mismatch: package has: ", $plist->localbase, " , user wants: ", $state->{localbase}, "\n"; + $state->say("Localbase mismatch: package has: ", $plist->localbase, " , user wants: ", $state->{localbase}); $location->close_with_client_error; $location->wipe_info; $handle->set_error(BAD_PACKAGE); @@ -197,8 +196,8 @@ sub get_plist !$plist->uses_old_libs)) { $handle->{tweaked} = OpenBSD::Add::tweak_package_status($pkgname, $state); - print "Not reinstalling $pkgname\n" if $state->{verbose} and - !$handle->{tweaked}; + $state->say("Not reinstalling $pkgname") + if $state->{verbose} and !$handle->{tweaked}; $state->tracker->{installed}->{$pkgname} = 1; $location->close_now; $location->wipe_info; @@ -207,7 +206,7 @@ sub get_plist } if ($pkg ne '-') { if (!defined $pkgname or $pkg ne $pkgname) { - print "Package name is not consistent ???\n"; + $state->say("Package name is not consistent ???"); $location->close_with_client_error; $location->wipe_info; $handle->set_error(BAD_PACKAGE); @@ -225,13 +224,13 @@ sub get_location my $location = OpenBSD::PackageLocator->find($name, $state->{arch}); if (!$location) { - print $state->deptree_header($name); + $state->print($state->deptree_header($name)); $handle->set_error(NOT_FOUND); $handle->{tweaked} = OpenBSD::Add::tweak_package_status($handle->pkgname, $state); if (!$handle->{tweaked}) { - print "Can't find $name\n"; + $state->say("Can't find $name"); } return; } diff --git a/usr.sbin/pkg_add/OpenBSD/PackageInfo.pm b/usr.sbin/pkg_add/OpenBSD/PackageInfo.pm index 2bec8fb57e6..ac939502304 100644 --- a/usr.sbin/pkg_add/OpenBSD/PackageInfo.pm +++ b/usr.sbin/pkg_add/OpenBSD/PackageInfo.pm @@ -1,5 +1,5 @@ # ex:ts=8 sw=4: -# $OpenBSD: PackageInfo.pm,v 1.42 2009/11/11 11:18:24 espie Exp $ +# $OpenBSD: PackageInfo.pm,v 1.43 2009/11/11 12:04:19 espie Exp $ # # Copyright (c) 2003-2007 Marc Espie <espie@openbsd.org> # @@ -256,8 +256,9 @@ sub solve_installed_names if ($state->{defines}->{ambiguous}) { my @l = map {$_->name} @$r; - $state->print("Ambiguous: $pkgname could be ", join(' ', @l),"\n"); - print "$msg\n"; + $state->say("Ambiguous: $pkgname could be ", + join(' ', @l)); + $state->say($msg); push(@$new, @l); for my $p (@$r) { $seen->{$p} = 1; diff --git a/usr.sbin/pkg_add/OpenBSD/PackingElement.pm b/usr.sbin/pkg_add/OpenBSD/PackingElement.pm index 1ba458ee1d0..a36379356cc 100644 --- a/usr.sbin/pkg_add/OpenBSD/PackingElement.pm +++ b/usr.sbin/pkg_add/OpenBSD/PackingElement.pm @@ -1,5 +1,5 @@ # ex:ts=8 sw=4: -# $OpenBSD: PackingElement.pm,v 1.159 2009/11/11 11:13:16 espie Exp $ +# $OpenBSD: PackingElement.pm,v 1.160 2009/11/11 12:04:19 espie Exp $ # # Copyright (c) 2003-2007 Marc Espie <espie@openbsd.org> # @@ -410,7 +410,7 @@ sub check_digest " does not match"); } if ($state->{very_verbose}) { - print "Checksum match for ", $self->fullname, "\n"; + $state->say("Checksum match for ", $self->fullname); } } @@ -1195,7 +1195,8 @@ sub run my ($self, $state) = @_; OpenBSD::PackingElement::Lib::ensure_ldconfig($state); - print $self->keyword, " ", $self->{expanded}, "\n" if $state->{beverbose}; + $state->say($self->keyword, " ", $self->{expanded}) + if $state->{beverbose}; $state->system(OpenBSD::Paths->sh, '-c', $self->{expanded}) unless $state->{not}; } @@ -1376,7 +1377,8 @@ sub finish_fontdirs require OpenBSD::Error; map { update_fontalias($_) } @l unless $state->{not}; - print "You may wish to update your font path for ", join(' ', @l), "\n"; + $state->say("You may wish to update your font path for ", + join(' ', @l)); return if $state->{not}; run_if_exists($state, OpenBSD::Paths->mkfontscale, @l); run_if_exists($state, OpenBSD::Paths->mkfontdir, @l); @@ -1508,7 +1510,8 @@ sub run return if $state->{dont_run_scripts}; OpenBSD::PackingElement::Lib::ensure_ldconfig($state); - print $self->beautify, " script: $name $pkgname ", join(' ', @args), "\n" if $state->{beverbose}; + $state->say($self->beautify, " script: $name $pkgname ", + join(' ', @args)) if $state->{beverbose}; return if $not; chmod 0755, $name; return if $state->system($name, $pkgname, @args) == 0; diff --git a/usr.sbin/pkg_add/OpenBSD/ProgressMeter.pm b/usr.sbin/pkg_add/OpenBSD/ProgressMeter.pm index 2543feb63f4..e70984071c4 100644 --- a/usr.sbin/pkg_add/OpenBSD/ProgressMeter.pm +++ b/usr.sbin/pkg_add/OpenBSD/ProgressMeter.pm @@ -1,5 +1,5 @@ # ex:ts=8 sw=4: -# $OpenBSD: ProgressMeter.pm,v 1.13 2009/11/10 11:36:56 espie Exp $ +# $OpenBSD: ProgressMeter.pm,v 1.14 2009/11/11 12:04:19 espie Exp $ # # Copyright (c) 2004-2007 Marc Espie <espie@openbsd.org> # @@ -34,7 +34,12 @@ our %sizeof; sub new { my $class = shift; - $isatty = -t STDERR; + $isatty = -t STDOUT; + if ($isatty) { + my $oldfh = select(STDOUT); + $| = 1; + select($oldfh); + } return bless {}, $class; } @@ -47,7 +52,7 @@ sub find_window_size $sizeof{'struct winsize'} = 8; require 'sys/ttycom.ph'; $width = 80; - if (ioctl(STDERR, &TIOCGWINSZ, $r)) { + if (ioctl(STDOUT, &TIOCGWINSZ, $r)) { my ($rows, $cols, $xpix, $ypix) = unpack($wsz_format, $r); $width = $cols; @@ -100,7 +105,7 @@ sub message return if $d eq $lastdisplay && !$continued; $lastdisplay=$d; $continued = 0; - print STDERR $d, "\r"; + print $d, "\r"; } sub show @@ -119,20 +124,20 @@ sub show } return if $d eq $lastdisplay; $lastdisplay=$d; - print STDERR $d, "\r"; + print $d, "\r"; } sub clear { my $self = shift; return unless $isatty; - print STDERR ' 'x length($lastdisplay), "\r"; + print ' 'x length($lastdisplay), "\r"; } sub print { shift->clear; - print STDERR @_; + print @_; } sub next @@ -140,7 +145,7 @@ sub next my $self = shift; return unless $isatty; $self->clear; - print STDERR"$header: complete\n"; + print "$header: complete\n"; } 1; diff --git a/usr.sbin/pkg_add/OpenBSD/Replace.pm b/usr.sbin/pkg_add/OpenBSD/Replace.pm index d0adb8e56e1..2de93736369 100644 --- a/usr.sbin/pkg_add/OpenBSD/Replace.pm +++ b/usr.sbin/pkg_add/OpenBSD/Replace.pm @@ -1,5 +1,5 @@ # ex:ts=8 sw=4: -# $OpenBSD: Replace.pm,v 1.52 2009/11/10 11:36:56 espie Exp $ +# $OpenBSD: Replace.pm,v 1.53 2009/11/11 12:04:19 espie Exp $ # # Copyright (c) 2004-2006 Marc Espie <espie@openbsd.org> # @@ -116,7 +116,8 @@ sub extract $d = dirname($d); } if ($state->{not}) { - print "extracting tempfile under $d\n" if $state->{very_verbose}; + $state->say("extracting tempfile under $d") + if $state->{very_verbose}; $state->{archive}->skip; } else { if (!-e _) { @@ -124,7 +125,7 @@ sub extract } my ($fh, $tempname) = OpenBSD::Temp::permanent_file($d, "pkg"); - print "extracting $tempname\n" if $state->{very_verbose}; + $state->say("extracting $tempname") if $state->{very_verbose}; $self->{tempname} = $tempname; # XXX don't apply destdir twice @@ -144,7 +145,8 @@ sub extract return if -e $destdir.$fullname; $self->SUPER::extract($state); - print "new directory ", $destdir, $fullname, "\n" if $state->{very_verbose}; + $state->say("new directory ", $destdir, $fullname) + if $state->{very_verbose}; return if $state->{not}; File::Path::mkpath($destdir.$fullname); } @@ -335,8 +337,8 @@ sub can_old_package_be_replaced push(@r, $wanting) if !defined $ignore->{$wanting}; } if (@r) { - print "Verifying dependencies still match for ", - join(', ', @r), "\n" if $state->{verbose}; + $state->say("Verifying dependencies still match for ", + join(', ', @r)) if $state->{verbose}; for my $wanting (@wantlist) { my $p2 = OpenBSD::PackingList->from_installation( $wanting, \&OpenBSD::PackingList::DependOnly); @@ -413,12 +415,12 @@ sub adjust_depends_closure { my ($oldname, $plist, $state) = @_; - print "Packages that depend on those shared libraries:\n" + $state->say("Packages that depend on those shared libraries:") if $state->{beverbose}; my $write = OpenBSD::RequiredBy->new($plist->pkgname); for my $pkg (OpenBSD::RequiredBy->compute_closure($oldname)) { - print "\t$pkg\n" if $state->{beverbose}; + $state->say("\t", $pkg) if $state->{beverbose}; $write->add($pkg); OpenBSD::Requiring->new($pkg)->add($plist->pkgname); } @@ -435,7 +437,7 @@ sub save_old_libraries my $libs = {}; my $p = {}; - print "Looking for changes in shared libraries\n" + $state->say("Looking for changes in shared libraries") if $state->{beverbose}; $o->{plist}->mark_lib($libs, $p); for my $n ($set->newer) { @@ -443,12 +445,14 @@ sub save_old_libraries } if (%$libs) { - print "Libraries to keep: ", join(",", sort(keys %$libs)), "\n" + $state->say("Libraries to keep: ", + join(",", sort(keys %$libs))) if $state->{verbose}; ($o->{plist}, my $stub_list) = split_libs($o->{plist}, $libs); my $stub_name = $stub_list->pkgname; my $dest = installed_info($stub_name); - print "Keeping them in $stub_name\n" if $state->{verbose}; + $state->say("Keeping them in $stub_name") + if $state->{verbose}; if ($state->{not}) { $stub_list->to_cache; $o->{plist}->to_cache; @@ -470,7 +474,8 @@ sub save_old_libraries adjust_depends_closure($oldname, $stub_list, $state); } else { - print "No libraries to keep\n" if $state->{verbose}; + $state->say("No libraries to keep") + if $state->{verbose}; } } } diff --git a/usr.sbin/pkg_add/OpenBSD/Update.pm b/usr.sbin/pkg_add/OpenBSD/Update.pm index 261656fa470..757a76a424d 100644 --- a/usr.sbin/pkg_add/OpenBSD/Update.pm +++ b/usr.sbin/pkg_add/OpenBSD/Update.pm @@ -1,5 +1,5 @@ # ex:ts=8 sw=4: -# $OpenBSD: Update.pm,v 1.92 2009/11/11 11:18:24 espie Exp $ +# $OpenBSD: Update.pm,v 1.93 2009/11/11 12:04:19 espie Exp $ # # Copyright (c) 2004-2006 Marc Espie <espie@openbsd.org> # @@ -44,14 +44,12 @@ sub process_handle my ($self, $set, $h, $state) = @_; my $pkgname = $h->pkgname; if (defined $h->{update}) { - $state->progress->clear; - print "Update to $pkgname already found\n"; + $state->say("Update to $pkgname already found"); return 0; } if ($pkgname =~ m/^(?:\.libs\d*|partial)\-/o) { - $state->progress->clear; - print "Not updating $pkgname, remember to clean it\n"; + $state->say("Not updating $pkgname, remember to clean it"); return 0; } my @search = (); @@ -112,8 +110,7 @@ sub process_handle } if (@$l == 1) { if ($state->{defines}->{pkgpath}) { - $state->progress->clear; - print "Directly updating $pkgname -> ", $l->[0]->name, "\n"; + $state->say("Directly updating $pkgname -> ", $l->[0]->name); $self->add_updateset($set, $h, $l->[0]); return 1; } @@ -121,12 +118,13 @@ sub process_handle !$plist->uses_old_libs && !$state->{defines}->{installed}) { my $msg = "No need to update $pkgname"; $state->progress->message($msg); - print "$msg\n" if $state->{beverbose}; + $state->say($msg) if $state->{beverbose}; return 0; } } - $state->print("Candidates for updating $pkgname -> ", join(' ', map {$_->name} @$l), "\n"); + $state->say("Candidates for updating $pkgname -> ", + join(' ', map {$_->name} @$l)); my $r = $state->choose_location($pkgname, $l); if (defined $r) { diff --git a/usr.sbin/pkg_add/OpenBSD/UpdateSet.pm b/usr.sbin/pkg_add/OpenBSD/UpdateSet.pm index 19b44016c46..8c3012b52d7 100644 --- a/usr.sbin/pkg_add/OpenBSD/UpdateSet.pm +++ b/usr.sbin/pkg_add/OpenBSD/UpdateSet.pm @@ -1,5 +1,5 @@ # ex:ts=8 sw=4: -# $OpenBSD: UpdateSet.pm,v 1.18 2009/11/11 11:17:11 espie Exp $ +# $OpenBSD: UpdateSet.pm,v 1.19 2009/11/11 12:04:19 espie Exp $ # # Copyright (c) 2007 Marc Espie <espie@openbsd.org> # @@ -94,6 +94,12 @@ sub print $self->progress->print(@_); } +sub say +{ + my $self = shift; + $self->progress->print(@_, "\n"); +} + sub system { my $self = shift; @@ -132,7 +138,7 @@ sub choose_location { my ($state, $name, $list) = @_; if (@$list == 0) { - $state->print("Can't find $name\n"); + $state->say("Can't find $name"); return undef; } elsif (@$list == 1) { return $list->[0]; @@ -147,8 +153,7 @@ sub choose_location my $result = OpenBSD::Interactive::ask_list("Ambiguous: choose package for $name", 1, sort keys %h); return $h{$result}; } else { - $state->print("Ambiguous: $name could be ", - join(' ', keys %h), "\n"); + $state->say("Ambiguous: $name could be ", join(' ', keys %h)); return undef; } } @@ -168,7 +173,7 @@ sub set_header {} sub print { shift; - print STDERR @_; + print @_; } # an UpdateSet is a list of packages to remove/install. diff --git a/usr.sbin/pkg_add/pkg_add b/usr.sbin/pkg_add/pkg_add index 56021dc8896..335be41175b 100644 --- a/usr.sbin/pkg_add/pkg_add +++ b/usr.sbin/pkg_add/pkg_add @@ -1,7 +1,7 @@ #! /usr/bin/perl # ex:ts=8 sw=4: -# $OpenBSD: pkg_add,v 1.376 2009/11/11 11:13:16 espie Exp $ +# $OpenBSD: pkg_add,v 1.377 2009/11/11 12:04:18 espie Exp $ # # Copyright (c) 2003-2009 Marc Espie <espie@openbsd.org> # @@ -58,7 +58,7 @@ sub has_new_sig if (!defined $plist->{new_sig}) { my $n = OpenBSD::PackingList->from_installation($plist->pkgname)->signature; my $o = $plist->signature; - print "Comparing full signature for ", $plist->pkgname, " \"$o\" vs. \"$n\": ", $n eq $o ? "equal\n" : "different\n" + $state->say("Comparing full signature for ", $plist->pkgname, " \"$o\" vs. \"$n\": ", $n eq $o ? "equal" : "different") if $state->{very_verbose}; $plist->{new_sig} = $n ne $o; } @@ -205,16 +205,16 @@ sub can_install if (!$state->{allow_replacing}) { if ($state->{defines}->{conflicts}) { - print "Forcing install of $pkgname in the presence of conflicts (",join(',', @conflicts), ")\n"; + $state->say("Forcing install of $pkgname in the presence of conflicts (",join(',', @conflicts), ")"); return 1; } - print "Can't install $pkgname because of conflicts (",join(',', @conflicts), ")\n"; + $state->say("Can't install $pkgname because of conflicts (",join(',', @conflicts), ")"); $handle->set_error(OpenBSD::Handle::CANT_INSTALL); return; } if (@conflicts > 5) { - print "Can't install $pkgname because of conflicts (",join(',', @conflicts), ")\n"; + $state->say("Can't install $pkgname because of conflicts (",join(',', @conflicts), ")"); $handle->set_error(OpenBSD::Handle::CANT_INSTALL); return; } @@ -222,7 +222,7 @@ sub can_install require OpenBSD::Replace; if (!OpenBSD::Replace::is_new_package_safe($plist, $state)) { - print "Can't safely update to $pkgname (use -F update to force it)\n"; + $state->say("Can't safely update to $pkgname (use -F update to force it)"); $handle->set_error(OpenBSD::Handle::CANT_INSTALL); return; } @@ -244,7 +244,7 @@ sub can_install if (!OpenBSD::Replace::can_old_package_be_replaced($old_handle->plist, $pkgname, $state, \%conflicts)) { - print "Can't update $toreplace into $pkgname\n"; + $state->say("Can't update $toreplace into $pkgname"); $handle->set_error(OpenBSD::Handle::CANT_INSTALL); return; } @@ -485,7 +485,7 @@ sub install_set if ($handle->plist->has('arch')) { unless ($handle->plist->{arch}->check($state->{arch})) { - print $handle->pkgname, " is not for the right architecture\n"; + $state->say($handle->pkgname, " is not for the right architecture"); return () unless $defines{arch}; } } @@ -508,8 +508,8 @@ sub install_set if (@baddeps) { for my $handle ($set->newer) { - print "Can't install ", $handle->pkgname, - ": can't resolve ", join(',', @baddeps), "\n"; + $state->say("Can't install ", $handle->pkgname, + ": can't resolve ", join(',', @baddeps)); $handle->location->close_now; } $bad++; @@ -721,7 +721,7 @@ if ($opt_l || $opt_z) { $l = $inst->find($pkgname, $state->{arch}); } if (!defined $l) { - print "Problem finding $pkgname\n"; + $state->say("Problem finding $pkgname"); } else { my $set = OpenBSD::UpdateSet->new->add_older(OpenBSD::Handle->from_location($l)); my $problem = 0; @@ -742,7 +742,7 @@ if ($opt_l || $opt_z) { my @cantupdate = $state->tracker->cant_update; if (@cantupdate > 0) { - print "Cannot find updates for ", join(', ', (@cantupdate > 50 ? @cantupdate[1..50] : @cantupdate)); + $state->print("Cannot find updates for ", join(', ', (@cantupdate > 50 ? @cantupdate[1..50] : @cantupdate))); print " and more" if @cantupdate > 50; print "\n"; if (@todo2 == 0) { @@ -760,10 +760,11 @@ if ($opt_l || $opt_z) { } } if (defined $state->{issues}) { - print "There are some ambiguities. Please run in interactive ". - "mode again.\n"; + $state->say("There are some ambiguities. ", + "Please run in interactive mode again."); } - print "Running update ", join(',', map {$_->short_print} @todo2), "\n"; + $state->say("Running update ", + join(',', map {$_->short_print} @todo2)); } else { # 3/ actual names find_truelocations(\@ARGV, \@todo2, $state); @@ -801,7 +802,7 @@ OpenBSD::PackingElement::Lib::ensure_ldconfig($state); # and display delayed thingies. if ($state->{packages_with_sig}) { - print "Packages with signatures: ", $state->{packages_with_sig}; + $state->print("Packages with signatures: ", $state->{packages_with_sig}); if ($state->{packages_without_sig}) { print ". UNSIGNED PACKAGES: ", join(', ', keys %{$state->{packages_without_sig}}); } diff --git a/usr.sbin/pkg_add/pkg_delete b/usr.sbin/pkg_add/pkg_delete index 04d44e0477d..7f5c9c3c96c 100644 --- a/usr.sbin/pkg_add/pkg_delete +++ b/usr.sbin/pkg_add/pkg_delete @@ -1,6 +1,6 @@ #!/usr/bin/perl # ex:ts=8 sw=4: -# $OpenBSD: pkg_delete,v 1.117 2009/11/11 11:13:16 espie Exp $ +# $OpenBSD: pkg_delete,v 1.118 2009/11/11 12:04:18 espie Exp $ # # Copyright (c) 2003-2007 Marc Espie <espie@openbsd.org> # @@ -124,8 +124,9 @@ if (@todo > @realnames) { my $details = $state->{very_verbose} || $defines{verbosedeps}; my $show = sub { my ($p, $d) = @_; - print "Can't remove ".join(' ', @$p)." without also removing:\n" - .join(' ', @$d)."\n"; + $state->say("Can't remove ", join(' ', @$p), + " without also removing:\n", + join(' ', @$d)); }; if ($state->{interactive} || !$details) { my %deps = map {($_, 1)} @todo; @@ -150,7 +151,7 @@ if (@todo > @realnames) { my $them = @todo > 1 ? 'them' : 'it'; if ($defines{dependencies} or OpenBSD::Interactive::confirm("Do you want to remove $them as well", $state->{interactive}, 0)) { - print "(removing $them as well)\n"; + $state->say("(removing $them as well)"); } else { $bad = 1; } @@ -174,7 +175,7 @@ eval { DELETE: for my $pkgname (@todo) { next if $done{$pkgname}; unless (is_installed($pkgname)) { - print "$pkgname was not installed\n"; + $state->say("$pkgname was not installed"); $done{$pkgname} = 1; $removed++; next; |