Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Makefile Modules: Change nothing except for Makefile moved to make/02_mod.mk #555

Closed
wants to merge 1 commit into from

Conversation

maelvls
Copy link
Member

@maelvls maelvls commented Aug 12, 2024

Makefile Modules are too tempting, I want to tap into them, especially the tools module.

As a first "baby" step, I propose to move Makefile to make/02_mod.mk: nothing else changes. The GitHub Actions are the same, the old targets in Makefile work the same.

My idea is to migrate more and more to native makefile-modules targets without having to migrate the whole thing at once.

@maelvls maelvls force-pushed the step1-makefile-modules branch 3 times, most recently from 937f974 to dcf90f4 Compare August 12, 2024 16:56
The Makefile was just renamed to make/02_mod.mk, nothing else has
changed (except for the relative paths that were changed to
$(ROOT_DIR)). The goal is to migrate more and more to native
makefile-modules targets without having to migrate the whole thing at
once.
@maelvls maelvls force-pushed the step1-makefile-modules branch from dcf90f4 to 8694365 Compare August 12, 2024 17:18
@maelvls
Copy link
Member Author

maelvls commented Aug 12, 2024

Ooof that's a lot of work to fix make build-docker-image. The makefile itself runs within the container, it's just super complex for not many reasons (buildx, qemu, etc).

Closing for now.

@maelvls maelvls closed this Aug 12, 2024
@inteon
Copy link
Contributor

inteon commented Aug 13, 2024

@maelvls the PR looks good from a Makefile modules perspective. Let's keep it as reference for when we have a bit more time to figure out these buildx, qemu etc. tools.

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