On Sun, Oct 04, 2015 at 02:11:49PM -0400, Michael Reed wrote:
> The use of an ellipsis in manuals usually seems to already imply
> optional arguments, e.g., `.Op Ar file ...', so constructs like
> `Ar pkg-name Op Ar ...' seem redundant to me.
>
> While here, also denote the `pkg-name' argument to pkg_delete
> as optional, since you can do things like `pkg_delete -nX'.
>
fixed, thanks.
jmc
>
>
> Index: pkg_add.1
> ===================================================================
> RCS file: /cvs/src/usr.sbin/pkg_add/pkg_add.1,v
> retrieving revision 1.132
> diff -u -p -r1.132 pkg_add.1
> --- pkg_add.1 16 Apr 2015 20:01:39 -0000 1.132
> +++ pkg_add.1 4 Oct 2015 17:54:40 -0000
> @@ -31,7 +31,7 @@
> .Op Fl L Ar localbase
> .Op Fl l Ar file
> .Op Fl P Ar type
> -.Ar pkg-name Op Ar ...
> +.Ar pkg-name ...
> .Ek
> .Sh DESCRIPTION
> The
> Index: pkg_delete.1
> ===================================================================
> RCS file: /cvs/src/usr.sbin/pkg_add/pkg_delete.1,v
> retrieving revision 1.50
> diff -u -p -r1.50 pkg_delete.1
> --- pkg_delete.1 8 Apr 2015 17:25:58 -0000 1.50
> +++ pkg_delete.1 4 Oct 2015 17:54:40 -0000
> @@ -26,7 +26,7 @@
> .Op Fl acIimnqsvXx
> .Op Fl B Ar pkg-destdir
> .Op Fl D Ar name Ns Op = Ns Ar value
> -.Ar pkg-name Op Ar ...
> +.Op Ar pkg-name ...
> .Sh DESCRIPTION
> The
> .Nm
> Index: pkg_info.1
> ===================================================================
> RCS file: /cvs/src/usr.sbin/pkg_add/pkg_info.1,v
> retrieving revision 1.50
> diff -u -p -r1.50 pkg_info.1
> --- pkg_info.1 8 Sep 2014 01:27:55 -0000 1.50
> +++ pkg_info.1 4 Oct 2015 17:54:40 -0000
> @@ -30,8 +30,7 @@
> .Op Fl l Ar str
> .Op Fl Q Ar query
> .Op Fl r Ar pkgspec
> -.Op Ar pkg-name
> -.Op Ar ...
> +.Op Ar pkg-name ...
> .Ek
> .Sh DESCRIPTION
> The
> Index: OpenBSD/PkgAdd.pm
> ===================================================================
> RCS file: /cvs/src/usr.sbin/pkg_add/OpenBSD/PkgAdd.pm,v
> retrieving revision 1.87
> diff -u -p -r1.87 PkgAdd.pm
> --- OpenBSD/PkgAdd.pm 30 Jun 2015 19:20:08 -0000 1.87
> +++ OpenBSD/PkgAdd.pm 4 Oct 2015 17:54:40 -0000
> @@ -107,7 +107,7 @@ sub handle_options
> my $state = shift;
> $state->SUPER::handle_options('ruUzl:A:P:',
> '[-acinqrsUuvxz] [-A arch] [-B pkg-destdir] [-D name[=value]]',
> - '[-L localbase] [-l file] [-P type] pkg-name [...]');
> + '[-L localbase] [-l file] [-P type] pkg-name ...');
>
> $state->{arch} = $state->opt('A');
>
> Index: OpenBSD/PkgDelete.pm
> ===================================================================
> RCS file: /cvs/src/usr.sbin/pkg_add/OpenBSD/PkgDelete.pm,v
> retrieving revision 1.34
> diff -u -p -r1.34 PkgDelete.pm
> --- OpenBSD/PkgDelete.pm 29 Nov 2014 10:42:51 -0000 1.34
> +++ OpenBSD/PkgDelete.pm 4 Oct 2015 17:54:40 -0000
> @@ -101,7 +101,7 @@ sub handle_options
> {
> my $state = shift;
> $state->SUPER::handle_options('X',
> - '[-acimnqsvXx] [-B pkg-destdir] [-D name[=value]] pkg-name [...]');
> + '[-acimnqsvXx] [-B pkg-destdir] [-D name[=value]] [pkg-name ...]');
>
> my $base = $state->opt('B') // $ENV{'PKG_DESTDIR'} // '';
> if ($base ne '') {
> Index: OpenBSD/PkgInfo.pm
> ===================================================================
> RCS file: /cvs/src/usr.sbin/pkg_add/OpenBSD/PkgInfo.pm,v
> retrieving revision 1.35
> diff -u -p -r1.35 PkgInfo.pm
> --- OpenBSD/PkgInfo.pm 6 Apr 2015 12:19:35 -0000 1.35
> +++ OpenBSD/PkgInfo.pm 4 Oct 2015 17:54:40 -0000
> @@ -559,7 +559,7 @@ sub parse_and_run
> $state->{no_exports} = 1;
> $state->handle_options('cCdfF:hIKLmPQ:qr:RsSUe:E:Ml:aAt',
> '[-AaCcdfIKLMmPqRSstUv] [-D nolock][-E filename] [-e pkg-name] ',
> - '[-l str] [-Q query] [-r pkgspec] [pkg-name] [...]');
> + '[-l str] [-Q query] [-r pkgspec] [pkg-name ...]');
>
> if ($state->opt('r')) {
>
>
