On 2023/08/17 20:23:48 +0200, Theo Buehler <t...@theobuehler.org> wrote:
> If I wanted to document them, I would grep the ports tree and see which
> ones are used most. Then send small diffs to document a group of those
> that belongs together.
> 
> > +.It MODCARGO_WANTLIB
> > +Needed Rust libraries, for use with WANTLIB.
> 
> I don't think it hurts to be a bit more explicit here. The reason is
> that using an explicit entry for c++abi means that WANTLIB are wrong for
> sparc64.
> 
> So I'd do something like this.

fwiw I'd like for this to be added.  I knew about MODCARGO_WANTLIB due
to reading the list, should be documented.

> Index: cargo-module.5
> ===================================================================
> RCS file: /cvs/src/share/man/man5/cargo-module.5,v
> retrieving revision 1.5
> diff -u -p -r1.5 cargo-module.5
> --- cargo-module.5    17 Aug 2023 05:43:09 -0000      1.5
> +++ cargo-module.5    17 Aug 2023 18:18:46 -0000
> @@ -106,6 +106,15 @@ Needs to be set for some virtual manifes
>  Name of the local directory for vendoring crates.
>  Defaults to
>  .Pa ${WRKSRC}/modcargo-crates .
> +.It MODCARGO_WANTLIB

not an issue with the diff itself since no item has it, but should we
amend the list to use Ev to mark up the variables?  i.e.

+.It Ev MODCARGO_WANTLIB

> +Architecture-specific
> +.Ev WANTLIB
> +entries required by all Rust code.
> +Should be used instead of explicit entries for
> +.Dq c ,
> +.Dq pthread ,
> +and
> +.Dq c++abi .
>  .El
>  .Pp
>  This module adds three


Reply via email to