How to use shake with CMake?

[ghost]

How am I supposed to use shake with CMake?

> cmake -S . -B build/shake -G "Ninja" -D CMAKE_MAKE_PROGRAM=C:/git/ItoGraphics/shake.exe
-- The CXX compiler identification is Clang 11.0.0 with GNU-like command-line
-- Check for working CXX compiler: C:/Program Files/LLVM/bin/clang++.exe
CMake Error:
  The detected version of Ninja (Shake build system, version 0.19.1) is less
  than the version of Ninja required by CMake (1.3).


CMake Error at C:/Program Files/CMake/share/cmake-3.20/Modules/CMakeTestCXXCompiler.cmake:46 (try_compile):
  Failed to generate test project build system.
Call Stack (most recent call first):
  CMakeLists.txt:7 (project)


-- Configuring incomplete, errors occurred!
>

Based on several things I thought shake was compatible with CMake. How do I get this to work?

[ndmitchell]

Thanks for the report, and sorry it's taken so long for me to look at it.

In the past, I have used Shake as a replacement for Ninja in CMake. However, I guess at that point whatever Ninja version assertion is now failing either wasn't present, or was present but Shake was high enough to trigger.

It does seem that Shake isn't a valid replacement for Ninja because of the version detection. I'm not sure what to do about that - we probably need to figure out how CMake detects the Ninja version. Once we have that, maybe we could write version 0.19.1, compilable with Ninja 1.3 or something? Perhaps a .bat script which spots --version and writes out a fake 1.3 would be sufficient to see if there are any further issues?

[ghost]

It does seem that Shake isn't a valid replacement for Ninja because of the version detection. I'm not sure what to do about that - we probably need to figure out how CMake detects the Ninja version. Once we have that, maybe we could write version 0.19.1, compilable with Ninja 1.3 or something? Perhaps a .bat script which spots --version and writes out a fake 1.3 would be sufficient to see if there are any further issues?

I guess you could do something like that for backwards compatibility. Although I think the proper fix is to upstream changes into CMake for future releases.