Module Name:    src
Committed By:   lukem
Date:           Sun Jun 21 03:39:21 UTC 2020

Modified Files:
        src/share/mk: bsd.dep.mk

Log Message:
bsd.dep.mk: fix "make tags"

Fix "make tags" to actually build a tags file:
- Use !commands() instead of !target(), so that the rule actually works
- Write to ${.OBJDIR}/tags for read-only source (don't know why ${.TARGET}
  isn't sufficient).
- Only match *.[cly] from ${.ALLSRCS} - just excluding *.h causes failures
  because of ${targ}: subdir-${targ} in bsd.subdir.mk.

Thanks to uwe@ for assistance.


To generate a diff of this commit:
cvs rdiff -u -r1.84 -r1.85 src/share/mk/bsd.dep.mk

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: src/share/mk/bsd.dep.mk
diff -u src/share/mk/bsd.dep.mk:1.84 src/share/mk/bsd.dep.mk:1.85
--- src/share/mk/bsd.dep.mk:1.84	Mon Jan 21 21:11:54 2019
+++ src/share/mk/bsd.dep.mk	Sun Jun 21 03:39:21 2020
@@ -1,4 +1,4 @@
-#	$NetBSD: bsd.dep.mk,v 1.84 2019/01/21 21:11:54 christos Exp $
+#	$NetBSD: bsd.dep.mk,v 1.85 2020/06/21 03:39:21 lukem Exp $
 
 ##### Basic targets
 realdepend:	beforedepend .depend afterdepend
@@ -94,16 +94,16 @@ _MKDEP_FILEFLAGS=
 
 ##### Clean rules
 .if defined(SRCS) && !empty(SRCS)
-CLEANDIRFILES+= .depend ${__DPSRCS.d} ${__DPSRCS.d:.d=.d.tmp} ${.CURDIR}/tags ${CLEANDEPEND}
+CLEANDIRFILES+= .depend ${__DPSRCS.d} ${__DPSRCS.d:.d=.d.tmp} tags ${CLEANDEPEND}
 .endif
 
 ##### Custom rules
-.if !target(tags)
+.if !commands(tags)
 tags: ${SRCS}
 .if defined(SRCS) && !empty(SRCS)
 	${_MKTARGET_CREATE}
-	-cd "${.CURDIR}"; ctags -f /dev/stdout ${.ALLSRC:N*.h} | \
-	    ${TOOL_SED} "s;\${.CURDIR}/;;" > tags
+	-cd "${.CURDIR}"; ctags -f /dev/stdout ${.ALLSRC:M*.[cly]} | \
+	    ${TOOL_SED} "s;\${.CURDIR}/;;" > ${.OBJDIR}/tags
 .endif
 .endif
 

Reply via email to