git: f183965bb8f5 - stable/14 - pkgbase: fix inclusion of tests in ssh, bsnmp, clibs-dev

From: Ed Maste <emaste_at_FreeBSD.org>
Date: Thu, 27 Feb 2025 19:45:50 UTC
The branch stable/14 has been updated by emaste:

URL: https://cgit.FreeBSD.org/src/commit/?id=f183965bb8f5cf68edbd60a7c39e20ad211d6bd5

commit f183965bb8f5cf68edbd60a7c39e20ad211d6bd5
Author:     Isaac Freund <ifreund@freebsdfoundation.org>
AuthorDate: 2024-12-23 21:54:29 +0000
Commit:     Ed Maste <emaste@FreeBSD.org>
CommitDate: 2025-02-27 19:45:40 +0000

    pkgbase: fix inclusion of tests in ssh, bsnmp, clibs-dev
    
    Currently, files that belong in the tests package are included in the
    ssh, bsnmp, and clibs-dev packages:
    
    ssh.plist
    24:@dir(root,wheel,0755,) /usr/tests/secure/libexec
    25:@(root,wheel,0444,) /usr/tests/secure/libexec/Kyuafile
    
    bsnmp.plist
    82:@dir(root,wheel,0755,) /usr/tests/lib/libbsnmp
    83:@(root,wheel,0444,) /usr/tests/lib/libbsnmp/Kyuafile
    84:@(root,wheel,0555,) /usr/tests/lib/libbsnmp/bsnmpd_test
    
    clibs-dev.plist
    2518:@dir(root,wheel,0755,) /usr/tests/lib/csu
    2519:@(root,wheel,0444,) /usr/tests/lib/csu/Kyuafile
    
    This is caused by the PACKAGE=foo assignment in foo/Makefile.inc which
    overrides the default PACKAGE?=tests in bsd.test.mk.
    
    To fix this, instead use PACKAGE?=foo in foo/Makefile.inc and set
    PACKAGE=tests in foo/tests/Makefile.
    
    PR:             249144
    Reviewed by:    bapt, emaste
    Sponsored by:   The FreeBSD Foundation
    Differential Revision: https://reviews.freebsd.org/D47025
    
    (cherry picked from commit 3a56015a2f5d630910177fa79a522bb95511ccf7)
---
 lib/csu/Makefile.inc          | 2 +-
 lib/csu/tests/Makefile        | 1 +
 lib/libbsnmp/Makefile.inc     | 2 +-
 lib/libbsnmp/tests/Makefile   | 2 ++
 secure/libexec/Makefile.inc   | 2 +-
 secure/libexec/tests/Makefile | 1 +
 6 files changed, 7 insertions(+), 3 deletions(-)

diff --git a/lib/csu/Makefile.inc b/lib/csu/Makefile.inc
index 1509a1ece2c3..1c9d3853b0d4 100644
--- a/lib/csu/Makefile.inc
+++ b/lib/csu/Makefile.inc
@@ -1,4 +1,4 @@
-PACKAGE=	clibs-dev
+PACKAGE?=	clibs-dev
 
 NO_WMISSING_VARIABLE_DECLARATIONS=
 # Can't instrument these files since that breaks non-sanitized programs.
diff --git a/lib/csu/tests/Makefile b/lib/csu/tests/Makefile
index 3f95193420ea..c9a0e6eff5da 100644
--- a/lib/csu/tests/Makefile
+++ b/lib/csu/tests/Makefile
@@ -1,3 +1,4 @@
+PACKAGE=	tests
 
 SUBDIR=		dso
 TESTS_SUBDIRS=	dynamic
diff --git a/lib/libbsnmp/Makefile.inc b/lib/libbsnmp/Makefile.inc
index 80478f392410..d9ae6f7deb8c 100644
--- a/lib/libbsnmp/Makefile.inc
+++ b/lib/libbsnmp/Makefile.inc
@@ -1,5 +1,5 @@
 
 INCSDIR=	${INCLUDEDIR}/bsnmp
-PACKAGE=	bsnmp
+PACKAGE?=	bsnmp
 
 .include "../Makefile.inc"
diff --git a/lib/libbsnmp/tests/Makefile b/lib/libbsnmp/tests/Makefile
index 529622104449..ca8bcda8401f 100644
--- a/lib/libbsnmp/tests/Makefile
+++ b/lib/libbsnmp/tests/Makefile
@@ -1,6 +1,8 @@
 
 .include <bsd.own.mk>
 
+PACKAGE=	tests
+
 ATF_TESTS_C+=	bsnmpd_test
 
 SRCS.bsmpd_test=	bsnmpd_test.c
diff --git a/secure/libexec/Makefile.inc b/secure/libexec/Makefile.inc
index e5beeed87cac..6cb7a9096b7b 100644
--- a/secure/libexec/Makefile.inc
+++ b/secure/libexec/Makefile.inc
@@ -1,5 +1,5 @@
 
 BINDIR?=	/usr/libexec
-PACKAGE=	ssh
+PACKAGE?=	ssh
 
 .include "../Makefile.inc"
diff --git a/secure/libexec/tests/Makefile b/secure/libexec/tests/Makefile
index b8996030aafc..b8e8a3828b9c 100644
--- a/secure/libexec/tests/Makefile
+++ b/secure/libexec/tests/Makefile
@@ -1,5 +1,6 @@
 
 .PATH:		${SRCTOP}/tests
 KYUAFILE=	yes
+PACKAGE=	tests
 
 .include <bsd.test.mk>