Add QNX 7.1 remote toolchain support and address PR #12 review comments#14
Closed
Subramanian-K812 wants to merge 11 commits into
Closed
Add QNX 7.1 remote toolchain support and address PR #12 review comments#14Subramanian-K812 wants to merge 11 commits into
Subramanian-K812 wants to merge 11 commits into
Conversation
…x71_toolchain for QNX 7.1
PiotrKorkus
reviewed
May 19, 2026
| name = "score_qnx_aarch64_ifs_toolchain", | ||
| sdp_to_import = "@score_qcc_aarch64_toolchain_pkg", | ||
| sdp_version = "7.1.0", | ||
| sdp_version = "8.0.0", |
Collaborator
There was a problem hiding this comment.
we need another one for 7.1 as well
| build:qnx71-aarch64 --sandbox_writable_path=/var/tmp | ||
| build:qnx71-aarch64 --platforms=@score_bazel_platforms//:aarch64-qnx-sdp_7.1.0-posix | ||
| build:qnx71-aarch64 --extra_toolchains=@score_qcc_aarch64_qnx71_toolchain//:aarch64-qnx-sdp_7.1.0 | ||
| build:qnx71-aarch64 --extra_toolchains=@score_toolchains_rust//toolchains/ferrocene:ferrocene_aarch64_unknown_nto_qnx800 |
Collaborator
There was a problem hiding this comment.
shouldnt be 7.1 toolchain as well?
Author
There was a problem hiding this comment.
I could see only toolchain for aarch64-unknown-nto-qnx800 (QNX 8.0) present actually and no aarch64-unknown-nto-qnx710 binary for QNX 7.1 in eclipse-score/ferrocene_toolchain_builder releases. Is there any other way to address this?
| build:qnx71-aarch64 --sandbox_writable_path=/var/tmp | ||
| build:qnx71-aarch64 --platforms=@score_bazel_platforms//:aarch64-qnx-sdp_7.1.0-posix | ||
| build:qnx71-aarch64 --extra_toolchains=@score_qcc_aarch64_qnx71_toolchain//:aarch64-qnx-sdp_7.1.0 | ||
| build:qnx71-aarch64 --extra_toolchains=@score_toolchains_rust//toolchains/ferrocene:ferrocene_aarch64_unknown_nto_qnx800 |
Collaborator
There was a problem hiding this comment.
ifs for 7.1 from score_qnx_aarch64_ifs_toolchain shall be included as well, or?
Collaborator
There was a problem hiding this comment.
this file should be not needed, changes should be applied directly to repo https://github.com/eclipse-score/bazel_cpp_toolchains + git_override
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Addresses all review comments from PR #12
Adds proper QNX 7.1 (ProAI 8650) build support using only remote dependencies
and
git_override— no local path overrides.Changes
MODULE.bazellocal_path_override→git_overrideforscore_bazel_cpp_toolchainsat commit
9a9f063(Subramanian-K812_add_qnx71_aarch64_sdp_support)bazel_common/score_qnx_toolchains.MODULE.bazelqcc.sdp()blockscore_qcc_aarch64_toolchain→ QNX 8.0 (sdp_version = "8.0.0", GCC 12.2.0)score_qcc_aarch64_qnx71_toolchain→ QNX 7.1 (sdp_version = "7.1.0",GCC 8.3.0) with
use_default_package = Trueandlicense_info_variable.bazelrcqnx-aarch64config to QNX 8.0 platforms and toolchainsqnx71-aarch64config for ProAI 8650 QNX 7.1 with:--action_env=QNXLM_LICENSE_FILE(forwards FLEXlm license into Bazel sandbox)--sandbox_writable_path=/var/tmp(required by QNX license daemon)aarch64-qnx-sdp_7.1.0platform and toolchain labelsthird_party/qnx710/BUILD.bazelandqnx710_sdp.BUILD(dead code — no longer referencedsince
use_default_package=Truehandles the SDP viascore_bazel_cpp_toolchains)External Dependencies (new branches)
qorix-group/bazel_cpp_toolchainsSubramanian-K812_add_qnx71_aarch64_sdp_support9a9f063sdp.BUILDtarget names (:qcc→:cc,:qpp→:cxx), removed GCC 8.3 incompatible flag-Wno-error=mismatched-new-deleteqorix-group/score-cratesSubramanian-K812_add_qnx71_iceoryx2_patches8f29fa5AF_UNIXsocket pair support on SA8650)SDP Path:
https://github.com/qorix-group/bazel_cpp_toolchains/releases/download/v0.1.0/qnx710_sdp.tar.gz
Build Verification