Source-Changes-D archive
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]
Re: CVS commit: src/usr.bin/ktruss
> > Modified Files:
> > src/usr.bin/ktruss: Makefile
> > - AWK=${TOOL_AWK:Q} CPP=${CPP:Q} CPPFLAGS=${CPPFLAGS:Q} \
> > + ${GENCMD} ${.TARGET} \
> > + AWK="${TOOL_AWK:Q}" CPP="${CPP:Q}" CPPFLAGS="${CPPFLAGS:Q}" \
>
> The extra quotes here look wrong. ${VAR:Q} should do the correct
> amount of quoting. If you need two levels of quoting for some
> reason that I don't see, then use ${VAR:Q:Q}, not "${VAR:Q}".
:Q:Q seems to work, thanks. The reason is ${GENCMD} uses eval. Unlike
${MAKEDIRTARGET} arguments can have spaces... This needs thought, yes.
Masao
--
Masao Uebayashi / Tombi Inc. / Tel: +81-90-9141-4635
Home |
Main Index |
Thread Index |
Old Index