open-source-parsers / jsoncpp

A C++ library for interacting with JSON.
Other
8.15k stars 2.64k forks source link

CMake: remove ccache babysitting #1447

Closed SpaceIm closed 1 month ago

SpaceIm commented 1 year ago

Is your feature request related to a problem? Please describe. Currently jsoncpp unconditionally tries to use ccache if found on system, so build may not be reproducible depending on build machine configuration, or even fails due to local cache corruption in ccache.

Describe the solution you'd like Completely remove ccache micro-management in CMakeLists since it's not a usage requirement of jsoncpp. There is a canonical way to enable ccache usage: set CMAKE_C_COMPILER_LAUNCHER & CMAKE_CXX_COMPILER_LAUNCHER externally, so that users have full control over ccache usage, regardless of what is installed on their system.