Makefile reworking - #107
Conversation
- fix staleness/dependency issues - refactored common test rules into common.mk - removed no6502 target - should build and test in parallel with -j8 - assume GNU make 3.81 (macOS) or higher
11eacec to
02a173f
Compare
|
Looks good overall! I'm glad to finally have a solution for the blobs not updating properly. But why get rid of the no6502 target? |
|
I don't think no6502 worked anyway, because you have to build table*.h to build aambundle, which means you need to assemble the .bin files. A couple blobs are checked in, but for example c64_frontend.bin is not. If we really need no6502, maybe it could make zero-length blobs or something, with a big warning in aambundle that this is a cursed build? |
|
That makes sense. The intent of no6502, if I remember right, was to allow building aamrun without needing xa; but aamrun is a very cursed executable in the first place, and now that it's part of the binary distribution I don't think people will need that much any more. So yeah, I think we can just make xa be a prerequisite. But I'll tag in @susan-davis who I believe originally requested it. |
I am no makefile guru, but seems like an improvement. Now if you "make all" a second time it will do nothing.
I have not tested the mingw32 cross-compile but I think you need mingw64 these days anyway.