Skip to content
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

Add CI with TRIK toolchain #725

Merged
merged 6 commits into from
Aug 30, 2023
Merged

Conversation

WoWaster
Copy link
Contributor

@WoWaster WoWaster commented Aug 26, 2023

At some point after the TRIK 20220613 firmware release, the build using the TRIK toolchain broke.
Fix required:

  • wrapping some code for newer Python version with preprocessor directives;
  • downgrading Google Test to v1.10.0. Despite v1.12.1 being the last version to support C++11, GCC 4.9 in the TRIK toolchain has incomplete support of C++11.

To prevent this issue the addition of CI workflow with the TRIK toolchain is required. This PR adds workflow with two build configurations:

  • [TRIK brick] compiles runtime as if it was compiled for TRIK controller;
  • [QEMU + NoPython] compiles runtime for ARM without TRIK-specific hardware info and then runs test suite using QEMU. Currently, the Python engine is broken for some reason and thus must be disabled to pass the tests.

Due to GCC 4.9 in TRIK toolchain last supported Google Test version is v1.10.0
iakov
iakov previously approved these changes Aug 28, 2023
@iakov iakov added this to the 19.10 milestone Aug 28, 2023
Must be used to run ARM binaries using QEMU, otherwise the app tries to
use brick hardware which is unavailable
PyPreConfig is new feature of Python 3.8. Currently TRIK toolchain uses Python 3.5 which fails to build without guards
safe.directory option must be configured in global scope for security
reasons
@iakov iakov self-requested a review August 30, 2023 14:02
@iakov iakov merged commit b58f84d into trikset:master Aug 30, 2023
11 checks passed
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