Some small fixes

[oxidase]

⬆️ Bump cpm from 0.38.2 to 0.40.2

to fix deprecation warnings

CMake Warning (dev) at /opt/homebrew/share/cmake/Modules/FetchContent.cmake:1953 (message):
  Calling FetchContent_Populate(cicd-repo-infrastructure) is deprecated, call
  FetchContent_MakeAvailable(cicd-repo-infrastructure) instead.  Policy
  CMP0169 can be set to OLD to allow
  FetchContent_Populate(cicd-repo-infrastructure) to be called directly for
  now, but the ability to call it with declared details will be removed
  completely in a future version.

🐛 Allow empty prefix in undefined symbols lines

This should allow parsing strings like

compile-time-init-build/build/test/log/CMakeFiles/catalog2_lib.dir/catalog2a_lib.cpp.o:
__Unwind_Resume
unsigned int module<sc::module_string<sc::undefined<void, char, (char)100, (char)101, (char)102, (char)97, (char)117, (char)108, (char)116>>>()
unsigned int catalog<sc::message<(logging::level)7, sc::undefined<sc::args<int, int>, char, (char)68, (char)32, (char)115, (char)116, (char)114, (char)105, (char)110, (char)103, (char)32, (char)119, (char)105, (char)116, (char)104, (char)32, (char)123, (char)125, (char)32, (char)97, (char)110, (char)100, (char)32, (char)123, (char)125, (char)32, (char)112, (char)108, (char)97, (char)99, (char)101, (char)104, (char)111, (char)108, (char)100, (char)101, (char)114>>>()
std::terminate()
___cxa_begin_catch
___gxx_personality_v0
___stack_chk_fail
___stack_chk_guard
_log_calls

compile-time-init-build/build/test/log/CMakeFiles/catalog2_lib.dir/catalog2b_lib.cpp.o:
__Unwind_Resume
unsigned int module<sc::module_string<sc::undefined<void, char, (char)100, (char)101, (char)102, (char)97, (char)117, (char)108, (char)116>>>()
unsigned int catalog<sc::message<(logging::level)7, sc::undefined<sc::args<ns::E>, char, (char)69, (char)32, (char)115, (char)116, (char)114, (char)105, (char)110, (char)103, (char)32, (char)119, (char)105, (char)116, (char)104, (char)32, (char)123, (char)125, (char)32, (char)112, (char)108, (char)97, (char)99, (char)101, (char)104, (char)111, (char)108, (char)100, (char)101, (char)114>>>()
std::terminate()
___cxa_begin_catch
___gxx_personality_v0
___stack_chk_fail
___stack_chk_guard
_log_calls

⚠️ Fix "suggest braces around initialization of subobject" warnings

Warnings are visible as errors with compiler

g++ --version
Apple clang version 16.0.0 (clang-1600.0.26.4)
Target: arm64-apple-darwin24.1.0
Thread model: posix
InstalledDir: /Library/Developer/CommandLineTools/usr/bin

[elbeno]

Thanks for the PR. The CPM update and the regex fix are fine.

The missing braces thing we need to fix differently. Adding extra braces everywhere is not easy to read, not easy to get right, and leaks implementation details. That warning is turned off upstream for clang... do we need to turn it off for GCC also?

You appear to be running g++ but it's reporting itself as "Apple clang" - I wonder if that is affecting how CMake views it. We don't have the CI infrastructure to support Apple clang, I'm afraid.

[oxidase]

@elbeno thanks! I have dropped the braces commit.

My toolchain is indeed Apple clang with was installed with brew

/usr/bin/nm --version
llvm-nm, compatible with GNU nm
Apple LLVM version 16.0.0
   (clang-1600.0.26.4)Optimized build.

/usr/bin/g++ --version
Apple clang version 16.0.0 (clang-1600.0.26.4)
Target: arm64-apple-darwin24.1.0
Thread model: posix
InstalledDir: /Library/Developer/CommandLineTools/usr/bin

This explains different output of nm and g++ warnings.

I'll check upstream warnings repo why the warning is not turned in my case.