Skip to content

Add QNX 7.1 remote toolchain support and address PR #12 review comments#14

Closed
Subramanian-K812 wants to merge 11 commits into
proAI_test_qnx71from
Subramanian-K812_fix_qnx71_local_path_overrides
Closed

Add QNX 7.1 remote toolchain support and address PR #12 review comments#14
Subramanian-K812 wants to merge 11 commits into
proAI_test_qnx71from
Subramanian-K812_fix_qnx71_local_path_overrides

Conversation

@Subramanian-K812

Copy link
Copy Markdown

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.bazel

  • Replaced local_path_overridegit_override for score_bazel_cpp_toolchains
    at commit 9a9f063 (Subramanian-K812_add_qnx71_aarch64_sdp_support)

bazel_common/score_qnx_toolchains.MODULE.bazel

  • Removed invalid qcc.sdp() block
  • Restored score_qcc_aarch64_toolchain → QNX 8.0 (sdp_version = "8.0.0", GCC 12.2.0)
  • Added new score_qcc_aarch64_qnx71_toolchain → QNX 7.1 (sdp_version = "7.1.0",
    GCC 8.3.0) with use_default_package = True and license_info_variable

.bazelrc

  • Restored qnx-aarch64 config to QNX 8.0 platforms and toolchains
  • Added qnx71-aarch64 config 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)
    • Correct aarch64-qnx-sdp_7.1.0 platform and toolchain labels

third_party/qnx710/

  • Removed BUILD.bazel and qnx710_sdp.BUILD (dead code — no longer referenced
    since use_default_package=True handles the SDP via score_bazel_cpp_toolchains)

External Dependencies (new branches)

Repo Branch Commit Purpose
qorix-group/bazel_cpp_toolchains Subramanian-K812_add_qnx71_aarch64_sdp_support 9a9f063 VERSION_MATRIX entry for QNX 7.1 SDP, fixed sdp.BUILD target names (:qcc:cc, :qpp:cxx), removed GCC 8.3 incompatible flag -Wno-error=mismatched-new-delete
qorix-group/score-crates Subramanian-K812_add_qnx71_iceoryx2_patches 8f29fa5 iceoryx2 pipe fallback patches for QNX 7.1 (no AF_UNIX socket 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

# QNX 7.1 — requires QNXLM_LICENSE_FILE set
export QNXLM_LICENSE_FILE=/path/to/license.dat  # or 27000@server
bazel --output_base=build/qnx71-aarch64 build --config=qnx71-aarch64 //showcases/standalone:comm
# Result: Build completed successfully, 956 total actions

name = "score_qnx_aarch64_ifs_toolchain",
sdp_to_import = "@score_qcc_aarch64_toolchain_pkg",
sdp_version = "7.1.0",
sdp_version = "8.0.0",

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we need another one for 7.1 as well

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

added

Comment thread .bazelrc
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

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

shouldnt be 7.1 toolchain as well?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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?

Comment thread .bazelrc
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

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ifs for 7.1 from score_qnx_aarch64_ifs_toolchain shall be included as well, or?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this file should be not needed, changes should be applied directly to repo https://github.com/eclipse-score/bazel_cpp_toolchains + git_override

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removed.

@PiotrKorkus PiotrKorkus closed this Jun 1, 2026
@PiotrKorkus
PiotrKorkus deleted the Subramanian-K812_fix_qnx71_local_path_overrides branch June 1, 2026 09:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants