Skip to content

Commit

Permalink
Coin: Set COIN_CROSS_PLATFORM_VERSION_TEST environment variable
Browse files Browse the repository at this point in the history
We have certain platforms where we build on one OS version (the
latest) and then test on an older version.
macOS is an example of such a platform.

Set the COIN_CROSS_PLATFORM_VERSION_TEST environment variable to true
for such platforms. They usually have a dependency on the main OS
version and are marked with the TestOnly feature.

This is useful information that can be used by CMake build tests in
order to skip those that do not account for the difference in OS
versions.

Change-Id: I6eeeac98abe6beb00ce431e17589f8e093ed23e4
Reviewed-by: Toni Saario <[email protected]>
Reviewed-by: Qt CI Bot <[email protected]>
  • Loading branch information
alcroito committed Mar 4, 2024
1 parent 3f26fde commit 7684814
Showing 1 changed file with 16 additions and 0 deletions.
16 changes: 16 additions & 0 deletions coin/instructions/cmake_run_ctest_enforce_exit_code.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -88,6 +88,22 @@ instructions:

- !include "{{qt/qtbase}}/coin_module_test_android_start_emulator.yaml"

- type: EnvironmentVariable
variableName: COIN_CROSS_PLATFORM_VERSION_TEST
variableValue: "TRUE"
enable_if:
condition: and
conditions:
- condition: property
property: platformDependency
not_equals_value: null
- condition: property
property: platformDependency.target.osVersion
not_equals_property: target.osVersion
- condition: property
property: features
contains_value: TestOnly

- type: ExecuteCommand
command: "{{.Env.TESTS_ENV_PREFIX}} ctest {{.Env.CTEST_ARGS}}"
executeCommandArgumentSplitingBehavior: SplitAfterVariableSubstitution
Expand Down

0 comments on commit 7684814

Please sign in to comment.