Skip to content

Makefile reworking - #107

Merged
dstelzer merged 3 commits into
Dialog-IF:mainfrom
sehugg:makefile-update
Sep 2, 2026
Merged

Makefile reworking#107
dstelzer merged 3 commits into
Dialog-IF:mainfrom
sehugg:makefile-update

Conversation

@sehugg

@sehugg sehugg commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

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.

  • 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

sehugg added 2 commits August 31, 2026 00:10
- 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
@dstelzer

Copy link
Copy Markdown
Contributor

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?

@sehugg

sehugg commented Aug 31, 2026

Copy link
Copy Markdown
Contributor Author

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?

@dstelzer

Copy link
Copy Markdown
Contributor

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.

@dstelzer
dstelzer merged commit 9f6a64a into Dialog-IF:main Sep 2, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants