-
Notifications
You must be signed in to change notification settings - Fork 40
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add make release rule #49
Conversation
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.
Worked for me, cool.
Makefile
Outdated
@make rebar | ||
@make ui | ||
@make | ||
@./bootstrap |
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.
I haven't tried this but you should be able to express dependencies better with
all: rebar
@$(REBAR) get-deps compile
...
release: all ui
@./bootstrap
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.
You're right. I did this way to allow echo before everything. Is there any way to write it your way and preserve echo?
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.
I don't think so and am not sure how useful that echo
is, especially when you can see make release
right above
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.
Probably you're right.
@baransu is it finished now? :) |
Yeah, I thinks so. Let's merge it! |
Building Erlang Perf Lab manually requires few commands to run. I propose new rule which contains all commands which are necessary to run.
I'm not an Make expert so I'm counting on your review 馃檪