Naming convention of variables
higaski opened this issue · comments
Just a minor thing. It might be a good idea to keep consistency with CMake's naming convention regarding the FetchContent module. CMake follows a naming convention where all FetchContent output variables contain the passed dependency in lower case, so e.g.
<lowercaseName>_SOURCE_DIR
CPM.cmake takes the dependency name "as is". While I'd actually prefer that it's kinda confusing for people who don't know this detail.