Re: [gentoo-dev] [PATCH 3/4] acct-group.eclass: EAPI 8 support

[David Michael]

On Tue, Jun 22, 2021 at 1:15 PM Ulrich Mueller <ulm@g.o> wrote:
> >>>>> On Tue, 22 Jun 2021, David Michael wrote:
> > -# Then you add appropriate dependency to your package.  The dependency
> > -# type(s) should be:
> > -# - DEPEND (+ RDEPEND) if the group is already needed at build time,
> > -# - RDEPEND if it is needed at install time (e.g. you 'fowners' files
> > -#   in pkg_preinst) or run time.
>
> "man acct-group.eclass" currently renders that paragraph like this:
>
>        Then you add appropriate dependency to your  package.   The  dependency
>        type(s)  should be: - DEPEND (+ RDEPEND) if the group is already needed
>        at build time, - RDEPEND if it is needed  at  install  time  (e.g.  you
>        'fowners' files
>          in pkg_preinst) or run time.
>
> In other words, there's very little formatting support in eclassdoc.
>
> > +# Then you add appropriate dependencies to your package.  Note that the build
> > +# system might need to resolve names, too.  The dependency type(s) should be:
> > +# - BDEPEND if the group must be resolvable at build time (e.g. it is used by
> > +#   'fowners' in src_install),
> > +# - IDEPEND if it must be resolvable at install time (e.g. it is used by
> > +#   'fowners' in pkg_preinst),
> > +# - RDEPEND in every case.
>
> So, while at it, could you reformat it that it will be better readable
> in the generated man page?
>
> (Same issue in acct-user.eclass.)

I made these changes and pushed the commits to
https://github.com/gentoo/gentoo/pull/21377 in case that's easier to
track changes and merge.  I could resend everything if that's
preferred.

Thanks.

David