-
Notifications
You must be signed in to change notification settings - Fork 32
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
test PR for build changes #815
Conversation
b8811da
to
d46bc63
Compare
.github/workflows/ci-project.yml
Outdated
@@ -13,7 +13,7 @@ on: | |||
build_params: | |||
type: string | |||
description: "parameters to build.sh script" | |||
default: 'core fs test project image' # by default don't build ports | |||
default: 'host core fs test project image' # by default don't build ports |
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.
WARN: rignt now host target was added for each of the CI build target.
Maybe it would be better to:
- create separate CI job for building host utils
- don't build host utils for every target
- (in testing we're using hostutils from docker image anyway)
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.
the idea (to be implemented after merging all respective changes is to:
- for whole project build host utils only for target
host-generic-pc
- introduce separate CI in
phoenix-rtos-hostutils
d46bc63
to
7463aba
Compare
7463aba
to
65562f0
Compare
65562f0
to
a27e14f
Compare
|
Test PR for CI to pass
the idea (to be implemented after merging all respective changes is to:
for whole project build host utils only for target host-generic-pc
introduce separate CI in phoenix-rtos-hostutils
Description
summary of changes:
Motivation and Context
Types of changes
How Has This Been Tested?
Checklist:
Special treatment