Re: Warning with loader Makefile?

On Fri, Aug 08, 2003 at 11:25:49AM +1200, Andrew Turner wrote:
[...]
> I found this patch worked by removing the secound ${PROG} target if 
> there was already one there.
> 
> --- /usr/src/share/mk/bsd.prog.mk	Mon Jun 30 06:16:26 2003
> +++ /usr/share/mk/bsd.prog.mk	Mon Aug  4 17:54:22 2003
> _at__at_ -31,11 +31,13 _at__at_
>  
>  OBJS+=  ${SRCS:N*.h:R:S/$/.o/g}
>  
> +.if !target(${PROG})
>  ${PROG}: ${OBJS}
>  .if defined(PROG_CXX)
>  	${CXX} ${CXXFLAGS} ${LDFLAGS} -o ${.TARGET} ${OBJS} ${LDADD}
>  .else
>  	${CC} ${CFLAGS} ${LDFLAGS} -o ${.TARGET} ${OBJS} ${LDADD}
> +.endif
>  .endif
>  
>  .else !defined(SRCS)

No, thanks.  If a makefile has a

${PROG}: foo

line, just to add an additional dependency, this patch will break
building the ${PROG}.


Cheers,
-- 
Ruslan Ermilov		Sysadmin and DBA,
ru_at_sunbay.com		Sunbay Software Ltd,
ru_at_FreeBSD.org		FreeBSD committer