kmod.mk - modules build framework [was: svn commit: r193818 - head/sys/modules/sound/sound]

[Bruce Evans]

On Wed, 10 Jun 2009, Bjoern A. Zeeb wrote:

> On Wed, 10 Jun 2009, Bruce Evans wrote:
>
>> On Tue, 9 Jun 2009, Bjoern A. Zeeb wrote:
>> 
>>> On Tue, 9 Jun 2009, Bjoern A. Zeeb wrote:
>>>> Log:
>>>>  Depend on @ machine (_ILINKS) as we do with other modules so that @
>>>>  is there for parallel (-jN) builds.  Ideally beforedepends in kmod.mk
>>>>  should do the right thing but it seems it does not.
>>> 
>>> Anyone with lots of build framework know how may want to look at this.
>> 
>> Failures only for parallel builds normally mean missing dependencies.
>> 
>>>> -feeder_eq_gen.h:
>>>> +feeder_eq_gen.h:	@ machine
>>>> 	${AWK} -f @/tools/feeder_eq_mkfilter.awk -- ${FEEDER_EQ_PRESETS} > 
>>>> ${.TARGET}
>> 
>> Here there is still a missing dependency on the
>> @/tools/feeder_eq_mkfilter.awk (fixed in the next commit).  This dependency
>> is not very important, but since the utility has "@" in its pathname,
>> running it certainly depends on "@".
>
> And adding it again messes with @ not being there for parallel
> builds:(
>
>> This missing dependency seems to be a general bug.  There is an ordering
>> requirement that beforedepend is before built before ${DEPENDFILE},
>> but this apparently doesn't extend to ${DEPENDFILE}'s prerequisites,
>> and I don't know of any general way to make it do so and it probably
>> shouldn't do so in general (some of the prerequisites might be needed
>> before beforedepend).
>
> Well no; that should not be the case. I would say that would be a bug
> as beforedepnd only adds the @ and machine symlinks to my memory from
> reading the other day.
>
> All after all that's why I said this needs someone with the devotion
> to sort all this out, know what the right way to do things is, have
> the time, ..

See my private reply to the next commit message.  It says to copy the
handling of this problem for vnode_if.awk in kmod.mk, where it took 3
commits between 1999 and 2001 to get the subtleties right!

Bruce