[src/trunk]: src/share/mk protect the variable name, and explain why setting ...

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

details:   https://anonhg.NetBSD.org/src/rev/c1a8cfb1c1a0
branches:  trunk
changeset: 334994:c1a8cfb1c1a0
user:      christos <christos%NetBSD.org@localhost>
date:      Fri Dec 19 22:25:39 2014 +0000

description:
protect the variable name, and explain why setting it does not work.

diffstat:

 share/mk/bsd.man.mk |  14 +++++++-------
 1 files changed, 7 insertions(+), 7 deletions(-)

diffs (50 lines):

diff -r 5f4f0772bdca -r c1a8cfb1c1a0 share/mk/bsd.man.mk
--- a/share/mk/bsd.man.mk       Fri Dec 19 21:43:11 2014 +0000
+++ b/share/mk/bsd.man.mk       Fri Dec 19 22:25:39 2014 +0000
@@ -1,4 +1,4 @@
-#      $NetBSD: bsd.man.mk,v 1.116 2014/12/19 21:43:11 christos Exp $
+#      $NetBSD: bsd.man.mk,v 1.117 2014/12/19 22:25:39 christos Exp $
 #      @(#)bsd.man.mk  8.1 (Berkeley) 6/8/93
 
 .include <bsd.init.mk>
@@ -14,10 +14,10 @@
 # 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.
-
+# the CASE_INSENSITIVE_DEST macro is currently not defined anywhere,
+# and the expansion does not really work because of make(1).
 .if defined(CASE_INSENSITIVE_DEST)
-FLATTEN?=      :tl
+_FLATTEN?=tl:
 .endif
 
 ##### Default values
@@ -120,7 +120,7 @@
 
 # Handle case conflicts carefully, when _dst occurs
 # more than once after case flattening
-.if ${MKUPDATE} == "no" || ${MLINKS:${FLATTEN}M${_dst:${FLATTEN}Q}:[\#]} > 1
+.if ${MKUPDATE} == "no" || ${MLINKS:${_FLATTEN}M${_dst:${_FLATTEN}Q}:[\#]} > 1
 ${_t}!         ${_l} __linkinstallpage
 .else
 ${_t}:         ${_l} __linkinstallpage
@@ -188,7 +188,7 @@
 
 # Handle case conflicts carefully, when _dst occurs
 # more than once after case flattening
-.if ${MKUPDATE} == "no" || ${MLINKS:${FLATTEN}M${_dst:${FLATTEN}Q}:[\#]} > 1
+.if ${MKUPDATE} == "no" || ${MLINKS:${_FLATTEN}M${_dst:${_FLATTEN}Q}:[\#]} > 1
 ${_t}!         ${_l} __linkinstallpage
 .else
 ${_t}:         ${_l} __linkinstallpage
@@ -260,7 +260,7 @@
 
 # Handle case conflicts carefully, when _dst occurs
 # more than once after case flattening
-.if ${MKUPDATE} == "no" || ${MLINKS:${FLATTEN}M${_dst:${FLATTEN}Q}:[\#]} > 1
+.if ${MKUPDATE} == "no" || ${MLINKS:${_FLATTEN}M${_dst:${_FLATTEN}Q}:[\#]} > 1
 ${_t}!         ${_l} __linkinstallpage
 .else
 ${_t}:         ${_l} __linkinstallpage