Re: git: 49e44e06d1ad - main - man7: Move atf.7 and tests.7 to the right package

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 <ivy@FreeBSD.org>
> AuthorDate: 2025-09-23 21:57:10 +0000
> Commit:     Lexi Winter <ivy@FreeBSD.org>
> 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)?