Skip to content

add support for QNX 7.1/8.0 #318

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

Merged
merged 15 commits into from
Jun 11, 2025
Merged

Conversation

pkleymonov-qnx
Copy link
Contributor

@pkleymonov-qnx pkleymonov-qnx commented May 8, 2025

This PR provides changes for QNX 7.1/8.0 running on aarch64le/x86_64
issue: #314

NOTE: QNX cross-compilation are only supported from a Linux(x86_64) build machine.

Before building up-cpp we need to build all dependencies from up-conan-recipes:

Please follow instruction for QNX build in file up-conan-recipes/README.md

Pre-requisite:

Build up-cpp for QNX

# setup path to up-conan-recipes
export QNX_CONAN_ROOT=<path_to_up-conan-recipes>

# Install conan toolchain for QNX target
#
# <profile-name>: nto-7.1-aarch64-le, nto-7.1-x86_64, nto-8.0-aarch64-le, nto-8.0-x86_64
# <version-number>: 1.0.0-rc0, 1.0.0, 1.0.1-rc1, 1.0.1
#
conan install -pr:h=$QNX_CONAN_ROOT/tools/profiles/<profile-name> --version=<version-number> --build=missing .

cmake --preset conan-release

cmake --build build/Release -- -j

# all tests you can find under build/Release/bin/
# copy test binaries to your QNX target

closes #314

Copy link

github-actions bot commented May 8, 2025

Code coverage report is ready! 📈

Copy link
Contributor

@PLeVasseur PLeVasseur left a comment

Choose a reason for hiding this comment

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

Thanks @pkleymonov-qnx for the contribution!

Could you open an issue for adding tests for building for QNX in CI as well?

fix whitespace

Co-authored-by: Pete LeVasseur <[email protected]>
Copy link

Code coverage report is ready! 📈

pkleymonov-qnx and others added 6 commits May 13, 2025 14:49
@pkleymonov-qnx pkleymonov-qnx requested a review from PLeVasseur June 5, 2025 13:25
Copy link

github-actions bot commented Jun 7, 2025

Code coverage report is ready! 📈

Copy link
Contributor

@PLeVasseur PLeVasseur left a comment

Choose a reason for hiding this comment

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

Couple of suggestions, otherwise looks good. Thanks @pkleymonov-qnx!

Let's wait on eclipse-uprotocol/up-conan-recipes#36 merging and then merge this one 👍

Fix some links to QNX build instructions.
Copy link

Code coverage report is ready! 📈

@PLeVasseur PLeVasseur merged commit 94ec934 into eclipse-uprotocol:main Jun 11, 2025
12 checks passed
@PLeVasseur
Copy link
Contributor

Thank you @pkleymonov-qnx!

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.

Adding QNX build support
2 participants