On Mon, 10 May 2004 17:38:56 -0700 (PDT), "Jeremy C. Reed" writes:
>On Mon, 10 May 2004, Simon J. Gerraty wrote:
>
>> Anyone see anything wrong with the patch below?
>
>I didn't try yet, but I have comments.
>
>> @@ -14,16 +14,19 @@
>>  ONLY_FOR_PLATFORM=	NetBSD-*-*
>
>You mention OPSYS below. It seems this implies we can get rid of this
>ONLY_FOR_PLATFORM definition. (I package for Linux, for example.)

I have no idea why the ONLY_FOR_PLATFORM is there - the whole point of
bmake is building on multiple platforms ;-)

>> +	${INSTALL_MAN} ${WRKDIR}/${OPSYS}/bmake.cat1 ${PREFIX}/man/cat1/bmake.0
>> +	${INSTALL_MAN} ${WRKDIR}/${OPSYS}/bmake.1 ${PREFIX}/man/man1
>
>What if bmake.1 is not rendered? (I should look before asking. but this
>hit me several times before.)

bmake.1 is generated from make.1 so should always be there.