Makefile refactor to use the Makes makefile system#75
Closed
ingydotnet wants to merge 1 commit intoglojurelang:mainfrom
Closed
Makefile refactor to use the Makes makefile system#75ingydotnet wants to merge 1 commit intoglojurelang:mainfrom
ingydotnet wants to merge 1 commit intoglojurelang:mainfrom
Conversation
ingydotnet
force-pushed
the
makefile
branch
2 times, most recently
from
9273c7c to
0234615
Compare
ingydotnet
changed the title
ingydotnet
force-pushed
the
makefile
branch
2 times, most recently
from
1aa31ac to
b1c1c81
Compare
ingydotnet
added a commit
to ingydotnet/glojure
that referenced
this pull request
Aug 15, 2025
To avoid adding things that are often lying around... The .clj and .glj are to ignore top level scripts for testing stuff. The GNUmakefile and .cache are for alternate Makefile stuff like I do in glojurelang#75 If that doesn't get merged I can just symlink to it since GNUmakefile has precedence over Makefile (for GNU make).
ingydotnet
force-pushed
the
makefile
branch
from
78eac50 to
311ce15
Compare
See: https://github.com/makeplus/makes With this setup, one not need to have Go or Clojure installed. The Makefile takes care of everything. You can easily override Go and Clojure versions with `GO-VERSION` and `CLOJURE-VERSION` make variables: make test GO-VERSION=1.23.4 Makefile autoinstalls the `makes` repo, Go and Clojure deps under `.cache/`. Adds support for: * make shell - Start subshell with local go in PATH * make build - Build glj * make clean - Remove build artifacts * make distclean - Clean and remove .cache dir Inside shell after `make build`, `glj` will be in PATH. We lock the makeplus/makes repo to commit ca8c2c25e66cf6bfcf8c993502de5b98da5beaf5 to be make sure everyone gets the same results forever. Updates Makefile variables to use `-` instead of `_`. Variables with `_` can come from the environment. With `-` they are only recognized by make. You can override in the shell with `make something FOO-BAR=123`. `.PHONY` is only needed when phony targets have matching file system names. `test` is the only one here.
ingydotnet
force-pushed
the
makefile
branch
from
311ce15 to
0a34e9b
Compare
Contributor
Author
|
Replaced by #119 |
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
1 participant
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.
See: https://github.com/makeplus/makes
With this setup, one not need to have Go or Clojure installed.
The Makefile takes care of everything.
You can easily override Go and Clojure versions with
GO-VERSIONandCLOJURE-VERSIONmake variables:Makefile autoinstalls the
makesrepo, Go and Clojure deps under.cache/.Adds support for:
Inside shell after
make build,gljwill be in PATH.We lock the makeplus/makes repo to commit
ca8c2c25e66cf6bfcf8c993502de5b98da5beaf5 to be make sure everyone gets
the same results forever.
Updates Makefile variables to use
-instead of_.Variables with
_can come from the environment.With
-they are only recognized by make.You can override in the shell with
make something FOO-BAR=123..PHONYis only needed when phony targets have matching file systemnames.
testis the only one here.