-
Notifications
You must be signed in to change notification settings - Fork 14
Support correct targets & toolchains #73
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
base: main
Are you sure you want to change the base?
Support correct targets & toolchains #73
Conversation
License Check Results🚀 The license check job ran with the Bazel command: bazel run //:license-checkStatus: Click to expand output |
|
The created documentation from the pull request is available at: docu-html |
3f4c927 to
9ac7bfe
Compare
|
For the reviewers the failed check QNX8 Build / qnx-build / Build QNX target (pull_request_target) is because of a sporadic failure. One commit before this check passed no relevant files changed. |
|
@paulquiring comments may be helpfull to fix qnx. Keep in mind that your chnages will only affect QNX CI only after merge so you shall test them locally . This means we need a collector in examples and mention it in CI/CD ? |
@pawelrutkaq I tested successfully locally and ensured that the examples are also build in the qnx workflow. |
|
@paulquiring please fix CI |
MODULE.bazel
Outdated
| remote = "https://github.com/eclipse-score/toolchains_rust.git", | ||
| module_name = "rules_rust", | ||
| remote = "https://github.com/pawelrutkaq/rules_rust.git", # To be fixed once rule_rust is in score bazel registry | ||
| commit = "d655071dfc8783bbdad4e27db349abd221ba170c", |
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.
Please use 293337fd6402ec3dfbeb8f2d589f38ad9124dadd
pawelrutkaq
left a comment
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.
Fix red CI
33f4b69 to
fa071ea
Compare
This pull request updates the Bazel build and module configuration to improve multi-platform support, streamline toolchain management, and update dependencies. The main changes include significant refactoring of
.bazelrcandMODULE.bazelto support explicit platform and toolchain configurations for both Linux and QNX targets on x86_64 and arm64 architectures, as well as updates to workflow and dependency versions.