CMAKE_CXX_STANDARD BOOL?

[N-Dekker]

.github/workflows/cxx-python.yml overrules the C++ Standard version used to compile the project, by saying -DCMAKE_CXX_STANDARD:BOOL=20:

ITK-Wasm/.github/workflows/cxx-python.yml

Line 20 in 3818ff2

itk-cmake-options: '-DCMAKE_CXX_STANDARD:BOOL=20'

Three questions:

• Why is the C++ Standard version specified as a Boolean value? That looks like a bug.
• Is it useful to specify its type anyway? Wouldn't -DCMAKE_CXX_STANDARD=20 work just fine?
• Wouldn't it be preferable to "hard-code" the C++ Standard version in the CMakeLists.txt, instead of specifying this "on the fly", at the yml/cmake command-line?

[thewtex]

That looks like a bug.

Yes 🐛 👀

Is it useful to specify its type anyway? Wouldn't -DCMAKE_CXX_STANDARD=20 work just fine?

It keeps the correct type in CMakeCache.txt, which is considered a best practice.

Wouldn't it be preferable to "hard-code" the C++ Standard version in the CMakeLists.txt, instead of specifying this "on the fly", at the yml/cmake command-line?

Yes, we are effectively doing this:

ITK-Wasm/CMakeLists.txt

Line 4 in a473cb9

set(CMAKE_CXX_STANDARD 20)

And it can be removed: #1430