-
Notifications
You must be signed in to change notification settings - Fork 34
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
Enhance Makefile
#445
Enhance Makefile
#445
Conversation
* `install` recipe does not rebuild binary * Add `help` target and documentation * Add several config variables * Add sources as dependencies for (in-)validation Based on template https://gist.github.com/straight-shoota/275685fcb8187062208c0871318c4a23
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👏
@straight-shoota This PR broke my local setup (macOS 12.7) :(
|
Are you sure any of the source files changed since you last ran You could also try |
Yeah, I'm sure. Also,
|
Hm, looks like make doesn't recognize the prerequisites ( Maybe try this diff to see if there are any errors with the find command? Would be very strange though... @@ -28,7 +28,7 @@ INSTALL_BIN ?= /usr/bin/install
SHARD_BIN ?= ../../bin
CRFLAGS ?= -Dpreview_mt
-SRC_SOURCES ::= $(shell find src -name '*.cr' 2>/dev/null)
+SRC_SOURCES ::= $(shell find src -name '*.cr')
DOC_SOURCE ::= src/** Looks like your make version is really ancient. You might want to update to a more modern version. (Dunno if that could be of any relevance here) |
Nope, doesn't work. Also what's interesting when I run this command from my terminal I get: λ Shabdiz → shell find src -name '*.cr'
zsh: command not found: shell |
It means run So could you try if |
Yeah, that works just fine. |
Solution was to install |
I figured the
Makefile
could use some love after it failed me in some capacities.install
recipe does not rebuild binaryhelp
target and documentationBased on template https://gist.github.com/straight-shoota/275685fcb8187062208c0871318c4a23