Build errors?

[owenfromcanada]

I'm getting a couple of errors when I attempt to build. My compile commands are:

cmake -DARM_NONE_EABI_TOOLCHAIN_PATH="$PWD/../../comp" -DNRF5_SDK_PATH="$PWD/../../sdk" -DBUILD_DFU=1 -DBUILD_RESOURCES=1 -S .. --log-level=WARNING
make -j8 --quiet pinetime-mcuboot-app

I get one error about a debug macro in the LFS library:

/data/Projects/nrf52/InfiniTime/src/libs/littlefs/lfs.c:1812:9: error: implicit declaration of function 'LOG_INTERNAL_7'; did you mean 'LOG_INTERNAL_0'? [-Wimplicit-function-declaration]
 1812 |         LFS_DEBUG("Relocating {0x%"PRIx32", 0x%"PRIx32"} "
      |         ^~~~~~~~~

I got around that by disabling debug printing for LFS (I added add_definitions(-DLFS_NO_DEBUG) to src/CMakeLists.txt).

I also got a false positive for a dangling reference in the Navigation screen:

/data/Projects/nrf52/InfiniTime/src/displayapp/screens/Navigation.cpp:194:15: error: possibly dangling reference to a temporary [-Werror=dangling-reference]
  194 |   const auto& image = GetIcon("flag");
      |               ^~~~~
/data/Projects/nrf52/InfiniTime/src/displayapp/screens/Navigation.cpp:194:30: note: the temporary was destroyed at the end of the full expression '{anonymous}::GetIcon(std::__cxx11::basic_string<char>(((const char*)"flag"), std::allocator<char>()))'
  194 |   const auto& image = GetIcon("flag");
      |                       ~~~~~~~^~~~~~~~

I got around that by reducing the dangling reference detection to a warning (set(CXX_FLAGS -fno-rtti -Wno-dangling-reference)) in src/CMakeLists.txt.

If others aren't seeing these issues, any idea why? If they are, I can submit a PR to add these changes.

[FintasticMan]

What compiler and SDK versions are you using? Those are the biggest factors involved in variance in compiler errors and warnings.

[owenfromcanada]

The compiler version is 14.2.rel1, and the SDK is 15.3.0. I got the links for them directly from doc/buildAndProgram.md.

[FintasticMan]

Try with the older compiler version listed there: 10.3-2021.10.

[owenfromcanada]

Yeah, that was it. I'll use the older version then. Thanks!