-
Notifications
You must be signed in to change notification settings - Fork 21
add build support for arm64-qnx8_0 #90
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?
add build support for arm64-qnx8_0 #90
Conversation
3fc7532 to
1c4d83a
Compare
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.
Pull request overview
Adds Bazel and CI configuration to build for the arm64-qnx8_0 target.
Changes:
- Enable
arm64-qnx8_0in the reusable smoke-test workflow matrix. - Add Bazel build configs for
arm64-linux,x86_64-qnx8_0, andarm64-qnx8_0. - Update toolchain repo usage to include QNX aarch64 and a base GCC toolchain repo.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
score_toolchains.MODULE.bazel |
Adjusts toolchain repositories used for GCC/QCC, including QNX aarch64. |
.github/workflows/reusable_smoke-test.yml |
Enables the arm64-qnx8_0 CI job. |
.bazelrc |
Introduces/extends Bazel configs for arm64 Linux and QNX 8.0 (x86_64 + arm64). |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| build:arm64-qnx8_0 --platforms=@score_bazel_platforms//:arm64-qnx8_0 | ||
| build:arm64-qnx8_0 --extra_toolchains=@score_qcc_aarch64_toolchain//:aarch64-qnx-sdp_8.0.0-posix |
Copilot
AI
Feb 11, 2026
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.
This config mixes "arm64" (platform name) with "aarch64" (toolchain name). If your repo treats these as synonyms that’s fine, but it would be clearer to align naming (e.g., use a consistent arm64/aarch64 convention for both platform and toolchain labels) to reduce confusion when adding more QNX variants.
|
The created documentation from the pull request is available at: docu-html |
afb2da5 to
605d7aa
Compare
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.
Pull request overview
Copilot reviewed 5 out of 5 changed files in this pull request and generated 5 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
No description provided.