Subject: Re: CVS commit: [gdamore-uart] src/sys/arch/m68k/fpsp
To: None <source-changes@netbsd.org>
From: Christos Zoulas <christos@astron.com>
List: source-changes
Date: 06/20/2006 16:00:14
In article <4497F783.6000302@tadpole.com>,
Garrett D'Amore  <garrett_damore@tadpole.com> wrote:
>Izumi Tsutsui wrote:
>> garrett_damore@tadpole.com wrote:
>>
>>   
>>> Looks fine to me, though I'd have used
>>>
>>> ${SED:=sed}
>>>
>>> instead.
>>>     
>>
>> Is it okay to commit this?
>>   
>
>Why ${TOOL_SED:Q}  instead of ${TOOL_SED}?
>
>On Solaris at least, the construct ${TOOL_SED:Q} generates a "bad
>substitution" error.

But that is a bmake expansion. Are you sure your bmake is up-to-spec?

christos