[src/trunk]: src/share/mk Explain why used to fold case when deciding if we a...

[christos <christos%NetBSD.org@localhost>]

details:   https://anonhg.NetBSD.org/src/rev/5f4f0772bdca
branches:  trunk
changeset: 334993:5f4f0772bdca
user:      christos <christos%NetBSD.org@localhost>
date:      Fri Dec 19 21:43:11 2014 +0000

description:
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.

diffstat:

 share/mk/bsd.man.mk |  19 +++++++++++++++----
 1 files changed, 15 insertions(+), 4 deletions(-)

diffs (54 lines):

diff -r 0d9fc008b00b -r 5f4f0772bdca share/mk/bsd.man.mk
--- a/share/mk/bsd.man.mk       Fri Dec 19 20:50:35 2014 +0000
+++ b/share/mk/bsd.man.mk       Fri Dec 19 21:43:11 2014 +0000
@@ -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 @@
 
 # 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 @@
 
 # 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 @@
 
 # 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