svn commit: r296637 - in head: contrib/bmake contrib/bmake/mk contrib/bmake/unit-tests share/mk usr.bin/bmake

[Simon J. Gerraty]

Bryan Drewery <bdrewery at FreeBSD.org> wrote:
> >  # MACHINE specific depend files are supported, but *not* default.
> >  # If any already exist, we should follow suit.
> >  _aml = ${ALL_MACHINE_LIST:Uarm amd64 i386 powerpc:N${MACHINE}} ${MACHINE}
> > -# MACHINE must be the last entry in _aml ;-)
> > +# make sure we restore MACHINE
> > +_m := ${MACHINE}
> >  _e := ${_aml:@MACHINE@${.MAKE.DEPENDFILE_PREFERENCE:@m@${exists($m):?$m:}@}@}
> > +MACHINE := ${_m}
> >  .if !empty(_e)
> >  .MAKE.DEPENDFILE ?= ${.MAKE.DEPENDFILE_PREFERENCE:M*${MACHINE}:[1]}
> >  .endif
> 
> 
> What changed in bmake to require this? Now in older checkouts I don't

deleting the :@ loop iterator variable.

> get MACHINE defined with DIRDEPS_BUILD without this change.  That's not
> ideal since it breaks older builds.

Yes, I should mfc the above change? it is harmess with older bmake's.