-
Notifications
You must be signed in to change notification settings - Fork 3
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
stagex refactor #433
stagex refactor #433
Conversation
9e2978d
to
139ea33
Compare
725ae22
to
25a8193
Compare
Makefile
Outdated
cd out/build-base; \ | ||
tar -cf - . | docker load |
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.
Doesn't this change the directory for the following touch
command as well?
cd out/build-base; \ | |
tar -cf - . | docker load | |
(cd out/build-base; tar -cf - . | docker load) |
or can you use the the -C
argument to tar?
-C, --directory=DIR
Change to DIR before performing any operations. This option is order-sensitive, i.e. it affects all options that follow.
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.
Macs dont have -C sadly. that is why I remove -C
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 know mac env
doesn't have -C
but I thought their tar
would.
Highlights
make
: deterministically build all the things with stagex build containermake test
: run all tests in build containermake lint
: run automated linting from build containermake format
: run rustfmt from build containermake docs
: run rustdocs from build containermake gen
: run "make -C proto" under stagex with stagex-provided tools