ndsev / zserio

zero sugar, zero fat, zero serialization overhead
https://zserio.org/
BSD 3-Clause "New" or "Revised" License
102 stars 27 forks source link

Check for minimum C++ standard version doesn't work #614

Closed Mi-La closed 1 week ago

Mi-La commented 2 weeks ago

Zserio version and language Zserio: 2.14.0 Language: C++

Describe the bug Check for minimum C++ standard doesn't work in some cases.

Expected behavior The check will fire an error when C++ standard is lower than C++11.

Additional context The check is here https://github.com/ndsev/zserio/blob/aba07d11b3b8732966d8f6a6962258e0203764db/compiler/extensions/cpp/runtime/src/CMakeLists.txt#L20-L22.

It seems that when compilers uses a default C++ standard version which is lower than C++11, the check doesn't work.

See failed builds on conan-center-index for MacOS Clang 13 and gcc 5.

Mi-La commented 2 weeks ago

See https://github.com/conan-io/conan-center-index/pull/23846#discussion_r1599519020.