On Wed, 24 Sep 2025 00:07:04 +0200, Lexi Winter wrote:
> 
> The branch main has been updated by ivy:
> 
> URL: 
> https://cgit.FreeBSD.org/src/commit/?id=49e44e06d1ade850cff5e84bf8c5fc425e4411c8
> 
> commit 49e44e06d1ade850cff5e84bf8c5fc425e4411c8
> Author:     Lexi Winter <[email protected]>
> AuthorDate: 2025-09-23 21:57:10 +0000
> Commit:     Lexi Winter <[email protected]>
> CommitDate: 2025-09-23 22:06:33 +0000
> 
>     man7: Move atf.7 and tests.7 to the right package
>     
>     atf.7 should be in -atf, and tests.7 should be in -tests.
>     
>     MFC after:      1 day
>     Differential Revision:  https://reviews.freebsd.org/D52661
> ---
>  share/man/man7/Makefile | 18 +++++++++++++-----
>  1 file changed, 13 insertions(+), 5 deletions(-)
> 
> diff --git a/share/man/man7/Makefile b/share/man/man7/Makefile
> index 1e50242a1754..ef8a5a555a07 100644
> --- a/share/man/man7/Makefile
> +++ b/share/man/man7/Makefile
> @@ -1,5 +1,7 @@
>  .include <src.opts.mk>
>  
> +MANGROUPS=   MAN
> +
>  #MISSING: eqnchar.7 ms.7 term.7
>  MAN= arch.7 \
>       ascii.7 \
> @@ -31,7 +33,6 @@ MAN=        arch.7 \
>       stats.7 \
>       stdint.7 \
>       sticky.7 \
> -     tests.7 \
>       tracing.7 \
>       tuning.7
>  
> @@ -54,14 +55,21 @@ MLINKS+= networking.7 wifi.7
>  .endif
>  
>  .if ${MK_TESTS} != "no"
> -ATF= ${SRCTOP}/contrib/atf
> -.PATH:       ${ATF}/doc
> -MAN+=        atf.7
> +MANGROUPS+=  ATF TESTS
> +
> +TESTS=               tests.7
> +TESTSPACKAGE=        tests
> +
> +ATF=         atf.7
> +ATFPACKAGE=  atf
> +
> +ATFPATH=     ${SRCTOP}/contrib/atf
> +.PATH:       ${ATFPATH}/doc
>  
>  CLEANFILES+= atf.7
>  atf.7: atf.7.in
>       sed -e 's,__DOCDIR__,/usr/share/doc/atf,g' \
> -         <"${ATF}/doc/atf.7.in" >atf.7
> +         <"${ATFPATH}/doc/atf.7.in" >atf.7
>  .endif
>  
>  .include <bsd.prog.mk>

According to commit e1c6a283f3eaacf77244672cc9a09aab482dfba6 tests.7
is installed unconditionally.

But this is no longer the case, right?

Should the man page be added to
tools/build/mk/OptionalObsoleteFiles.inc (MK_TESTS)?


Reply via email to