diff options
| author | dsl <dsl> | 2008-12-23 22:34:26 +0000 |
|---|---|---|
| committer | dsl <dsl> | 2008-12-23 22:34:26 +0000 |
| commit | 8d019d64c92596d48308fde544e7ad3abdff4876 (patch) | |
| tree | 0d9f8c100784ab9cde51ad88eb17a8ed57e24792 /mk | |
| parent | 0663d96cae3c4142bc1d3045139d0fcfcf316750 (diff) | |
| download | pkgsrc-8d019d64c92596d48308fde544e7ad3abdff4876.tar.gz | |
Use a temporary variable to hold ${_dep_:C/\:.*$//} since it is used
as a modifier on another variable and would need to be escaped
as :M${_dep_\:C/\\\:.*$//} in order for the modifiers to be parsed correctly.
This all used to work because _dep_ is a .for loop control variable which
was substituted in an earlier pass through the line.
Recent changes leave .for loop control variables to be substituted during
normal expansion.
Adding the \ would leave the script non-working with theolder make.
OK'd by Beefy
Diffstat (limited to 'mk')
| -rw-r--r-- | mk/tools/replace.mk | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/mk/tools/replace.mk b/mk/tools/replace.mk index 406a9d94486..87e1fcd3277 100644 --- a/mk/tools/replace.mk +++ b/mk/tools/replace.mk @@ -1,4 +1,4 @@ -# $NetBSD: replace.mk,v 1.216 2008/12/15 12:53:28 rillig Exp $ +# $NetBSD: replace.mk,v 1.217 2008/12/23 22:34:26 dsl Exp $ # # Copyright (c) 2005 The NetBSD Foundation, Inc. # All rights reserved. @@ -1249,7 +1249,8 @@ FIND_PREFIX:= ${TOOLS_FIND_PREFIX} ##### . if defined(_TOOLS_DEPMETHOD.${_t_}) && defined(TOOLS_DEPENDS.${_t_}) . for _dep_ in ${TOOLS_DEPENDS.${_t_}} -. if empty(${_TOOLS_DEPMETHOD.${_t_}}:C/\:.*$//:M${_dep_:C/\:.*$//}) +_dep_test := ${_dep_:C/\:.*$//} +. if empty(${_TOOLS_DEPMETHOD.${_t_}}:C/\:.*$//:M${_dep_test}) ${_TOOLS_DEPMETHOD.${_t_}}+= ${_dep_} . endif . endfor |