-
Notifications
You must be signed in to change notification settings - Fork 515
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
build: use twoliter make #3342
build: use twoliter make #3342
Conversation
The command has changed from |
Simplify |
Cleanup and twoliter now installs to a tempdir so we don't need an ignored directory. |
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
fe96dfc
to
680533e
Compare
Uses the published version of The Twoliter.toml file isn't doing much right now, I'm thinking about what to do here. This push isn't super helpful. |
|
In order support out-of-tree builds (see bottlerocket-os#2669 and https://github.com/bottlerocket-os/twoliter), we need to move the build system to the Twoliter repository. This commit starts this process by moving the Makefile.toml logic to Twoliter and using Twoliter as a facade over the build system.
Rebase over develop. This change now needs to be applied in the Twoliter git repo https://github.com/bottlerocket-os/bottlerocket/pull/2757/files#diff-0f47e53927f2a76fca4bf29143c158748d4969cceac86d63c48c9f360a5a8656 I created an issue for this. |
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.
Changes look good to me. Checked out locally and verified the ability to build. A little extra delay getting twoliter set up initially, but no issues getting a good build.
TODO
twoliter make
command in the Twoliter repository and add a tagIssue number:
Related to #2669
Description of changes:
In order support out-of-tree builds (see #2669 and https://github.com/bottlerocket-os/twoliter), we need to move the build system to the Twoliter repository. This commit starts this process by moving the Makefile.toml logic to Twoliter and using Twoliter as a facade over the build system.
Testing done:
cargo make build
cargo make ami
Terms of contribution:
By submitting this pull request, I agree that this contribution is dual-licensed under the terms of both the Apache License, version 2.0, and the MIT license.