Re: pkg_add: show -n updates without a progress meter

On Sun, 05 Jul 2026 01:14:50 +0200, Kirill A. Korinsky <kirill@korins.ky> wrote: > > On Thu, 25 Jun 2026 12:51:52 +0200, > Marc Espie <marc.espie.openbsd@gmail.com> wrote: > > > > > @@ -374,7 +374,10 @@ which is more discriminating. > > > Force update even if dependencies no longer match. > > > .El > > > .It Fl I > > > -Force non-interactive mode. > > > +Force non-interactive mode, disabling prompts. > > > +This does not affect progress meter output; use > > > +.Fl x > > > +to disable it. > > I disagree with this manpage change. What do you think non-interactive mode > > means ? > > > > As for progress meter and interactive mode being orthogonal options, that's > > the same thing. > > Drop that hunk. > > Ok? > Anyone? Index: usr.sbin/pkg_add/pkg_add.1 =================================================================== RCS file: /home/cvs/src/usr.sbin/pkg_add/pkg_add.1,v diff -u -p -r1.170 pkg_add.1 --- usr.sbin/pkg_add/pkg_add.1 22 May 2025 01:40:21 -0000 1.170 +++ usr.sbin/pkg_add/pkg_add.1 4 Jun 2026 20:53:16 -0000 @@ -416,8 +419,8 @@ Causes .Nm to always display the progress meter in cases it would not do so by default. .It Fl n -Don't actually install a package, just report the steps that -would be taken if it was. +Don't actually install a package, just report package additions or updates +that would be performed. Will still copy packages to .Ev PKG_CACHE if applicable. Index: usr.sbin/pkg_add/OpenBSD/PkgAdd.pm =================================================================== RCS file: /home/cvs/src/usr.sbin/pkg_add/OpenBSD/PkgAdd.pm,v diff -u -p -r1.152 PkgAdd.pm --- usr.sbin/pkg_add/OpenBSD/PkgAdd.pm 6 May 2025 18:36:20 -0000 1.152 +++ usr.sbin/pkg_add/OpenBSD/PkgAdd.pm 4 Jun 2026 20:45:49 -0000 @@ -831,8 +831,12 @@ sub really_add($set, $state) $set, $state)); } }); - $set->setup_header($state); - $state->progress->next($state->ntogo(-1)); + my $shown = $set->setup_header($state); + my $todo = $state->ntogo(-1); + $state->progress->next($todo); + if ($state->{not} && !$shown) { + $state->say("#1: #2", $state->{setheader}, $todo); + } for my $handle ($set->newer) { my $pkgname = $handle->pkgname; my $plist = $handle->plist; -- wbr, Kirill