Issue #1067, Fixed Add QNX to cFS multi-target build - #1114
dmolock-NASA wants to merge 1 commit into
Conversation
| ############## | ||
| # QNX builds | ||
| ############## | ||
| PREP_OPTS_aarch64le_qnx_gnu += -DENABLE_UNIT_TESTS=FALSE |
There was a problem hiding this comment.
whats the reason unit test is set to false?
| CONFIG_NAMES += qemu_yocto_linux | ||
| CONFIG_NAMES += osal edslib | ||
| CONFIG_NAMES += aarch64le_qnx_gnu aarch64le_qnx_gnu_eds | ||
| CONFIG_NAMES += x86_64_qnx_gnu x86_64_qnx_gnu_eds |
There was a problem hiding this comment.
The idea with the multitarget build is to craft a small number of configs that (as a whole) try to span a good sampling of the build variations/options we offer. It is not (and will never be) a comprehensive set covering every option.
In that sense, I don't think we need 4 different QNX configs. Just 1 would suffice.
Also -- don't name it "x86_64" or "aarch64le" ... instead make it a specific board/target name. Does the development board being used for QNX testing have a name (like Raspberry Pi/rpi?). Preference is to use that board name because this config encompasses everything about running on that board, not just the processor type (this is just one detail out of many)
name: FSW Code Change
about: Flight Software code changes
labels: fsw
Description of Change
Linked Issue
Closes #
Requirements Impact
Testing Evidence
Unit Tests (UT Assert)
COSMOS Test Suite
Areas of Expertise Touched
Author Checklist
.clang-formatReviewer Checklist
.clang-formathas been appliedReviewer Testing Notes