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

chore: update GNU Arm toolchain to 14.2.rel1 #5263

Merged
merged 6 commits into from
Dec 31, 2024
Merged

Conversation

pfeerick
Copy link
Member

@pfeerick pfeerick commented Jul 7, 2024

Summary of changes:

  • update to arm-gnu-toolchain-14.2.rel1
  • bring EdgeTX out of the C++ stone age (C++17) (ok, maybe not 😆)

Bringing this up for increased visibility and testing... depends on EdgeTX/build-edgetx#23

@pfeerick pfeerick added the compilation Related to compiling the firmware and firmware options label Jul 7, 2024
radio/src/CMakeLists.txt Outdated Show resolved Hide resolved
@pfeerick pfeerick self-assigned this Jul 9, 2024
@pfeerick pfeerick force-pushed the toolchain_update branch from 85c266c to 9578252 Compare July 9, 2024 05:54
@pfeerick
Copy link
Member Author

pfeerick commented Jul 9, 2024

Tests will fail to even start if FATAL_ERROR... e.g. https://github.com/EdgeTX/edgetx/actions/runs/9851434593

@philmoz
Copy link
Collaborator

philmoz commented Dec 25, 2024

Why 13.2 and not 13.3 or 14.2?

@gagarinlg
Copy link
Member

Because that was the release that was current, when I started this a while ago. Since 14.2 is current now, I am happy to use that

@philmoz
Copy link
Collaborator

philmoz commented Dec 30, 2024

Version check should probably be updated to 14.2.1 as well.

@pfeerick pfeerick changed the title chore: update arm toolchain used chore: update GNU Arm toolchain to 14.2.rel1 Dec 31, 2024
@pfeerick pfeerick merged commit 8c5894a into main Dec 31, 2024
49 checks passed
@pfeerick pfeerick deleted the toolchain_update branch December 31, 2024 04:03
@elecpower
Copy link
Collaborator

Compiling the libsims now report

CMake Warning at radio/src/CMakeLists.txt:119 (message):
Only Arm GNU Toolchain version 14.2.rel1 is supported

@pfeerick
Copy link
Member Author

pfeerick commented Dec 31, 2024

Yeah, it is possible it is reporting different versions for different OSes... i.e. instead of 14.2.1 it looks like it is reporting v9 and 14.2.0 ... will poke around a bit more to see what is going on, otherwise Malte's version warning goes bye bye... it got changed from a fatal error to a warning 6 months ago as it is 😆

msys

CMake Warning at radio/src/CMakeLists.txt:119 (message):
  Only Arm GNU Toolchain version 14.2.rel1 is supported
TARGET simu/libsimulator: cpp compiler /usr/bin/c++ v9

linux

CMake Warning at radio/src/CMakeLists.txt:119 (message):
  Only Arm GNU Toolchain version 14.2.rel1 is supported
TARGET simu/libsimulator: cpp compiler D:/a/_temp/msys64/mingw64/bin/c++.exe v14.2.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
compilation Related to compiling the firmware and firmware options
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants