edgetx icon indicating copy to clipboard operation
edgetx copied to clipboard

chore: update arm toolchain used

Open pfeerick opened this issue 1 year ago • 1 comments

Summary of changes:

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

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

pfeerick avatar Jul 07 '24 12:07 pfeerick

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

pfeerick avatar Jul 09 '24 05:07 pfeerick

Why 13.2 and not 13.3 or 14.2?

philmoz avatar Dec 25 '24 01:12 philmoz

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

gagarinlg avatar Dec 25 '24 06:12 gagarinlg

Version check should probably be updated to 14.2.1 as well.

philmoz avatar Dec 30 '24 06:12 philmoz

Compiling the libsims now report

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

elecpower avatar Dec 31 '24 22:12 elecpower

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

pfeerick avatar Dec 31 '24 22:12 pfeerick