search

NLnetLabs / simdzone

Fast and standards compliant DNS zone parser
BSD 3-Clause "New" or "Revised" License
63 stars 11 forks source link

Fix build dependencies #193

Closed wcawijngaards closed 4 months ago

wcawijngaards commented 4 months ago

This fixes build dependencies. The objects depended on the source file in the build directory and not in the srcdir. Also the combined list of objects depended on the combined list of sources, causing objects to be rebuilt as the dependency tracking file for another source file was updated. The change fixes it.

k0ekk0ek commented 4 months ago

@wcawijngaards, I made an attempt to not have to "manually" specify that which we generate. This really is inception level stuff, but I managed to get something working. Let's just say I learned a lot about GNU make and BSD make. I replaced the sources + depends by a .depend file that gets generated and basically contains Makefile rules. The same change you made really, but automated. For simdzone, it really wasn't worth the effort, but the same mechanism could make the NSD Makefile a lot cleaner. I chose .depend because BSD mentions it in their man page, don't know if I should've chosen something else. Anyway, please take a look.