2015-09-26 21:49 GMT+03:00 Michael Reed <m.r...@mykolab.com>:
> The `clean' target takes optional arguments, so denote that in
> the item header.  Additionally, the subtargets are fixed strings,
> not variables, so change the use of Va to Cm to reflect that.

Well, technically "clean=" is not a target, but a hack that looks like
a parametrized target. As a result, the ordering in the following
cases will differ, and that matters:

make package clean
make package clean=work

Even worse, the wording is confusing, because the term "subtarget" is
used in other parts of this manual page with a different semantics.

Can't comment on mdoc macro choosing, sorry.

> Index: bsd.port.mk.5
> ===================================================================
> RCS file: /cvs/src/share/man/man5/bsd.port.mk.5,v
> retrieving revision 1.426
> diff -u -p -r1.426 bsd.port.mk.5
> --- bsd.port.mk.5       23 Sep 2015 01:38:36 -0000      1.426
> +++ bsd.port.mk.5       26 Sep 2015 18:46:05 -0000
> @@ -178,47 +178,49 @@ will try to fetch a version with the cor
>  main archive site in the case of a checksum mismatch.
>  .Ev NO_CHECKSUM
>  can be used to avoid all checksumming steps.
> -.It Cm clean
> +.It Cm clean Ns Op Pf = Ar subtarget
>  Clean ports contents.
> -By default, it will clean the work directory.
> -It can be invoked as
> -make clean='[depends build bulk work fake flavors dist install sub package
> -packages plist]'.
> +If
> +.Ar subtarget
> +is omitted, it will clean the work directory.
> +Otherwise,
> +.Ar subtarget
> +can be set to one of the following:
>  .Bl -tag -width packages
> -.It Va work
> +.It Cm work
>  Clean work directory.
> -.It Va bulk
> +.It Cm bulk
>  Clean bulk cookie.
> -.It Va build
> +.It Cm build
>  Clean the
>  .Va WRKBUILD
>  directory (only useful if
>  .Va SEPARATE_BUILD
>  is set).
> -.It Va depends
> +.It Cm depends
>  Recurse into dependencies.
> -.It Va dist
> +.It Cm dist
>  Clean distribution files.
> -.It Va fake
> +.It Cm fake
>  Clean fake installation directory.
> -.It Va flavors
> +.It Cm flavors
>  Clean all work directories.
> -.It Va install
> +.It Cm install
>  Uninstall package.
> -.It Va package
> +.It Cm package
>  Remove all copies of package file.
> -.It Va plist
> +.It Cm plist
>  Remove registered packing lists of all subpackages.
> -.It Va sub
> +.It Cm sub
>  With
> -.Va install
> +.Cm install
>  or
> -.Va package ,
> +.Cm package ,
>  clean subpackages as well.
> -.It Va packages
> +.It Cm packages
>  Shorthand for
>  .Sq sub package .
> -.It Va all
> +.It Cm all
>  Shorthand for
>  .Sq work flavors packages plist .
>  .El
>

--
  WBR,
  Vadim Zhukov

Reply via email to