git: 6647af9b56b3 - main - bsd.progs.mk must pass META_XTRAS to gendirdeps.mk

From: Simon J. Gerraty <sjg_at_FreeBSD.org>
Date: Sat, 04 Nov 2023 17:18:29 UTC
The branch main has been updated by sjg:

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

commit 6647af9b56b3998bc6e57c55a2a96115366bd62f
Author:     Simon J. Gerraty <sjg@FreeBSD.org>
AuthorDate: 2023-11-04 17:16:33 +0000
Commit:     Simon J. Gerraty <sjg@FreeBSD.org>
CommitDate: 2023-11-04 17:17:40 +0000

    bsd.progs.mk must pass META_XTRAS to gendirdeps.mk
    
    The indirection used by bsd.progs.mk is setting META_XTRAS
    means the value needs to be passed in the environment to
    gendirdeps.mk, as any expansion before then will be empty.
    
    Remove a now misleading comment from bsd.progs.mk
    before it includes bsd.prog.mk
    
    Update gendirdeps.mk to accommodate this.
    
    Reviewed by:    stevek
---
 share/mk/bsd.progs.mk  | 3 ++-
 share/mk/gendirdeps.mk | 6 +++++-
 2 files changed, 7 insertions(+), 2 deletions(-)

diff --git a/share/mk/bsd.progs.mk b/share/mk/bsd.progs.mk
index 946c44f76570..fba617b12595 100644
--- a/share/mk/bsd.progs.mk
+++ b/share/mk/bsd.progs.mk
@@ -64,6 +64,8 @@ all: ${PROGS}
 .endif
 
 META_XTRAS+=	${cat ${PROGS:S/$/*.meta_files/} 2>/dev/null || true:L:sh}
+# the above does no use unless we pass it on to gendirdeps.mk
+GENDIRDEPS_ENV += META_XTRAS='${META_XTRAS}'
 
 .if ${MK_STAGING} != "no" && !empty(PROGS)
 # Stage from parent while respecting PROGNAME and BINDIR overrides.
@@ -88,7 +90,6 @@ $v =
 .endfor
 .endif
 
-# handle being called [bsd.]progs.mk
 .include <bsd.prog.mk>
 
 .if !defined(_SKIP_BUILD)
diff --git a/share/mk/gendirdeps.mk b/share/mk/gendirdeps.mk
index ec4e188cdf78..5d54aa232152 100644
--- a/share/mk/gendirdeps.mk
+++ b/share/mk/gendirdeps.mk
@@ -1,4 +1,4 @@
-# $Id: gendirdeps.mk,v 1.49 2023/04/20 17:45:03 sjg Exp $
+# $Id: gendirdeps.mk,v 1.50 2023/11/04 16:47:34 sjg Exp $
 
 # SPDX-License-Identifier: BSD-2-Clause
 #
@@ -102,6 +102,10 @@ _DEPENDFILE := ${_CURDIR}/${.MAKE.DEPENDFILE:T}
 
 # caller should have set this
 META_FILES ?= ${.MAKE.META.FILES}
+# this sometimes needs to be passed separately
+.if !empty(META_XTRAS)
+META_FILES += ${META_XTRAS:N\*.meta}
+.endif
 
 .if !empty(META_FILES)