Module Name:    src
Committed By:   christos
Date:           Fri Dec 19 21:43:11 UTC 2014

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

Log Message:
Explain why used to fold case when deciding if we are going to install manual
page links, and change the behavior not to do this anymore.


To generate a diff of this commit:
cvs rdiff -u -r1.115 -r1.116 src/share/mk/bsd.man.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.man.mk
diff -u src/share/mk/bsd.man.mk:1.115 src/share/mk/bsd.man.mk:1.116
--- src/share/mk/bsd.man.mk:1.115	Tue Aug  5 11:40:59 2014
+++ src/share/mk/bsd.man.mk	Fri Dec 19 16:43:11 2014
@@ -1,4 +1,4 @@
-#	$NetBSD: bsd.man.mk,v 1.115 2014/08/05 15:40:59 apb Exp $
+#	$NetBSD: bsd.man.mk,v 1.116 2014/12/19 21:43:11 christos Exp $
 #	@(#)bsd.man.mk	8.1 (Berkeley) 6/8/93
 
 .include <bsd.init.mk>
@@ -9,6 +9,17 @@
 .PHONY:		lintmanpages
 realinstall:	${MANINSTALL}
 
+# If our install destination is case-preserving, but case-insensitive
+# then we do filesystem comparisons in lower case to make sure that
+# we always refresh the target when needed. In general we don't
+# want to do this, otherwise things like _exit.2 -> _Exit.2 get
+# installed on each build even when they don't need to. Note that
+# the CASE_INSENSITIVE_DEST macro is currently not defined anywhere.
+
+.if defined(CASE_INSENSITIVE_DEST)
+FLATTEN?=	:tl
+.endif
+
 ##### Default values
 .if ${USETOOLS} == "yes"
 TMACDEPDIR?=	${TOOLDIR}/share/groff/tmac
@@ -109,7 +120,7 @@ _t:=${DESTDIR}${MANDIR}/man${_dst:T:E}${
 
 # Handle case conflicts carefully, when _dst occurs
 # more than once after case flattening
-.if ${MKUPDATE} == "no" || ${MLINKS:tl:M${_dst:tl:Q}:[\#]} > 1
+.if ${MKUPDATE} == "no" || ${MLINKS:${FLATTEN}M${_dst:${FLATTEN}Q}:[\#]} > 1
 ${_t}!		${_l} __linkinstallpage
 .else
 ${_t}:		${_l} __linkinstallpage
@@ -177,7 +188,7 @@ _t:=${DESTDIR}${MANDIR}/cat${_dst:T:E}${
 
 # Handle case conflicts carefully, when _dst occurs
 # more than once after case flattening
-.if ${MKUPDATE} == "no" || ${MLINKS:tl:M${_dst:tl:Q}:[\#]} > 1
+.if ${MKUPDATE} == "no" || ${MLINKS:${FLATTEN}M${_dst:${FLATTEN}Q}:[\#]} > 1
 ${_t}!		${_l} __linkinstallpage
 .else
 ${_t}:		${_l} __linkinstallpage
@@ -249,7 +260,7 @@ _t:=${HTMLDIR}/html${_dst:T:E}${MANSUBDI
 
 # Handle case conflicts carefully, when _dst occurs
 # more than once after case flattening
-.if ${MKUPDATE} == "no" || ${MLINKS:tl:M${_dst:tl:Q}:[\#]} > 1
+.if ${MKUPDATE} == "no" || ${MLINKS:${FLATTEN}M${_dst:${FLATTEN}Q}:[\#]} > 1
 ${_t}!		${_l} __linkinstallpage
 .else
 ${_t}:		${_l} __linkinstallpage

Reply via email to