Skip to content

crypto: Add qnx test execution for test binaries (cc_test)#70

Open
OliverHeilwagen wants to merge 1 commit into
eclipse-score:mainfrom
etas-contrib:add_qnx_test_support
Open

crypto: Add qnx test execution for test binaries (cc_test)#70
OliverHeilwagen wants to merge 1 commit into
eclipse-score:mainfrom
etas-contrib:add_qnx_test_support

Conversation

@OliverHeilwagen

@OliverHeilwagen OliverHeilwagen commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

resolves #13

  • Update used toolchain version and add rules_imagefs for qnx images.
  • Update bazel dependencies to remove warnings.
  • Update devcontainer to support local execution.
  • Rename existing test workflow to test_linux.
  • Add test workflow for qnx.
  • Fix incorrect destruction orders identified by testing on qnx.

@github-actions

github-actions Bot commented Jul 3, 2026 •
edited
Loading

Copy link
Copy Markdown

License Check Results

🚀 The license check job ran with the Bazel command:

bazel run --lockfile_mode=error //:license-check

Status: ⚠️ Needs Review

Click to expand output 
[License Check Output]
Extracting Bazel installation...
Starting local Bazel server (8.3.0) and connecting to it...
INFO: Invocation ID: 372c4cf3-5eed-45a0-a003-b903e84aaa20
Computing main repo mapping: 
Computing main repo mapping: 
DEBUG: Rule 'abseil-cpp+' indicated that a canonical reproducible form can be obtained by modifying arguments integrity = "sha256-m3oGQwXp/ZTRJP+mzDWFkutCtdpYj7TgfQklSqQAhts="
DEBUG: Repository abseil-cpp+ instantiated at:
  <builtin>: in <toplevel>
Repository rule http_archive defined at:
  /home/runner/.bazel/external/bazel_tools/tools/build_defs/repo/http.bzl:394:31: in <toplevel>
Computing main repo mapping: 
DEBUG: Rule 'protobuf+' indicated that a canonical reproducible form can be obtained by modifying arguments integrity = "sha256-w6Cp7OiTLjHDtzbi2xixxC5wcM2biBOIsm0BqnHiTKI="
DEBUG: Repository protobuf+ instantiated at:
  <builtin>: in <toplevel>
Repository rule http_archive defined at:
  /home/runner/.bazel/external/bazel_tools/tools/build_defs/repo/http.bzl:394:31: in <toplevel>
Computing main repo mapping: 
DEBUG: Rule 'grpc+' indicated that a canonical reproducible form can be obtained by modifying arguments integrity = "sha256-CvN7gAlTEwtHwHW1ZoPuYL3D7aPDf8YAQZP1tWl1ggQ="
DEBUG: Repository grpc+ instantiated at:
  <builtin>: in <toplevel>
Repository rule http_archive defined at:
  /home/runner/.bazel/external/bazel_tools/tools/build_defs/repo/http.bzl:394:31: in <toplevel>
Computing main repo mapping: 
Loading: 
Loading: 1 packages loaded
Loading: 1 packages loaded
    currently loading: 
WARNING: Target pattern parsing failed.
ERROR: Skipping '//:license-check': no such target '//:license-check': target 'license-check' not declared in package '' defined by /home/runner/work/inc_security_crypto/inc_security_crypto/BUILD
ERROR: no such target '//:license-check': target 'license-check' not declared in package '' defined by /home/runner/work/inc_security_crypto/inc_security_crypto/BUILD
INFO: Elapsed time: 9.387s
INFO: 0 processes.
ERROR: Build did NOT complete successfully
ERROR: Build failed. Not running target

@OliverHeilwagen OliverHeilwagen self-assigned this Jul 3, 2026
@github-actions

github-actions Bot commented Jul 3, 2026

Copy link
Copy Markdown

The created documentation from the pull request is available at: docu-html

OliverHeilwagen
OliverHeilwagen commented Jul 3, 2026
View reviewed changes
Comment thread qnx_unit_tests.patch Outdated
@@ -0,0 +1,48 @@
--- a/src/arm64_qnx8/run_under_qnx.sh

@OliverHeilwagen OliverHeilwagen Jul 3, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Bring changes to qnx_unit_tests repository and remove patch.

@OliverHeilwagen OliverHeilwagen Jul 7, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Remove patch if this gets merged:
eclipse-score/qnx_unit_tests#33

@sunildevda sunildevda requested review from PandaeDo and se-lo July 3, 2026 11:29
@OliverHeilwagen OliverHeilwagen force-pushed the add_qnx_test_support branch 3 times, most recently from 6f516fa to 54803a0 Compare July 7, 2026 12:23
@OliverHeilwagen
crypto: Add qnx test execution for test binaries (cc_test)
916ddc8 
- Update used toolchain version and add rules_imagefs for
  qnx images.
- Update bazel dependencies to remove warnings.
- Update devcontainer to support local execution.
- Rename existing test workflow to test_linux.
- Add test workflow for qnx.
- Fix incorrect destruction orders identified by testing
  on qnx.
OliverHeilwagen
OliverHeilwagen commented Jul 7, 2026
View reviewed changes
Comment thread .devcontainer/devcontainer-lock.json
@@ -0,0 +1,9 @@
{

@OliverHeilwagen OliverHeilwagen Jul 7, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

remove and add to ignore.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@PandaeDo PandaeDo Awaiting requested review from PandaeDo
@se-lo se-lo Awaiting requested review from se-lo

At least 1 approving review is required to merge this pull request.

Assignees

@OliverHeilwagen OliverHeilwagen

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Test on QNX8 (local tests)

1 participant

@OliverHeilwagen