Add steps.mk and simplify some implementation Makefiles#738
Open
asarhaddon wants to merge 1 commit into
Open
Conversation
In the process, recommend steps.mk for compiled language, and a minimal Makefile for interpreted ones. steps.mk avoids the duplication of the step names in each Makefile. It provides a consistent and explicit naming scheme. BINS -> step0A:=.exe OBJS -> step0A:%=obj/.o EARLY_STEPS_SRC -> step03:=.js LATE_STEPS -> step4A It allows some patterns to carry an explicit list of desired matches. step%.o: step%.c -> (step0A:=.o): %.o: %.c Unrelated changes. Replace several obsolete (lastword (steps)) constructs with (stepA). ada: replace -p mkdir option with a Make order prerequisite ada: replace force with .PHONY ada: use -o compile option instea of renaming the object erlang: replace a template macro with a loop on targets go: replace most of a template with a loop on targets hare: makefile -> Makefile haxe, scheme, sml: group the recipes by _MODE for readability
Contributor
Author
|
Rust complains because the Docker image is old. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Hello.
This merge request suggests a
steps.mkhelper for the implementation Makefiles. Some parts of #735 require a rebase when/if the automated tests pass.Here is the rest of the commit message:
In the process, recommend steps.mk for compiled language, and a minimal Makefile for interpreted ones.
steps.mk avoids the duplication of the step names in each Makefile. It provides a consistent and explicit naming scheme.
BINS -> step0A:=.exe
OBJS -> step0A:%=obj/.o
EARLY_STEPS_SRC -> step03:=.js
LATE_STEPS -> step4A
It allows some patterns to carry an explicit list of desired matches. step%.o: step%.c -> (step0A:=.o): %.o: %.c
Unrelated changes.
Replace several obsolete (lastword (steps)) constructs with (stepA).
ada: replace -p mkdir option with a Make order prerequisite
ada: replace force with .PHONY
ada: use -o compile option instea of renaming the object
erlang: replace a template macro with a loop on targets
go: replace most of a template with a loop on targets
hare: makefile -> Makefile
haxe, scheme, sml: group the recipes by _MODE for readability